JDevlieghere added a comment. In https://reviews.llvm.org/D52772#1252656, @jingham wrote:
> Would it be possible for the exporter to notice empty settings and write > "settings clear" instead? I don't think we can if we want to re-use the existing dump infrastructure, unless there's a way to ask a `Property` if is empty/has a default value? > I'm worried that if you have a complicated setting, and the you do: > > (lldb) settings set target.some-complex-setting > > and decide you are wrong, you don't want to change the complex setting, then > you have to know to delete the text - hitting a return is actually > destructive. This sounds like a very specific issue, though. You have the same problem if the value is something like a string and the user accidentally types one more character. What kind of scenario do you have in mind where a user would be able to type the command but not know how to delete it? I'm definitely not saying it's not a valid reason to consider alternatives and the current behavior is nice, but do you consider this a showstopper? Repository: rLLDB LLDB https://reviews.llvm.org/D52772 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
