[Lldb-commits] [PATCH] Handle path slashes properly, when the binary is compiled in Windows and used in Linux and vice-versa
Zachary Turner
zturner at google.com
Fri Jul 25 09:07:30 PDT 2014
I don't think this works. \ and / are only identical on Windows. On other
platforms \ is a character that can actually appear in a string.
Furthermore, this doesn't address the issue in FileSpec, which also has
issues dealing with Windows paths. Incidentally, I was already working on
a patch to address exactly this. I think the correct fix is to fix it in
FileSpec, and then update PathMappingList to use FileSpecs instead of
ConstStrings.
On Fri, Jul 25, 2014 at 8:09 AM, Deepak Panickal <deepak2427 at gmail.com>
wrote:
> Hi,
>
> The slashes in the paths are not currently handled properly, especially
> when the binary is compiled in Windows and used in Linux and vice-versa.
> This is also a problem for source file mapping. The patch introduces a new
> function to handle strncmp for paths independently.
>
> http://reviews.llvm.org/D4666
>
> Files:
> source/Target/PathMappingList.cpp
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20140725/2c2f7531/attachment.html>
More information about the lldb-commits
mailing list