Hi,<div><br></div><div>         I did some formatting changes to the code according to vtk guidelines and added the patch given by John.  The code can be found in <a href="http://review.source.kitware.com/#change,2362">http://review.source.kitware.com/#change,2362</a>.</div>
<div><br></div><div>Some formatting still needs to be done.</div><div><br></div><div>Helper classes, vtkCellTree and vtkCellTreeNode, are made public in the current version to make them accessible by sub-classing to test for additional functions implemented in vtkModifiedBSPTree. This can be made protected after all the functions are available in vtkCellTreeLocator.</div>
<div><br></div><div>The standard template library functions are used in this class that might need to be changed as well.</div><div><br></div><div>Please let me know if you come across any other bugs or if there are  improvements that can be made to this code.</div>
<div><br></div><div>Thank you,</div><div><br></div><div>Tharindu </div><div><br><div class="gmail_quote">On Fri, Jul 29, 2011 at 4:02 PM, Biddiscombe, John A. <span dir="ltr"><<a href="mailto:biddisco@cscs.ch">biddisco@cscs.ch</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">





<div lang="EN-GB" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">I’ve fixed the crash caused by a numeric overflow. some compilers might not mind and not cause the segfault – I believe the intent was to allow the std:::numeric_limits
 max() to cause a large volume result which would exclude the bucket from consideration.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">I’d added a check to stop it. Might be worth adding a few other tests as this class is new and we have little experience with  it. I’ll add one myself I think.<u></u><u></u></span></p>

<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">patch attached<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D">JB<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;color:#1F497D"><u></u> <u></u></span></p>
</div>
</div>

<br>_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br></div>