Hi J-S,

On Mon, Jan 17, 2011 at 3:02 PM, Jean-Sébastien Guay
<[email protected]> wrote:
> As a general comment, about the new method:
>
>   inline unsigned int getNumTextureAttributeLists() const
>       { return _textureAttributeList.size(); }
>
> The name should probably be getNumTextureAttributes(), not
> getNumTextureAttributeLists(). There is only one _textureAttributeList, and
> the method returns its size. So it's the number of texture attributes. I
> would expect getNumTextureAttributeLists() to always return 1 :-)
>
> Same thing for getNumTextureModeLists() --> getNumTextureModes().

It's not quite that straight forward... as per texture unit there is a
whole AttributeList and ModeList assigned to it, so number of texture
units != number of texture modes or texture attributes.   The name as
checked in correct, there is also already checked in an explanation.

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

Reply via email to