<div dir="ltr">+greg.  Any chance you guys are willing to create Phabricator accounts?  Even if you don't use it to submit reviews, it's nice to just be able to type your name into the website and have it include you in the original email automatically.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 19, 2014 at 1:51 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">* Removes GetState / SetState<br>
* Adds a method IsJoinable()<br>
<br>
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.<br>
<br>
<a href="http://reviews.llvm.org/D5417" target="_blank">http://reviews.llvm.org/D5417</a><br>
<br>
Files:<br>
  include/lldb/Core/Debugger.h<br>
  include/lldb/Host/HostNativeThreadBase.h<br>
  include/lldb/Host/HostThread.h<br>
  include/lldb/Host/posix/HostThreadPosix.h<br>
  include/lldb/Target/Process.h<br>
  include/lldb/lldb-private-enumerations.h<br>
  source/Core/Communication.cpp<br>
  source/Core/Debugger.cpp<br>
  source/Host/common/HostNativeThreadBase.cpp<br>
  source/Host/common/HostThread.cpp<br>
  source/Host/posix/HostThreadPosix.cpp<br>
  source/Host/windows/HostThreadWindows.cpp<br>
  source/Plugins/Process/Linux/NativeProcessLinux.cpp<br>
  source/Plugins/Process/Linux/ProcessMonitor.cpp<br>
  source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp<br>
  source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp<br>
  source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp<br>
  source/Target/Process.cpp<br>
  tools/lldb-gdbserver/lldb-gdbserver.cpp<br>
</blockquote></div><br></div>