This is the mail archive of the gdb-patches@sourceware.org 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/commit] Improve gdb_realpath for Windows hosts


On 2011-12-27 22:12, Joel Brobecker wrote:
Can you explain that why you avoid converting to lowercase?

  /* The file system is case-insensitive but case-preserving.
     So we do not lowercase the path.  Otherwise, we might not
     be able to display the original casing in a given path.  */
I just thought that lowercase is another canonization of path, so you
can have a uniform path when you set breakpoints(file specification)
Two reasons, mostly:
   . It is unnecessary;
   . It changes the filename casing when displaying the name of
     the file where the breakpoint has been inserted.

Several Windows users at AdaCore often complain that GNU tools do not
properly preserve the filename casing, so I did not think it was
proper for us to do so. Even as a Unix user, I do feel that it is
important to preserve the casing as well.

I should say that it does not affect filename matching, which is
performed through another specialized function.

The original code in iberty library is like:
Honestly, I do not understand why they do it. There is no correlation,
IMO, between the fact that the FS is case-insensitive and the fact
that you'd want to change the filename casing (again, doing something
extra which is unnecessary, since it does not make a difference to
the filesystem).

OK, I understand now. Thank you Joel for your explanation and your time.

asmwarrior
ollydbg from codeblocks' forum


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