[Lldb-commits] [PATCH] D100262: [lldb] [gdb-remote client] Support switching PID along with TID

Michał Górny via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jul 2 12:10:44 PDT 2021


mgorny marked an inline comment as done.
mgorny added a comment.

I'll do one more test run and push, thanks!



================
Comment at: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.h:588-590
+  lldb::pid_t m_curr_pid =
+      LLDB_INVALID_PROCESS_ID; // Current gdb remote protocol process identifier
+                               // for all other operations
----------------
JDevlieghere wrote:
> Can we please make these doxygen comments `///` and put them above the variable? The trailing comments really don't work well with the 80-col limit. 
Sure, done.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100262/new/

https://reviews.llvm.org/D100262



More information about the lldb-commits mailing list