================
@@ -199,13 +183,13 @@ bool lldb_private::formatters::WCharSummaryProvider(
   options.SetBinaryZeroIsTerminator(false);
 
   switch (wchar_size) {
-  case 8:
+  case 1:
----------------
Nerixyz wrote:

No, this wasn't wrong. It used the _bit size_ before, and the libc++ summary 
used the _byte size_. I'm not sure which is better (I suppose 8 bits/byte is 
true for all targets supported by LLDB).

This reminds me of another difference: The `*CharSummaryProvider`s used 
`valobj.GetCompilerType().GetBasicTypeFromAST()` and the libc++ formatters used 
`ScratchTypeSystemClang::GetBasicType()`. Is there a big difference between 
them, and is one preferred over the other? 

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

Reply via email to