[cfe-commits] r167379 - in /cfe/trunk: include/clang/Basic/DiagnosticParseKinds.td test/PCH/empty-with-headers.c test/Parser/empty-translation-unit.c

Matt Beaumont-Gay matthewbg at google.com
Sun Nov 4 21:32:00 PST 2012


Author: matthewbg
Date: Sun Nov  4 23:32:00 2012
New Revision: 167379

URL: http://llvm.org/viewvc/llvm-project?rev=167379&view=rev
Log:
Our style for diagnostic messages is to not include a trailing dot.

Modified:
    cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
    cfe/trunk/test/PCH/empty-with-headers.c
    cfe/trunk/test/Parser/empty-translation-unit.c

Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=167379&r1=167378&r2=167379&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Sun Nov  4 23:32:00 2012
@@ -24,7 +24,7 @@
 let CategoryName = "Parse Issue" in {
 
 def ext_empty_translation_unit : Extension<
-  "ISO C requires a translation unit to contain at least one declaration.">,
+  "ISO C requires a translation unit to contain at least one declaration">,
   InGroup<DiagGroup<"empty-translation-unit">>;
 def warn_cxx98_compat_top_level_semi : Warning<
   "extra ';' outside of a function is incompatible with C++98">,

Modified: cfe/trunk/test/PCH/empty-with-headers.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/PCH/empty-with-headers.c?rev=167379&r1=167378&r2=167379&view=diff
==============================================================================
--- cfe/trunk/test/PCH/empty-with-headers.c (original)
+++ cfe/trunk/test/PCH/empty-with-headers.c Sun Nov  4 23:32:00 2012
@@ -24,4 +24,4 @@
 
 // This should only fire if the header is not included,
 // either explicitly or as a prefix header.
-// expected-error{{ISO C requires a translation unit to contain at least one declaration.}}
+// expected-error{{ISO C requires a translation unit to contain at least one declaration}}

Modified: cfe/trunk/test/Parser/empty-translation-unit.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Parser/empty-translation-unit.c?rev=167379&r1=167378&r2=167379&view=diff
==============================================================================
--- cfe/trunk/test/Parser/empty-translation-unit.c (original)
+++ cfe/trunk/test/Parser/empty-translation-unit.c Sun Nov  4 23:32:00 2012
@@ -7,4 +7,4 @@
 #define A_MACRO_IS_NOT_GOOD_ENOUGH 1
 
 // In C we should get this warning, but in C++ we shouldn't.
-// expected-warning{{ISO C requires a translation unit to contain at least one declaration.}}
+// expected-warning{{ISO C requires a translation unit to contain at least one declaration}}





More information about the cfe-commits mailing list