================
@@ -2074,6 +2076,64 @@ static const Definition *FindEntry(const llvm::StringRef 
&format_str,
   return parent;
 }
 
+/// Parses a single highlighting format specifier.
+///
+/// Example syntax for such specifier:
+/// \code
+/// ${function.name-with-args:%highlight_basename(ansi.fg.green)}
----------------
jimingham wrote:

Do we need this extra `language` node at all?  After all `plugin.cplusplus` 
already says this is a language setting?  And it's pretty clear that `settings 
set plugin.cplusplus.function-name-format <whatever>` is setting that format 
for the language cplusplus.  So that extra node doesn't seem to be expressing 
anything useful. 

If we do need a node, maybe something like `display` because this is setting 
that has to do with `display` of something for that language, and we might add 
more language specific display options?

https://github.com/llvm/llvm-project/pull/131836
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to