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>
This commit is contained in:
Bogdan Buzea 2024-10-18 20:52:22 +02:00 committed by Noel Grandin
parent dd4ec48bcf
commit 0481c2c75e

View file

@ -209,9 +209,6 @@ OfficeInstallationDirectories::getSupportedServiceNames()
void OfficeInstallationDirectories::initDirs()
{
if ( m_xOfficeBrandDir)
return;
std::unique_lock aGuard( m_aMutex );
if ( m_xOfficeBrandDir )
return;