<div dir="ltr">Hi Robert,<div><br></div><div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-size:12.8000001907349px">I am little concerned that some end user code will be using these<br></span><span style="font-size:12.8000001907349px">intersects methods and working around their inconsistency, so if we<br></span><span style="font-size:12.8000001907349px">fix them then we could end up breaking end user code.</span></blockquote><div class="" style="font-size:12.8000001907349px"></div><div><br></div><div>I am one of those users now ;-). However, I adopt easily. Others may not notice the change, though. Perhaps better solution would be to leave the functionality as is and clearly rename and/or comment the r1,r2 params to (rFromStart rFromEnd ?) so that inconsistency is clear and does not surprise future users ?</div><div><br></div><div>Cheers,</div><div>Wojtek</div><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-04-27 13:28 GMT+02:00 Robert Osfield <span dir="ltr"><<a href="mailto:robert.osfield@gmail.com" target="_blank">robert.osfield@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Wojtek,<br>
<br>
On 27 April 2015 at 12:15, Wojciech Lewandowski<br>
<span class=""><<a href="mailto:w.p.lewandowski@gmail.com">w.p.lewandowski@gmail.com</a>> wrote:<br>
> I believe both can be correct but it looks like in Box case r1 is ratio of<br>
> segment length measured from start and r2 measured backwards from the<br>
> segment end. For Sphere both r1 and r2 are measured from start. So here is<br>
> the inconsistency...<br>
<br>
</span>This is my assessment too.<br>
<br>
I have #if def'd out the LineSegment::intersect(const BoundingBox&<br>
bb,float& r1,float& r2) style methods from LineSegment and have been<br>
able to compile the whole OSG, so it looks like these methods have<br>
been written but not used and tested by the OSG itself so the errors<br>
haven't been picked up.<br>
<br>
This leaves us with deciding what to do with these erroneous methods.<br>
One route is to remove them, another is to change their behaviour so<br>
it's consistent and document this change.  To make the method<br>
consistent I feel that they should return the ratio between the start<br>
and end points, measured from the start.<br>
<br>
I am little concerned that some end user code will be using these<br>
intersects methods and working around their inconsistency, so if we<br>
fix them then we could end up breaking end user code.<br>
<span class="HOEnZb"><font color="#888888"><br>
Robert.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" target="_blank">http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org</a><br>
</div></div></blockquote></div><br></div>