Hi Jason,
I looks to me like the code in SVN will work, but just do a test that
isn't required i.e. to check in c:/ exists, the code reads:
if (pos_current == 2 && fullpath[1]==':')
directories.push_back(std::string(fullpath,0,
pos_current+1));
else
directories.push_back(std::string(fullpath,0, pos_current));
Note the first branch is what will detect the c: and then do a special
case for it. In your own case you've done:
if (pos_current != 2 && fullpath[1]!=':')
directories.push_back(std::string(fullpath,0, pos_current));
Which I guess has the intent of not producing the c:/ case at all,
which is equivalent to doing a not in the code in SVN i.e
if ( !(pos_current == 2 && fullpath[1]==':') )
directories.push_back(std::string(fullpath,0, pos_current));
Which if you move the ! into the nested brackect you get:
if ( pos_current != 2 || fullpath[1]!=':' )
directories.push_back(std::string(fullpath,0, pos_current));
Which is different from what you have - you have a && instead of a ||,
which makes me wonder if you are attempting to do the above, but have
just made a mistake in the logic.
Could you confirm what you are trying to achieve with your change, and
if my reading of the correct solution is appropriate? If it is I'll
check it in.
Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
