From b15bc26a63cb43e14867c908d01152709b43e5e2 Mon Sep 17 00:00:00 2001
From: Uwe Kindler <uwe_kindler@web.de>
Date: Sun, 12 Dec 2021 17:28:11 +0100
Subject: [PATCH] Some small coding style fixes

---
 src/DockManager.cpp | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/DockManager.cpp b/src/DockManager.cpp
index e658ce9..f22fa4b 100644
--- a/src/DockManager.cpp
+++ b/src/DockManager.cpp
@@ -759,7 +759,6 @@ void CDockManager::showEvent(QShowEvent *event)
 
 	// Fix Issue #380
 	restoreHiddenFloatingWidgets();
-
 	if (d->UninitializedFloatingWidgets.empty())
 	{
 		return;
@@ -777,8 +776,15 @@ void CDockManager::showEvent(QShowEvent *event)
 	d->UninitializedFloatingWidgets.clear();
 }
 
+
+//============================================================================
 void CDockManager::restoreHiddenFloatingWidgets()
 {
+	if (d->HiddenFloatingWidgets.isEmpty())
+	{
+		return;
+	}
+
 	// Restore floating widgets that were hidden upon hideManagerAndFloatingWidgets
 	for (auto FloatingWidget : d->HiddenFloatingWidgets)
 	{
@@ -788,17 +794,19 @@ void CDockManager::restoreHiddenFloatingWidgets()
 		// Could make sense to move this to CFloatingDockContainer::showEvent(QShowEvent *event)
 		// if experiencing CFloatingDockContainer being shown empty in other situations, but let's keep
 		// it here for now to make sure changes to fix Issue #380 does not impact existing behaviours
-		for ( auto dockWidget : FloatingWidget->dockWidgets() )
+		for (auto dockWidget : FloatingWidget->dockWidgets())
 		{
-			if ( dockWidget->toggleViewAction()->isChecked() )
+			if (dockWidget->toggleViewAction()->isChecked())
 			{
 				dockWidget->toggleView(true);
 				hasDockWidgetVisible = true;
 			}
 		}
 
-		if ( hasDockWidgetVisible )
+		if (hasDockWidgetVisible)
+		{
 			FloatingWidget->show();
+		}
 	}
 
 	d->HiddenFloatingWidgets.clear();