[vtk-developers] vtkMath::Dot2D

Bill Lorensen bill.lorensen at gmail.com
Wed Dec 9 20:40:06 EST 2009


Make sure all of vtk compiles after the change. If it requires changes
to any vtk code you can be sure other people will have problems.

On Wed, Dec 9, 2009 at 6:35 PM, David Doria <daviddoria+vtk at gmail.com> wrote:
> On Wed, Dec 9, 2009 at 6:11 PM, Moreland, Kenneth <kmorel at sandia.gov> wrote:
>> My guess would be that that is a copy-paste error.  Change it to 2-vectors
>> is probably the right thing to do.
>>
>> AFAIK, the array declaration in a function/method argument is treated just
>> like a pointer.  C++ does not do any range checking.  I think it just serves
>> to help document what size array you expect.
>>
>> -Ken
>
> This actually seems to be the case with almost all of the 2D vtkMath
> functions. I'm going to change them all as Ken suggested. Any
> objections?
>
> Thanks,
>
> David
> _______________________________________________
> 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
>
>



More information about the vtk-developers mailing list