[Lldb-commits] [PATCH] D80793: [lldb][NFC] Make ClangExpressionSourceCode's wrapping logic more consistent

Shafik Yaghmour via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Jun 1 18:58:07 PDT 2020


shafik added inline comments.


================
Comment at: lldb/source/Plugins/ExpressionParser/Clang/ClangExpressionSourceCode.h:41
+    /// Note that this is also used for static member functions of a C++ class.
+    Function
+  };
----------------
I would actually feel better have a separate enumerator for C++ static member functions and just having it fall-through when used. It would be self-documenting.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80793





More information about the lldb-commits mailing list