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 /sc/source/ui/unoobj | |
| 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 'sc/source/ui/unoobj')
| -rw-r--r-- | sc/source/ui/unoobj/PivotTableDataSequence.cxx | 2 | ||||
| -rw-r--r-- | sc/source/ui/unoobj/linkuno.cxx | 3 |
2 files changed, 0 insertions, 5 deletions
diff --git a/sc/source/ui/unoobj/PivotTableDataSequence.cxx b/sc/source/ui/unoobj/PivotTableDataSequence.cxx index 4acaf4dec05c..d1a3c278c6fc 100644 --- a/sc/source/ui/unoobj/PivotTableDataSequence.cxx +++ b/sc/source/ui/unoobj/PivotTableDataSequence.cxx @@ -132,8 +132,6 @@ uno::Sequence<OUString> SAL_CALL PivotTableDataSequence::getTextualData() OUString SAL_CALL PivotTableDataSequence::getSourceRangeRepresentation() { - SolarMutexGuard aGuard; - return m_aID; } diff --git a/sc/source/ui/unoobj/linkuno.cxx b/sc/source/ui/unoobj/linkuno.cxx index 63345c0c9761..8ee0910c3d13 100644 --- a/sc/source/ui/unoobj/linkuno.cxx +++ b/sc/source/ui/unoobj/linkuno.cxx @@ -1050,7 +1050,6 @@ void SAL_CALL ScDDELinkObj::setName( const OUString& /* aName */ ) OUString SAL_CALL ScDDELinkObj::getApplication() { - SolarMutexGuard aGuard; //! Test if the link is still in the document? return aAppl; @@ -1058,7 +1057,6 @@ OUString SAL_CALL ScDDELinkObj::getApplication() OUString SAL_CALL ScDDELinkObj::getTopic() { - SolarMutexGuard aGuard; //! Test if the link is still in the document? return aTopic; @@ -1066,7 +1064,6 @@ OUString SAL_CALL ScDDELinkObj::getTopic() OUString SAL_CALL ScDDELinkObj::getItem() { - SolarMutexGuard aGuard; //! Test if the link is still in the document? return aItem; |