[Lldb-commits] [lldb] [lldb] refactor highlighting function for image lookup command (PR #76112)

[José Lira Junior via lldb-commits]

junior-jl wrote:

Last commit changes:
- Renamed `pattern_info` to `settings` in all of its uses.
- Moved `HighlightSettings` declaration outside `for` loop.
- Edited commit message as suggested.

> So you're asking about "stacked PRs" and I myself haven't tried this. LLVM has recently enabled user branches on llvm/llvm-project so I think it is possible using some of the tools out there, but I have not tried it. I would look on the forums for anyone who's tried it, or ask on Discord for advice.
...

Thanks for the detailed explanation. I'll look better into this.

Question: Would be good/necessary to refactor `LookupSymbolInModule` into some smaller functions? At least separating the printing part?

https://github.com/llvm/llvm-project/pull/76112