Things look good. I do have questions about two methods which may really be
a factor of my becoming familiar with the lldb source code.
ProcessFreeBSD.cpp
ProcessFreeBSD::DoResume()
"resume_signal = 0"
Should this not be LLDB_INVALID_SIGNAL_NUMBER? The comments on SingleStep()
and Resume() suggest that any other value will cause that signal to be
delivered to the process.
What is the function of the "do_step = true" in the "m_suspended_tids" loop?
FreeBSD/ProcessMonitor.cpp
ProcessMonitor::GetCurrentThreadIDs()
Is it "safe" to simply "return 0" if either of the malloc() or PTRACE()
calls fail. Acknowledged that if malloc() fails, lldb is about to fall over.
Is there any instance when this PTRACE() call can fail when there are known
threads (tdcnt > 0) and the correct amount of space has been malloc'ed.
It is nit-picking, but should these not be covered by assertions?
-- John
http://llvm-reviews.chandlerc.com/D2267
_______________________________________________
lldb-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
