This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]