[Lldb-commits] [PATCH] D35618: Remove shared_pointer from NativeThreadProtocol

2017-10-17 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL316007: Remove shared_pointer from NativeThreadProtocol (authored by labath). Changed prior to commit: https://reviews.llvm.org/D35618?vs=107289&id=119338#toc Repository: rL LLVM https://reviews.llv

[Lldb-commits] [PATCH] D35618: Remove shared_pointer from NativeThreadProtocol

2017-07-19 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. I don't have particular opinion. This change makes code cleaner. https://reviews.llvm.org/D35618 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D35618: Remove shared_pointer from NativeThreadProtocol

2017-07-19 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. The NativeThread class is useless without the containing process (and in some places it is already assuming the process is always around). This makes it clear that the NativeProcessProtocol is the object owning the threads, and makes the destruction order deterministi