From 7ffa7e56c0202227f2ec75fce5234bb2bf89e455 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Fri, 18 Aug 2023 11:30:32 +0200 Subject: loplugin: new global analysis locking2 look for methods where we don't need to guard access to the field, because the field is never modified Change-Id: I62c33cc3f52881557515765d3733c4afc78547aa Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155836 Tested-by: Jenkins Reviewed-by: Noel Grandin --- canvas/source/cairo/cairo_canvasfont.cxx | 2 -- canvas/source/cairo/cairo_textlayout.cxx | 4 ---- 2 files changed, 6 deletions(-) (limited to 'canvas/source/cairo') diff --git a/canvas/source/cairo/cairo_canvasfont.cxx b/canvas/source/cairo/cairo_canvasfont.cxx index 82f300e8a205..2445f408853c 100644 --- a/canvas/source/cairo/cairo_canvasfont.cxx +++ b/canvas/source/cairo/cairo_canvasfont.cxx @@ -116,8 +116,6 @@ namespace cairocanvas rendering::FontRequest SAL_CALL CanvasFont::getFontRequest( ) { - SolarMutexGuard aGuard; - return maFontRequest; } diff --git a/canvas/source/cairo/cairo_textlayout.cxx b/canvas/source/cairo/cairo_textlayout.cxx index ff12a6092c13..31ef31fa7a49 100644 --- a/canvas/source/cairo/cairo_textlayout.cxx +++ b/canvas/source/cairo/cairo_textlayout.cxx @@ -244,8 +244,6 @@ namespace cairocanvas sal_Int8 SAL_CALL TextLayout::getMainTextDirection( ) { - std::unique_lock aGuard( m_aMutex ); - return mnTextDirection; } @@ -258,8 +256,6 @@ namespace cairocanvas rendering::StringContext SAL_CALL TextLayout::getText( ) { - std::unique_lock aGuard( m_aMutex ); - return maText; } -- cgit