diff options
| author | Bogdan Buzea <buzea.bogdan@libreoffice.org> | 2024-10-18 20:52:22 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2024-11-06 18:52:30 +0100 |
| commit | 0481c2c75efb47e7ca8a106a6e9703a2c4e4f0f0 (patch) | |
| tree | 18f409bf88382eace70e21aea95c5c266decc822 /comphelper/source | |
| parent | dd4ec48bcfc9ac2d6fd419b84650345e6f28e261 (diff) | |
tdf#163486: PVS: Redundant code.
V649 There are two 'if' statements with identical conditional expressions. The first 'if' statement contains function return. This means that the second 'if' statement is senseless. Check lines: 212, 216.
Change-Id: Id596419cc3134355904b4ca93a44c017ffa6635c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175153
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'comphelper/source')
| -rw-r--r-- | comphelper/source/officeinstdir/officeinstallationdirectories.cxx | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/comphelper/source/officeinstdir/officeinstallationdirectories.cxx b/comphelper/source/officeinstdir/officeinstallationdirectories.cxx index 72ef00e82f0b..c59d80501ddf 100644 --- a/comphelper/source/officeinstdir/officeinstallationdirectories.cxx +++ b/comphelper/source/officeinstdir/officeinstallationdirectories.cxx @@ -209,9 +209,6 @@ OfficeInstallationDirectories::getSupportedServiceNames() void OfficeInstallationDirectories::initDirs() { - if ( m_xOfficeBrandDir) - return; - std::unique_lock aGuard( m_aMutex ); if ( m_xOfficeBrandDir ) return; |