comphelper/source/officeinstdir/officeinstallationdirectories.cxx | 3 --- 1 file changed, 3 deletions(-)
New commits: commit 0481c2c75efb47e7ca8a106a6e9703a2c4e4f0f0 Author: Bogdan Buzea <[email protected]> AuthorDate: Fri Oct 18 20:52:22 2024 +0200 Commit: Noel Grandin <[email protected]> CommitDate: Wed Nov 6 18:52:30 2024 +0100 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 <[email protected]> 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;
