From 1fb0fea4dcfa6646d6c25232feaf03ca3d64d30b Mon Sep 17 00:00:00 2001
From: P-A Loizeau <p.-a.loizeau@gsi.de>
Date: Mon, 8 Nov 2021 18:13:14 +0100
Subject: [PATCH] Fix if-else brackets warning in mCBM check overlap macro

---
 macro/mcbm/mcbm_check_overlaps.C | 21 ++++++++++-----------
 1 file changed, 10 insertions(+), 11 deletions(-)

diff --git a/macro/mcbm/mcbm_check_overlaps.C b/macro/mcbm/mcbm_check_overlaps.C
index d5b9b61e52..7caeda955f 100644
--- a/macro/mcbm/mcbm_check_overlaps.C
+++ b/macro/mcbm/mcbm_check_overlaps.C
@@ -45,17 +45,16 @@ void mcbm_check_overlaps(const char* dataset = "test")
       std::cout << ov->GetTitle() << std::endl << std::endl;
     }
     // overlap of T0 detector with pipe vacuum geometry
-    else if (OverlapName.Contains("vacu20_1 overlapping"))
-      if (OverlapName.Contains("module_5_0")) {
-        std::cout << "Expected Overlap between pipevacuum and T0 counter" << endl;
-        std::cout << ov->GetTitle() << std::endl << std::endl;
-      }
-      else {
-        cout << "Unexpected Overlap:" << endl;
-        ov->PrintInfo();
-        cout << endl;
-        unexpectedOverlaps++;
-      }
+    else if (OverlapName.Contains("vacu20_1 overlapping") && OverlapName.Contains("module_5_0")) {
+      std::cout << "Expected Overlap between pipevacuum and T0 counter" << endl;
+      std::cout << ov->GetTitle() << std::endl << std::endl;
+    }
+    else {
+      cout << "Unexpected Overlap:" << endl;
+      ov->PrintInfo();
+      cout << endl;
+      unexpectedOverlaps++;
+    }
   }
   std::cout << std::endl;
 
-- 
GitLab