[Lldb-commits] [PATCH] D110115: [lldb] Remove Expression's dependency on CPlusPlusLanguagePlugin

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 20 17:06:02 PDT 2021


bulbazord added inline comments.


================
Comment at: lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp:524-530
+  if (param_and_qual_matches.size())
+    return param_and_qual_matches[0]; // It is assumed that there will be only
+                                      // one!
+  else if (param_matches.size())
+    return param_matches[0]; // Return one of them as a best match
+  else
+    return ConstString();
----------------
JDevlieghere wrote:
> I know you just copied this, but `else` after a return? 
I think there's a lot of opportunity to refactor this method. Think I should do it in this change or would a follow-up be more reasonable?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110115



More information about the lldb-commits mailing list