Commit 8f0a469e authored by Sean McBride's avatar Sean McBride

Fixed dead store warning by removing redundant 'break'

analyzer warned that setting 'nextPass' was never read afterwards,
which was true because there was a 'break' right after.  So just remove
the 'break' and allow the already-commented 'nextPass' logic to end the loop.
Changed 'nextPass' to bool also.

Change-Id: Iff85503f0235348a6c0d3ad9b0406cd96f1c4e35
parent 832f091a
......@@ -257,7 +257,7 @@ void vtkPixelExtent::Merge(deque<vtkPixelExtent> &exts)
// if a merger occurs the merged extent is added
// as a new target with the constituents marked empty
// and the current pass is terminated early
int nextPass = 0;
bool nextPass = false;
// current target
vtkPixelExtent &ext0 = tmpExts[t];
......@@ -303,8 +303,7 @@ void vtkPixelExtent::Merge(deque<vtkPixelExtent> &exts)
ext1.Clear();
// move to next target
nextPass = 1;
break;
nextPass = true;
}
}
if (nextPass)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment