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 10:21:12AM -0400, Daniel Jacobowitz wrote:
>>Let me explain my concerns about using lrealpath in its current shape:
>>
>> . The Windows code in lrealpath converts all forward slashes to
>>   backslashes, which I think is a bad idea.  By contrast, the current
>>   gdb_realpath would leave existing forward slashes alone.  That is
>>   an incompatible change, and my gut feeling is that we should not
>>   risk it.  On top of that, I think we always use '/' to generate
>>   file names, so with this change we will produce ugly file names
>>   like D:\foo\bar/baz.c.
>
>The Windows maintainers disagree with you.  So, for the record, do I.

Yes.  I have to say that I can't recall a situation where slashes and
backslashes make a difference but, if there was going to be a problem,
it seems like the problem would be with the use of slashes.

>It is not worth my time to continue arguing about filename handling;
>consider the patch withdrawn indefinitely.

That is a shame.  Having two similar functions really doesn't make a
lot of sense to me.

cgf


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