[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