[vtk-developers] memory bloat, with patch

[David Gobbi]

It's a good change, but I found one mistake (noted in the review) so
it needs re-checking.

I did a "git blame" on vtkImageData.cxx, and it turns out that Berk
did a very similar fix there just a few months ago.  You should take a
look at what he did and keep your work consistent with his.
See commit 2e68d14f2c59c4163208baf4f935f84034776b6b

 - David


On Fri, Mar 25, 2011 at 4:14 PM, tom fogal <tfogal at sci.utah.edu> wrote:
> One of the devs I work with sent along this patch:
>
>  http://review.source.kitware.com/1259
>
> which fortunately still applied to current master.  Builds fine on
> Linux, and some spot tests indicate nothing is amiss... plus the diff
> itself is pretty straightforward.  This cuts down on our memory use in
> VisIt pretty significantly.
>
> Please review || apply.
>
> -tom
> _______________________________________________
> 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
>
>