[Lldb-commits] [PATCH] D130660: [LLDB] Fix missing return value in SBBreakpointLocation::GetQueueName()

Slava Gurevich via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jul 27 13:04:53 PDT 2022


fixathon created this revision.
fixathon added reviewers: clayborg, aadsm, labath.
Herald added a project: All.
fixathon requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

Seems like a typo in the function that never returns a significant value


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D130660

Files:
  lldb/source/API/SBBreakpointLocation.cpp


Index: lldb/source/API/SBBreakpointLocation.cpp
===================================================================
--- lldb/source/API/SBBreakpointLocation.cpp
+++ lldb/source/API/SBBreakpointLocation.cpp
@@ -374,7 +374,7 @@
   if (loc_sp) {
     std::lock_guard<std::recursive_mutex> guard(
         loc_sp->GetTarget().GetAPIMutex());
-    loc_sp->GetQueueName();
+    return loc_sp->GetQueueName();
   }
   return nullptr;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130660.448138.patch
Type: text/x-patch
Size: 430 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20220727/1a1a8782/attachment.bin>


More information about the lldb-commits mailing list