STYLE: Remove continue from ParallelizeImageRegionRestrictDirection

Following C++ Core Guidelines, October 12, 2023, "Minimize the use of `break` and `continue` in loops", http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es77-minimize-the-use-of-break-and-continue-in-loops
InsightSoftwareConsortium · Dec 17, 2023 · 620b592 · 620b592
1 parent dd93015
commit 620b592
Showing 1 changed file with 8 additions and 10 deletions.
diff --git a/Modules/Core/Common/include/itkMultiThreaderBase.h b/Modules/Core/Common/include/itkMultiThreaderBase.h
@@ -378,13 +378,12 @@ ITK_GCC_PRAGMA_DIAG_POP()
       SplitRegionType splitRegion;
       for (unsigned int splitDimension = 0, dimension = 0; dimension < VDimension; ++dimension)
       {
-        if (dimension == restrictedDirection)
+        if (dimension != restrictedDirection)
         {
-          continue;
+          splitRegion.SetIndex(splitDimension, requestedRegion.GetIndex(dimension));
+          splitRegion.SetSize(splitDimension, requestedRegion.GetSize(dimension));
+          ++splitDimension;
         }
-        splitRegion.SetIndex(splitDimension, requestedRegion.GetIndex(dimension));
-        splitRegion.SetSize(splitDimension, requestedRegion.GetSize(dimension));
-        ++splitDimension;
       }
 
       this->ParallelizeImageRegion(
@@ -397,13 +396,12 @@ ITK_GCC_PRAGMA_DIAG_POP()
           restrictedRequestedRegion.SetSize(restrictedDirection, requestedRegion.GetSize(restrictedDirection));
           for (unsigned int splitDimension = 0, dimension = 0; dimension < VDimension; ++dimension)
           {
-            if (dimension == restrictedDirection)
+            if (dimension != restrictedDirection)
             {
-              continue;
+              restrictedRequestedRegion.SetIndex(dimension, index[splitDimension]);
+              restrictedRequestedRegion.SetSize(dimension, size[splitDimension]);
+              ++splitDimension;
             }
-            restrictedRequestedRegion.SetIndex(dimension, index[splitDimension]);
-            restrictedRequestedRegion.SetSize(dimension, size[splitDimension]);
-            ++splitDimension;
           }
           funcP(restrictedRequestedRegion);
         },