[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