[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