This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: RFA: Use lrealpath instead of gdb_realpath
On Sun, May 29, 2005 at 02:27:26PM -0400, Daniel Jacobowitz wrote:
>On Sun, May 29, 2005 at 09:09:25PM +0300, Eli Zaretskii wrote:
>> Disagree with what part? that backslashes might cause trouble or that
>> D:\foo\bar/baz.c is ugly?
>
>The former part. Sure, the latter is ugly; that's a cosmetic bug in
>whatever does the concatenation, as far as I'm concerned. Or you can
>pass the result back to lrealpath again and get a properly formatted
>filename for the system. Not a bad idea to do that anyway. Or
>to xfullpath, which might need some updated logic to support this.
>
>> > It is not worth my time to continue arguing about filename handling;
>> > consider the patch withdrawn indefinitely.
>>
>> My suggestion is to fix lrealpath so that it (a) converts all
>> backslashes to forward slashes, (b) does NOT downcase the result, and
>> (c) returns its argument unaltered if the absolute file name does not
>> point to an existing file. When modified like that, I think lrealpath
>> will work the same on all platforms; as for strdup vs xstrdup part, we
>> will need to change the code in GDB to test for a NULL return value.
>>
>> If this alternative is acceptable to other maintainers, I can
>> volunteer to produce a patch along these lines.
>
>If you want to do the work, I invite you to talk to the Windows
>maintainers about it; which would mean Chris and Danny Smith, who I do
>not believe reads this list. I doubt they'd be interested in (a),
>since Chris obviously disagrees with you about backslashes. But you
>might be able to reach agreements on (b) and (c).
I wouldn't care about (a) or (b) but I'm not sure that c would meet the
criteria of whatever else is using lrealpath.
Personally, I prefer seeing lower case filenames always but I wouldn't
that's not a deal breaker.
cgf