aaron.ballman added inline comments.

================
Comment at: clang/docs/ReleaseNotes.rst:103-137
+- When describing the failure of static assertion of `==` expression, clang 
prints the integer
+  representation of the value as well as its character representation when
+  the user-provided expression is of character type. If the character is
+  non-printable, clang now shows the escpaed character.
+  Clang also prints multi-byte characters if the user-provided expression
+  is of multi-byte character type.
+
----------------
cor3ntin wrote:
> @aaron.ballman One one hand this is nice, on the other hand maybe too 
> detailed. What do you think?
I'm happy with it -- better too much detail than too little, but this really 
helps users see what's been improved and why it matters.

That said, I think `0x0A` and `0x1F30D` would arguably be better than printing 
the values in decimal. For `\n`, perhaps folks remember that it's decimal value 
10, but nobody is going to know what `127757` means compared to the hex 
representation (esp because the value is specified in hex with the prefix 
printed in the error message). WDYT?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155610/new/

https://reviews.llvm.org/D155610

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to