Re: [Lldb-commits] [PATCH] D12237: [NativeProcessLinux] Pass around threads by reference

2015-08-24 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL245831: [NativeProcessLinux] Pass around threads by reference (authored by labath). Changed prior to commit: http://reviews.llvm.org/D12237?vs=32824&id=32937#toc Repository: rL LLVM http://reviews.l

Re: [Lldb-commits] [PATCH] D12237: [NativeProcessLinux] Pass around threads by reference

2015-08-21 Thread Tamas Berghammer via lldb-commits
tberghammer accepted this revision. tberghammer added a comment. This revision is now accepted and ready to land. Looks good (I assume when you dropped const from the argument it was intentional) http://reviews.llvm.org/D12237 ___ lldb-commits mail

[Lldb-commits] [PATCH] D12237: [NativeProcessLinux] Pass around threads by reference

2015-08-21 Thread Pavel Labath via lldb-commits
labath created this revision. labath added a reviewer: tberghammer. labath added a subscriber: lldb-commits. Most NPL private functions took (shared) pointers to threads as arguments. This meant that the callee could not be sure if the pointer was valid and so most functions were peppered with n