On 20 Sep 2002 at 6:48, Frank Heckenbach wrote:
Prof Abimbola Olowofoyeku wrote:
[...]
The attached diff seems to solve the problem. I can release a patched version of gpc.exe if it is deemed that the patch is appropriate for inclusion in GPC (I hope also that the C code is correct!).
The code looks correct (though a C wizard would do it differently ;-). However, I'm not sure if converting forth and back it so good.
No, I don't like it either.
Did you find the place where it changes back slashes to front slashes? I suppose it's the places where something is set to DIR_SEPARATOR in gpc.c.
There are so many instances of DIR_SEPARAT* in gpc.c, and quote often, "/" is also checked for in such places. It is difficult to know where exactly the change is taking place.
However, config/i386/xm-mingw32.h says:
#define DIR_SEPARATOR '\' #define DIR_SEPARATOR_2 '/'
Yes, that is correct.
So I still don't understand it. DIR_SEPARATOR should be correct
It is correct, AFAICS.
(unlike DIR_SEPARATOR_2), while Cygwin has them vice versa (which should also be good).
Anyway, you might want to try changing the assignments (search for ` = DIR_SEPARATOR', and also the declaration of dir_separator_str) to DIR_SEPARATOR_2, without applying your patch.
I don't really see why this should work, but if it does, at least I know what we're at ...
That was my first point of call, and nothing I tried there worked. I will have another go over the weekend ....
Best regards, The Chief -------- Prof. Abimbola A. Olowofoyeku (The African Chief) web: http://www.bigfoot.com/~African_Chief email: African_Chief@bigfoot.com