Hi Paul,

Thanks for the changes.  I've changed:

    if (!_queryGeode.valid())

To

    if (!_queryGeode)

As ref_ptr has a built in ! operator so the valid() isn't required in
this context.

On Mon, Feb 25, 2008 at 4:33 PM, Paul Martz <[EMAIL PROTECTED]> wrote:
> No problem.
>
>  Here's the same files again, this time with the setters/getters broken out.
>  I wasn't aware that ** might be an issue for other languages.

Its a potential problem.  Also for serialization having balance
get/set methods is far better.  Most of the OSG conforms to this
approach save for a few areas in older code, or areas that really
require it.  In general for all new code I'm trying to stick with the
convention for attributes that you have getAttribute() setAttribute()
pairs, and bit by bit change the old code across which isn't
compliant.

Changes are now checked into SVN.

Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to