[Lldb-commits] [lldb] Colorize output when searching for symbols in lldb (PR #69422)
David Spickett via lldb-commits
lldb-commits at lists.llvm.org
Mon Nov 6 07:33:50 PST 2023
================
@@ -253,9 +264,20 @@ void Symbol::GetDescription(Stream *s, lldb::DescriptionLevel level,
m_addr_range.GetBaseAddress().GetOffset());
}
ConstString demangled = GetMangled().GetDemangledName();
- if (demangled)
+
+ // Checking if the name (i.e., searched symbol is passed as an argument to the function)
+ // In that case, we use the re_pattern function to colorize the symbol.
+ if (demangled && name){
+ s->Printf(", name=");
+ Address::re_pattern(s, demangled.AsCString(), name);
+ }
+ else if(demangled && name == nullptr)
s->Printf(", name=\"%s\"", demangled.AsCString());
- if (m_mangled.GetMangledName())
+ if (m_mangled.GetMangledName() && name){
+ s->Printf(", mangled=");
+ Address::re_pattern(s, m_mangled.GetMangledName().AsCString(), name);
+ }
+ else if(m_mangled.GetMangledName() && name == nullptr)
----------------
DavidSpickett wrote:
The `if (m_mangled.GetMangledName..` should be the outer if. Within that, you can see if you have a name/pattern to use.
Also, you can do this trick to simplify things:
```
if (auto mangled_name = m_mangled.GetMangledName()) {
// If you get here then bool(m_mangled_name.GetMangledName()) is True so you can use it.
do_stuff(m_mangled_name);
} else {
// it was false
}
```
(which is more the code being from before that feature existed than anything you're doing wrong)
https://github.com/llvm/llvm-project/pull/69422
More information about the lldb-commits
mailing list