[Lldb-commits] [PATCH] D96656: [lldb] Remove unused bool return type of ThreadPlan::WillStop (NFC)

Dave Lee via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sat Feb 13 14:22:51 PST 2021


kastiglione created this revision.
kastiglione added a reviewer: jingham.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

The calls to `ThreadPlan::WillStop` do not use the return value. This changes `WillStop`
to a `void` return type, and since most subclasses don't implement this event, the base
class is given an empty implementation, instead of being pure virtual.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D96656

Files:
  lldb/include/lldb/Target/ThreadPlan.h
  lldb/include/lldb/Target/ThreadPlanBase.h
  lldb/include/lldb/Target/ThreadPlanCallFunction.h
  lldb/include/lldb/Target/ThreadPlanCallOnFunctionExit.h
  lldb/include/lldb/Target/ThreadPlanPython.h
  lldb/include/lldb/Target/ThreadPlanRunToAddress.h
  lldb/include/lldb/Target/ThreadPlanStepInstruction.h
  lldb/include/lldb/Target/ThreadPlanStepOut.h
  lldb/include/lldb/Target/ThreadPlanStepOverBreakpoint.h
  lldb/include/lldb/Target/ThreadPlanStepRange.h
  lldb/include/lldb/Target/ThreadPlanStepThrough.h
  lldb/include/lldb/Target/ThreadPlanStepUntil.h
  lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.cpp
  lldb/source/Plugins/LanguageRuntime/ObjC/AppleObjCRuntime/AppleThreadPlanStepThroughObjCTrampoline.h
  lldb/source/Target/ThreadPlan.cpp
  lldb/source/Target/ThreadPlanBase.cpp
  lldb/source/Target/ThreadPlanCallFunction.cpp
  lldb/source/Target/ThreadPlanCallOnFunctionExit.cpp
  lldb/source/Target/ThreadPlanPython.cpp
  lldb/source/Target/ThreadPlanRunToAddress.cpp
  lldb/source/Target/ThreadPlanStepInstruction.cpp
  lldb/source/Target/ThreadPlanStepOut.cpp
  lldb/source/Target/ThreadPlanStepOverBreakpoint.cpp
  lldb/source/Target/ThreadPlanStepRange.cpp
  lldb/source/Target/ThreadPlanStepThrough.cpp
  lldb/source/Target/ThreadPlanStepUntil.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D96656.323575.patch
Type: text/x-patch
Size: 13660 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20210213/108c6877/attachment-0001.bin>


More information about the lldb-commits mailing list