[clang-tools-extra] r191092 - Added colon to make proper error message prefix.

[John Thompson]

Author: jtsoftware
Date: Fri Sep 20 09:44:20 2013
New Revision: 191092

URL: http://llvm.org/viewvc/llvm-project?rev=191092&view=rev
Log:
Added colon to make proper error message prefix.

Modified:
    clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp
    clang-tools-extra/trunk/test/modularize/ProblemsExternC.modularize
    clang-tools-extra/trunk/test/modularize/ProblemsNamespace.modularize

Modified: clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp?rev=191092&r1=191091&r2=191092&view=diff
==============================================================================
--- clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp (original)
+++ clang-tools-extra/trunk/modularize/PreprocessorTracker.cpp Fri Sep 20 09:44:20 2013
@@ -935,14 +935,14 @@ public:
       if ((I->File == SourceHandle) && (I->Line >= BlockStartLine) &&
           (I->Line < BlockEndLine)) {
         returnValue = false;
-        OS << SourcePath << ":" << I->Line << ":" << I->Column << "\n";
+        OS << SourcePath << ":" << I->Line << ":" << I->Column << ":\n";
         OS << getSourceLine(PP, FileID, I->Line) << "\n";
         if (I->Column > 0)
           OS << std::string(I->Column - 1, ' ') << "^\n";
         OS << "error: Include directive within " << BlockIdentifierMessage
            << ".\n";
         OS << SourcePath << ":" << BlockStartLine << ":" << BlockStartColumn
-           << "\n";
+           << ":\n";
         OS << getSourceLine(PP, BlockStartLoc) << "\n";
         if (BlockStartColumn > 0)
           OS << std::string(BlockStartColumn - 1, ' ') << "^\n";

Modified: clang-tools-extra/trunk/test/modularize/ProblemsExternC.modularize
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/modularize/ProblemsExternC.modularize?rev=191092&r1=191091&r2=191092&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/modularize/ProblemsExternC.modularize (original)
+++ clang-tools-extra/trunk/test/modularize/ProblemsExternC.modularize Fri Sep 20 09:44:20 2013
@@ -2,11 +2,11 @@
 
 Inputs/IncludeInExtern.h
 
-# CHECK: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInExtern.h:2:3
+# CHECK: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInExtern.h:2:3:
 # CHECK-NEXT:   #include "Empty.h"
 # CHECK-NEXT:   ^
 # CHECK-NEXT: error: Include directive within extern "C" {}.
-# CHECK-NEXT: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInExtern.h:1:1
+# CHECK-NEXT: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInExtern.h:1:1:
 # CHECK-NEXT: extern "C" {
 # CHECK-NEXT: ^
 # CHECK-NEXT: The "extern "C" {}" block is here.

Modified: clang-tools-extra/trunk/test/modularize/ProblemsNamespace.modularize
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/modularize/ProblemsNamespace.modularize?rev=191092&r1=191091&r2=191092&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/modularize/ProblemsNamespace.modularize (original)
+++ clang-tools-extra/trunk/test/modularize/ProblemsNamespace.modularize Fri Sep 20 09:44:20 2013
@@ -2,11 +2,11 @@
 
 Inputs/IncludeInNamespace.h
 
-# CHECK: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInNamespace.h:2:3
+# CHECK: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInNamespace.h:2:3:
 # CHECK-NEXT:   #include "Empty.h"
 # CHECK-NEXT:   ^
 # CHECK-NEXT: error: Include directive within namespace MyNamespace {}.
-# CHECK-NEXT: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInNamespace.h:1:1
+# CHECK-NEXT: {{.*}}{{[/\\]}}Inputs{{[/\\]}}IncludeInNamespace.h:1:1:
 # CHECK-NEXT: namespace MyNamespace {
 # CHECK-NEXT: ^
 # CHECK-NEXT: The "namespace MyNamespace {}" block is here.