Hi,<br><br>in vtkPlanesIntersection.cxx, around line 229, there is this loop:<br><br>    for (plane=0; plane < nplanes; plane++)<br>      {<br>      where[plane] = this->EvaluateFacePlane(plane, R);<br><br>      if (allInside &&  (where[plane] != Inside))<br>

        {<br>        allInside = 0;<br>        }<br><br>      if (where[plane] == Outside)<br>        {<br>        intersects = 0;<br><br>        break;<br>        }<br>      }<br><br><br>my question is,<br>why isn't there a break; after allInside=0 ?<br>

<br>and, why are you using the where[] array at all, since it is not referenced in any other code?  (except to alloc and delete)<br><br>If you add a break and remove the where array, we can save a tiny bit of memory, simplify the code a little, and avoid calling EvaluateFacePlane() in a bunch of situations.<br>

<br>thanks,<br>Paul<br><br>