================ @@ -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