shafik added a comment.

Thank you for this diagnostic fix, please make sure you update 
`clang/test/CXX/dcl.decl/dcl.init/dcl.init.string/p2.cpp`.



================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:5956
 def err_initializer_string_for_char_array_too_long : Error<
-  "initializer-string for char array is too long">;
+  "initializer-string for char array is too long, expected %0 but was %1 
(including the null terminating character)">;
 def ext_initializer_string_for_char_array_too_long : ExtWarn<
----------------
Note. we are allowed to initialize with a string-literal that is smaller than 
the array size so maybe something along the lines of `array size is %0 but 
initializer has size %1`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141283

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

Reply via email to