[Lldb-commits] [PATCH] D13657: [lldb] char summary provider

Enrico Granata via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 12 12:57:50 PDT 2015

granata.enrico added a comment.

The consistency argument is not entirely unfair.

I would claim that Unicode is a more complex beast than plain ASCII, so it makes sense to go the extra mile in giving you details in that case, compared to the simpler char case.

If the only reason you're pursuing the patch is to appease the test case, I say "no" to the change.
If you think this is actually worthwhile on its own merits, I would rather much have either a separate "MI" formatters category.


More information about the lldb-commits mailing list