labath wrote:

I'd probably go with the first version. It doesn't leave a strange hanging 
default label, and it saves a line. It true that other settings show the 
default value after the actual one, but these settings are kind of special 
already (in that they print on multiple lines), so I think the consistency 
isn't that important.

Instead of "none", could we say "default: empty" ? As a user, I'm not sure how 
I would interpret "none". Regardless of how it's implemented internally, "empty 
list/dict/whatever" is the value of the setting you get if you don't change it, 
is it not?

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

Reply via email to