[cfe-commits] r49893 - /cfe/trunk/Driver/HTMLDiagnostics.cpp

Ted Kremenek kremenek at apple.com
Thu Apr 17 22:01:34 PDT 2008


Author: kremenek
Date: Fri Apr 18 00:01:33 2008
New Revision: 49893

URL: http://llvm.org/viewvc/llvm-project?rev=49893&view=rev
Log:
Fixed range highlighting bug in HTML diagnostics.

Modified:
    cfe/trunk/Driver/HTMLDiagnostics.cpp

Modified: cfe/trunk/Driver/HTMLDiagnostics.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/Driver/HTMLDiagnostics.cpp?rev=49893&r1=49892&r2=49893&view=diff

==============================================================================
--- cfe/trunk/Driver/HTMLDiagnostics.cpp (original)
+++ cfe/trunk/Driver/HTMLDiagnostics.cpp Fri Apr 18 00:01:33 2008
@@ -326,15 +326,13 @@
 
   if (EndColNo) {
     // Add in the length of the token, so that we cover multi-char tokens.
-    EndColNo += Lexer::MeasureTokenLength(Range.getEnd(), SM);
+    EndColNo += Lexer::MeasureTokenLength(Range.getEnd(), SM) - 1;
   }
   
   // Highlight the range.  Make the span tag the outermost tag for the
   // selected range.
     
-  SourceLocation E = LogicalEnd.getFileLocWithOffset(OldEndColNo > EndColNo
-                                                     ? -(OldEndColNo - EndColNo)
-                                                     : EndColNo - OldEndColNo);
+  SourceLocation E = LogicalEnd.getFileLocWithOffset(EndColNo - OldEndColNo);
   
   html::HighlightRange(R, LogicalStart, E,
                        "<span class=\"mrange\">", "</span>");





More information about the cfe-commits mailing list