[vtk-developers] vtkPlane non-static wrappers

[David Doria]

On Tue, Feb 2, 2010 at 10:08 AM, Bill Lorensen <bill.lorensen at gmail.com>wrote:

> This doxygen page maps classes to tests.
> http://www.vtk.org/doc/nightly/html/pages.html
>


Bill - Sure, but I'm recommending mapping classes directly to their coverage
as well.

I committed the new functions. I actually don't believe I needed to do
anything special to the normal (normalize) because the assumptions from the
static functions hold for the plane object (i.e. if the static function was
expecting a normalized 'n', the new non-static function will expect the
vtkPlane object to have a normalized normal). Please let me know if that is
incorrect.

I didn't add any new documentation as it seemed too verbose to copy the
description of the static functions and add "this is a convenience function"
so I just let the doxygen repeat the description. Any objections to this?

I haven't added any tests - I was waiting to see if we agree that I should
add TestPlane.cxx or add the tests in an existing test file.

Thanks,

David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20100202/8179745b/attachment.html>