[Lldb-commits] [PATCH] D12187: [NativeProcessLinux] Reduce the number of casts
Tamas Berghammer via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 20 04:05:58 PDT 2015
tberghammer accepted this revision.
tberghammer added a comment.
This revision is now accepted and ready to land.
Looks good.
One inconsistency what might worth a change is that you usually pass NPL as a shared_ptr (or a reference to a shared_ptr) but you created a few function where it isn't true (e.g. ThreadWasCreated). I would prefer to always pass it by shared_ptr (possibly as a reference) for consistency.
http://reviews.llvm.org/D12187
More information about the lldb-commits
mailing list