Looks good. > On Sep 19, 2014, at 1:51 PM, Zachary Turner <ztur...@google.com> wrote: > > * Removes GetState / SetState > * Adds a method IsJoinable() > > There is no need for a distinction between IsHandleValid and IsThreadRunning, > because even if a thread has run to completion, you are still required to > Join on it. So a better function simply checks whether a thread is joinable. > This is equivalent to checking if the handle is valid, because the Join() > operation can reset the handle to an invalid handle after it has been > performed. > > http://reviews.llvm.org/D5417 > > Files: > include/lldb/Core/Debugger.h > include/lldb/Host/HostNativeThreadBase.h > include/lldb/Host/HostThread.h > include/lldb/Host/posix/HostThreadPosix.h > include/lldb/Target/Process.h > include/lldb/lldb-private-enumerations.h > source/Core/Communication.cpp > source/Core/Debugger.cpp > source/Host/common/HostNativeThreadBase.cpp > source/Host/common/HostThread.cpp > source/Host/posix/HostThreadPosix.cpp > source/Host/windows/HostThreadWindows.cpp > source/Plugins/Process/Linux/NativeProcessLinux.cpp > source/Plugins/Process/Linux/ProcessMonitor.cpp > source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp > source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp > source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp > source/Target/Process.cpp > tools/lldb-gdbserver/lldb-gdbserver.cpp > <D5417.13889.patch>_______________________________________________ > lldb-commits mailing list > lldb-commits@cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
_______________________________________________ lldb-commits mailing list lldb-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits