Hui added inline comments.
================
Comment at: source/Plugins/Process/Windows/Common/ProcessDebugger.cpp:169-170
+
+Status ProcessDebugger::AttachProcess(lldb::pid_t pid,
+ const ProcessAttachInfo &attach_info,
+ DebugDelegateSP delegate) {
----------------
labath wrote:
> BTW, looking at the other patch, I just realized that this api is extremely
> redundant:
> a) `ProcessAttachInfo` already contains a `pid` field so passing it
> separately makes no sense
> b) `DebuggerThread` does not seem to be doing anything with the
> ProcessAttachInfo struct anyway.
I once checked the pid shipped in attach_info. It was LLDB_INVALID_PROCESS_ID.
Maybe it is an issue to fix.
Repository:
rLLDB LLDB
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63166/new/
https://reviews.llvm.org/D63166
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
