loplugin:flatten

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

1 files changed, 11 insertions, 11 deletions

diff --git a/starmath/source/smmod.cxx b/starmath/source/smmod.cxx
index 9404692c861d..208d6677f6c7 100644
--- a/starmath/source/smmod.cxx
+++ b/starmath/source/smmod.cxx
@@ -145,18 +145,18 @@ svtools::ColorConfig & SmModule::GetColorConfig()
 
 void SmModule::ConfigurationChanged(utl::ConfigurationBroadcaster* pBrdCst, ConfigurationHints)
 {
-    if (pBrdCst == mpColorConfig.get())
+    if (pBrdCst != mpColorConfig.get())
+        return;
+
+    SfxViewShell* pViewShell = SfxViewShell::GetFirst();
+    while (pViewShell)
     {
-        SfxViewShell* pViewShell = SfxViewShell::GetFirst();
-        while (pViewShell)
-        {
-            // FIXME: What if pViewShell is for a different document,
-            // but OTOH Math is presumably never used through
-            // LibreOfficeKit, so maybe an irrelevant concern?
-            if (dynamic_cast<const SmViewShell *>(pViewShell) != nullptr)
-                pViewShell->GetWindow()->Invalidate();
-            pViewShell = SfxViewShell::GetNext(*pViewShell);
-        }
+        // FIXME: What if pViewShell is for a different document,
+        // but OTOH Math is presumably never used through
+        // LibreOfficeKit, so maybe an irrelevant concern?
+        if (dynamic_cast<const SmViewShell *>(pViewShell) != nullptr)
+            pViewShell->GetWindow()->Invalidate();
+        pViewShell = SfxViewShell::GetNext(*pViewShell);
     }
 }