On Fri, 2006-11-03 at 13:20 +0000, Roger James wrote:
> //### check if we really have a texture
> if ((tex == NULL) || (!cot->getTexture()))
> {
> 
> cot->getTexture will always return NULL  because the texture if any has been
> set on the <diffuse> element.

Yes I think this check it wrong because it's testing cot (variable
reuse) that has been loaded with emission or ambient and that has no
bearing on diffuse processing. The overall assumption in the code is
that the OSG texture is bound to the diffuse channel.

> I guess it is back to reading RFCs again to see if % encoding should be
> being done/undone.

It should be decoded at least at the point of use, ie to access a file,
if not sooner as part of the conversion from URI to native path.

Regards,
Marcus


_______________________________________________
osg-users mailing list
[email protected]
http://openscenegraph.net/mailman/listinfo/osg-users
http://www.openscenegraph.org/

Reply via email to