loplugin:indentation improve checks for brace alignment

Change-Id: I333100fda7e181f68f36b03279b3fbb8cb768310
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117615
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

3 files changed, 9 insertions, 12 deletions

diff --git a/framework/source/jobs/jobdata.cxx b/framework/source/jobs/jobdata.cxx
index 6013812e0e7f..1101d77a94da 100644
--- a/framework/source/jobs/jobdata.cxx
+++ b/framework/source/jobs/jobdata.cxx
@@ -468,10 +468,10 @@ bool JobData::hasCorrectContext(const OUString& rModuleIdent) const
     {
         sal_Int32 nIndex = m_sContext.indexOf( rModuleIdent );
         if ( nIndex >= 0 && ( nIndex+nModuleIdLen <= nContextLen ))
-    {
-        OUString sContextModule = m_sContext.copy( nIndex, nModuleIdLen );
-        return sContextModule == rModuleIdent;
-    }
+        {
+            OUString sContextModule = m_sContext.copy( nIndex, nModuleIdLen );
+            return sContextModule == rModuleIdent;
+        }
     }
 
     return false;
diff --git a/framework/source/layoutmanager/layoutmanager.cxx b/framework/source/layoutmanager/layoutmanager.cxx
index 917e4182e1e2..a717de1fa00f 100644
--- a/framework/source/layoutmanager/layoutmanager.cxx
+++ b/framework/source/layoutmanager/layoutmanager.cxx
@@ -1626,8 +1626,8 @@ sal_Bool SAL_CALL LayoutManager::requestElement( const OUString& rResourceURL )
 
         if ( pToolbarManager && bComponentAttached )
         {
-                bNotify   = pToolbarManager->requestToolbar( rResourceURL );
-            }
+            bNotify   = pToolbarManager->requestToolbar( rResourceURL );
+        }
     }
     else if ( aElementType.equalsIgnoreAsciiCase("dockingwindow"))
     {
diff --git a/framework/source/recording/dispatchrecorder.cxx b/framework/source/recording/dispatchrecorder.cxx
index 70b6b9f41f2f..71639beb0bb5 100644
--- a/framework/source/recording/dispatchrecorder.cxx
+++ b/framework/source/recording/dispatchrecorder.cxx
@@ -392,9 +392,8 @@ sal_Int32 SAL_CALL DispatchRecorder::getCount()
 
 css::uno::Any SAL_CALL DispatchRecorder::getByIndex(sal_Int32 idx)
 {
-    if (idx >= static_cast<sal_Int32>(m_aStatements.size())) {
+    if (idx >= static_cast<sal_Int32>(m_aStatements.size()))
         throw css::lang::IndexOutOfBoundsException( "Dispatch recorder out of bounds"  );
-   }
 
     Any element(&m_aStatements[idx],
         cppu::UnoType<css::frame::DispatchStatement>::get());
@@ -411,12 +410,10 @@ void SAL_CALL DispatchRecorder::replaceByIndex(sal_Int32 idx, const css::uno::An
                           Reference< XInterface >(), 2 );
     }
 
-    if (idx >= static_cast<sal_Int32>(m_aStatements.size())) {
-                throw css::lang::IndexOutOfBoundsException(
+    if (idx >= static_cast<sal_Int32>(m_aStatements.size()))
+        throw css::lang::IndexOutOfBoundsException(
                         "Dispatch recorder out of bounds"  );
 
-        }
-
     auto pStatement = o3tl::doAccess<css::frame::DispatchStatement>(element);
 
     css::frame::DispatchStatement aStatement(