]> granicus.if.org Git - clang/commitdiff
HTMLDiagnostics: Always display diagnostics *below* the line in question.
authorTed Kremenek <kremenek@apple.com>
Wed, 18 Feb 2009 22:10:00 +0000 (22:10 +0000)
committerTed Kremenek <kremenek@apple.com>
Wed, 18 Feb 2009 22:10:00 +0000 (22:10 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@64959 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Driver/HTMLDiagnostics.cpp

index 1561fc455d7c70f7036e6d2f8f86fc70e7d37048..6a4bf2307f68f2fe89dbc0fad62671666961b2ec 100644 (file)
@@ -351,20 +351,14 @@ void HTMLDiagnostics::HandlePiece(Rewriter& R, FileID BugFileID,
   const char *TokInstantiationPtr =Pos.getInstantiationLoc().getCharacterData();
   const char *LineStart = TokInstantiationPtr-ColNo;
 
-  // Only compute LineEnd if we display below a line.
+  // Compute LineEnd.
   const char *LineEnd = TokInstantiationPtr;
-  
-  if (P.getDisplayHint() == PathDiagnosticPiece::Below) {
-    const char* FileEnd = Buf->getBufferEnd();
-
-    while (*LineEnd != '\n' && LineEnd != FileEnd)
-      ++LineEnd;
-  }
+  const char* FileEnd = Buf->getBufferEnd();
+  while (*LineEnd != '\n' && LineEnd != FileEnd)
+    ++LineEnd;
   
   // Compute the margin offset by counting tabs and non-tabs.
-  
-  unsigned PosNo = 0;
-  
+  unsigned PosNo = 0;  
   for (const char* c = LineStart; c != TokInstantiationPtr; ++c)
     PosNo += *c == '\t' ? 8 : 1;
   
@@ -432,19 +426,10 @@ void HTMLDiagnostics::HandlePiece(Rewriter& R, FileID BugFileID,
     os << html::EscapeText(Msg) << "</div></td></tr>";
 
     // Insert the new html.
-    unsigned DisplayPos;
-    switch (P.getDisplayHint()) {
-    default: assert(0 && "Unhandled hint.");
-    case PathDiagnosticPiece::Above:
-      DisplayPos = LineStart - FileStart;
-      break;
-    case PathDiagnosticPiece::Below:
-      DisplayPos = LineEnd - FileStart;
-      break;
-    }
-    
+    unsigned DisplayPos = LineEnd - FileStart;    
     SourceLocation Loc = 
       SM.getLocForStartOfFile(LPosInfo.first).getFileLocWithOffset(DisplayPos);
+
     R.InsertStrBefore(Loc, os.str());
   }