[lldb-dev] Minors fixes
Jean-Daniel Dupas
devlists at shadowlab.org
Tue Jul 6 13:04:53 PDT 2010
Here are 2 patches to make clang++ happy.
Explicitly use lldb::pid_t instead of letting the compiler choose between ::pid_t and lldb::pid_t.
In fact, most pid_t declarations already use the lldb:: prefix except the 3 updated by this patch.
It fixes a clang++ error which didn't know which type to choose between ::pid_t and lldb::pid_t.
The second patch rename an argument in a method declaration to avoid duplicate names (something else that clang++ really doesn't like).
I don't know if these errors are due to clang++ bugs or just because clang++ is stricter than GCC.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pid.patch
Type: application/octet-stream
Size: 1379 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20100706/47e31416/attachment.obj>
-------------- next part --------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: args.patch
Type: application/octet-stream
Size: 714 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20100706/47e31416/attachment-0001.obj>
-------------- next part --------------
-- Jean-Daniel
More information about the lldb-dev
mailing list