diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-08-18 11:30:32 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2023-08-19 17:40:47 +0200 |
| commit | 7ffa7e56c0202227f2ec75fce5234bb2bf89e455 (patch) | |
| tree | af56d098ea2fcaae7383eb5ff6b6bc5f67a4a131 /canvas/source/cairo | |
| parent | 5baac4e53128d3c0fc73b9918dc9a9c2777ace08 (diff) | |
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 <noel.grandin@collabora.co.uk>
Diffstat (limited to 'canvas/source/cairo')
| -rw-r--r-- | canvas/source/cairo/cairo_canvasfont.cxx | 2 | ||||
| -rw-r--r-- | canvas/source/cairo/cairo_textlayout.cxx | 4 |
2 files changed, 0 insertions, 6 deletions
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; } |