Hi Gerrit,

Gerrit Voss wrote:

Hi,

On Wed, 2005-05-25 at 12:48 +0200, Christoph Fuenfzig wrote:
Hi Andreas, hi Dirk,

Andreas Zieringer wrote:

Hi Christoph,

this patch works for windows but that's a bad idea for unix. Splitting something like "/usr/lib/...:/usr/bin/..." wouldn't work anymore. The problem is that the analysePath() method thinks this a unix path and tries to split it with ":" and not ";".
I try to fix the analysePath() method.

Andreas
Of course, you are right. That was a bad try.
A special case for absolute WIN32 paths with UNIX directory separators, like "d:/Model/gotsman/horse.wrl"

the small problem might be, that this is undecidable without external
knowledge inside the path handler.

That is not quite correct. You know you are running on WIN32, and then the established format for path lists
is usually "d:\;d:\Model" and "d:/;d:/Model" would be just for comfort.

I would put the fix inside findFile, where you know what you get is
one element and work from there. AnalysePath is most likely the wrong
place without and might break existing setups.

Of course, that is always possible. But PathHandler is already quite complex, so I thought it
is intended to offer such flexibility.

This still has the problem with adding a path list but it should not
break anything.

regards,
 gerrit

PS: just curios, is the D:/xxx/xx path the result from a file open
   dialog ??
No, not in my applications using FOX, as FOX already converts paths to the format with '\'. Lets say, it is a request of an external user ;-) using testprograms on the commandline and I also
remembered my personal experience with this
"

It's a standard way of representing a path. If you look at the way both QT

and boost::filesystem represent a path, "d:/Model/gotsman/horse.wrl" is a
perfectly valid path. It seems a bit strange to me that this does not work
by default. I'm not forcing anyone to change OpenSG's behaviour. I know my
work-around with replacing '/' by '\' works but I found it quite strange
and a bit disturbing that these paths did not work out of the box. Just my
2c.
"


Of course, it is not worth breaking anything ofr that.

Regards,
Christoph



-------------------------------------------------------
This SF.Net email is sponsored by Yahoo.
Introducing Yahoo! Search Developer Network - Create apps using Yahoo!
Search APIs Find out how you can build Yahoo! directly into your own
Applications - visit http://developer.yahoo.net/?fr=offad-ysdn-ostg-q22005
_______________________________________________
Opensg-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensg-users

Reply via email to