[Lldb-commits] [lldb] [lldb][breakpoint] Grey out disabled breakpoints (PR #91404)

[Pavel Labath via lldb-commits]

https://github.com/labath commented:

I'm kinda late to the party (particularly as there's an analogous "progress" setting), but I have to say I was surprised to see a `disable-breakpoint-ansi-suffix` setting. That sounds like too low level of a model. The model I would expect/find more natural is if there was a single setting describing the "normal" text colour, and then several (not necessarily one each item that needs colouring) settings defining the colour of items that need colouring.

I find it hard to imagine why one would ever want to set different values for the "suffix" colours, as that means the colour of random text would depend on what was printed before it.

I don't want to make this a blocking objection, but I would urge you to consider if this is the behavior you want.

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