<div class="gmail_quote">On Mon, Feb 1, 2010 at 10:54 PM, Francois Bertel <span dir="ltr"><<a href="mailto:francois.bertel@kitware.com" target="_blank">francois.bertel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

That would be more object-oriented for sure. Go for it.<br>
<br>
Just be careful that you cannot just pass the normal of the object to<br>
the static method as I don't see in the documentation of the class any<br>
class invariant stating that the magnitude of the normal has to be 1.<br>
On the contrary, the static method DistanceToPlane() explicitly has<br>
the magnitude of the normal to be 1  as a pre-condition.<br>
<div><div></div><div></div></div></blockquote></div><br><div><br></div><div>I did notice that some of them require magnitude 1 normals. The only one I plan to add a wrapper for that doesn't mention that is:</div><div>
<br></div><div><span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, Geneva, Arial, sans-serif; font-size: 12px; font-weight: bold; white-space: nowrap; -webkit-border-horizontal-spacing: 2px; -webkit-border-vertical-spacing: 2px; ">static int vtkPlane::IntersectWithLine</span></div>
<div><span class="Apple-style-span" style="font-family: 'Lucida Grande', Verdana, Geneva, Arial, sans-serif; font-size: 12px; font-weight: bold; white-space: nowrap; -webkit-border-horizontal-spacing: 2px; -webkit-border-vertical-spacing: 2px; "><br>
</span></div><div><font class="Apple-style-span" face="'Lucida Grande', Verdana, Geneva, Arial, sans-serif" size="3"><span class="Apple-style-span" style="font-size: 12px; white-space: nowrap; -webkit-border-horizontal-spacing: 2px; -webkit-border-vertical-spacing: 2px;"><b><span class="Apple-style-span" style="font-family: arial; font-weight: normal; white-space: normal; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; font-size: small; ">anyone know off the top of their head if this algorithm requires a unit normal?</span></b></span></font></div>
<div><br></div><div>Also, I was going to add tests for the new functions, but I noticed there is not a VTK/Common/Cxx/Testing/TestPlane.cxx. However, the coverage seems fine:</div><div><br></div><div><a href="http://www.cdash.org/CDash/viewCoverageFile.php?buildid=529419&fileid=10953777">http://www.cdash.org/CDash/viewCoverageFile.php?buildid=529419&fileid=10953777</a></div>
<div><br></div><div>Where would this be getting tested from? Should we make a TestPlane.cxx?</div><div><br clear="all">Thanks,<br><br>David</div>