[Plib-devel] texture filenames messed up

[Bram S. <br...@sa...>]

Hi,

The issue described here:
http://sourceforge.net/mailarchive/message.php?msg_id=1866706
and here:
http://sourceforge.net/mailarchive/message.php?msg_id=9644979

...may very well be caused by the fact that the code for
static ssgState *get_state ( _ssgMaterial *mat )
in ssg/ssgLoadAC.cxx contains unreachable code.

There are two clauses in the form of
   if (current_tfname != NULL) {
from which the 1st one will always return.
Therefore, the body of the 2nd clause is never executed.
This will skip the crucial st->setTexture()

This results in ssg trees, created from ac, to
incorrectly use old texture filenames.

You can use plibconvert.cxx, posted at ppe devel list
to trigger the bug.

To view ssgLoadAC.cxx, you can look here:
http://cvs.sourceforge.net/viewcvs.py/plib/plib/src/ssg/ssgLoadAC.cxx?rev=1.33&view=markup

I must say it is quite a hassle to follow the trail of textures in the system,
from ssgLoadAC to ssgLoad, to ssgRenderOptions, to ssgTextureArray, to 
ssgTexture with a detour over ssgState. Somewhere along this complicated
path, the ball was dropped.

   Bram