[vtk-developers] Cell Tree Locator Update

[Tharindu De Silva]

Thanks John for fixing this.

I will make some changes to this class to meet with vtk coding guidelines
and submit to gerrit.

On Fri, Jul 29, 2011 at 4:02 PM, Biddiscombe, John A. <biddisco at cscs.ch>wrote:

>  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.****
>
> ** **
>
> 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.****
>
> ** **
>
> patch attached****
>
> ** **
>
> JB****
>
> ** **
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20110729/1a77a9db/attachment.html>