kastiglione wrote:

@jimingham I agree that expanding the output of `v` while listing variables 
could be a regression. However note that this change is to `dwim-print`, not 
`v`, and so this change would not cause the problem you're describing. This is 
because `dwim-print` has neither a "print all variables", nor a "print more 
than one variable" mode.

In the discussion above @Michael137 asked whether `v` and `expr` should also 
have this expand behavior, and you've given a good argument for `v` to not have 
this behavior.

> We also have to make sure that we don't end up expanding variables that 
> people have summaries for

I hadn't thought about this. I'll add a test for this case.

Other than ensuring objects are not expanded when their summary dictates they 
not be expanded, are you ok with this change to `dwim-print`?

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

Reply via email to