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

[Enrico Granata via lldb-commits]

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.


http://reviews.llvm.org/D13657