Hi J.P,

Thanks for the pointer.  The code is wrong, so I've fixed it.  None of
the calling code was using the set values so the bug won't have
resulted in any downstream problems, but this doesn't mean a problem
might not have been introduced further down the line.

Robert.

On 8/17/07, J.P. Delport <[EMAIL PROTECTED]> wrote:
> Hi Robert/others,
>
> while stepping through some of my code in a debugger I stumbled across
> the following in include/osgUtil/CullVisitor. (It can easily be found by
> searching for "zn;")
>
> In both functions clampProjectionMatrix for Matrixf/d:
>
> if (result)
> {
>     znear = zn;
>     zfar = zn;
>     return true;
> }
>
> Is the setting of zfar to zn intentional? (I do not know the code good
> enough to know.) If so, maybe a comment that explains why would be in
> order?
>
> regards
> jp
>
> --
> This message is subject to the CSIR's copyright, terms and conditions and
> e-mail legal notice. Views expressed herein do not necessarily represent the
> views of the CSIR.
>
> CSIR E-mail Legal Notice
> http://mail.csir.co.za/CSIR_eMail_Legal_Notice.html
>
> CSIR Copyright, Terms and Conditions
> http://mail.csir.co.za/CSIR_Copyright.html
>
> For electronic copies of the CSIR Copyright, Terms and Conditions and the CSIR
> Legal Notice send a blank message with REQUEST LEGAL in the subject line to
> [EMAIL PROTECTED]
>
>
> This message has been scanned for viruses and dangerous content by 
> MailScanner,
> and is believed to be clean.  MailScanner thanks Transtec Computers for their 
> support.
>
> _______________________________________________
> osg-users mailing list
> osg-users@lists.openscenegraph.org
> http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
>
_______________________________________________
osg-users mailing list
osg-users@lists.openscenegraph.org
http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org

Reply via email to