junior-jl wrote: > Maybe this is me being used to monster functions in lldb, but this one seems > fine. At least the volume of code. What you could do is "early return" (/early continue) in a couple of places to reduce the indentation. > 1. You could early return if there are no matches. > 2. You could continue if Symbol *symbol is nullptr.
> This turns it into mostly straight line code (and from there if you did want > to break it into functions, it would be easier). I see. But this would need to be in another PR, right? (checking to see if I'm understanding the flow of contribution) https://github.com/llvm/llvm-project/pull/76112 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits