[vtk-developers] vtkImageReslice performance questions/improvemnts

David Gobbi david.gobbi at gmail.com
Fri Sep 24 12:46:19 EDT 2010


On Fri, Sep 24, 2010 at 10:06 AM, Sean McBride <sean at rogue-research.com> wrote:
> On Thu, 23 Sep 2010 17:11:54 -0600, David Gobbi said:

>>But I should definitely unify the code between GridWarp and
>>ImageReslice.  Thanks for pointing that out.
>
> ... I mostly meant that copypasta is bad.  If this one type of rounding/
> flooring is useful in two places, it seems it should be in one single
> place.  We could give it a particular name so that users know it's not a
> general-case rounder.  ex: vtkFunkyFloor() or whatever.

I know, I know.  I hate doing copy-paste.  But I started thinking,
what header file should I add it to?  I just couldn't bring myself to
make vtkMath any more cluttered, and certainly didn't want to make a
new header file.  It was a judgement call.

  David



More information about the vtk-developers mailing list