From 1739b6cc2006fbbdc68b0a4b2d32452c17878f03 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Fri, 27 Dec 2013 04:44:35 +0000 Subject: [PATCH] Use two variables here rather than reusing (and abusing) one. This is much more clear to me. I meant to make this change before committing the original patch, but forgot to merge it in. Sorry. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@198069 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Support/LineIterator.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/Support/LineIterator.cpp b/lib/Support/LineIterator.cpp index c14f96bf714..056d817526c 100644 --- a/lib/Support/LineIterator.cpp +++ b/lib/Support/LineIterator.cpp @@ -29,15 +29,14 @@ void line_iterator::advance() { const char *Pos = CurrentLine.end(); assert(Pos == Buffer->getBufferStart() || *Pos == '\n' || *Pos == '\0'); - size_t Length = 0; if (CommentMarker == '\0') { // If we're not stripping comments, this is simpler. - while (Pos[Length] == '\n') - ++Length; - Pos += Length; - LineNumber += Length; - Length = 0; + size_t Blanks = 0; + while (Pos[Blanks] == '\n') + ++Blanks; + Pos += Blanks; + LineNumber += Blanks; } else { // Skip comments and count line numbers, which is a bit more complex. for (;;) { @@ -60,6 +59,7 @@ void line_iterator::advance() { } // Measure the line. + size_t Length = 0; do { ++Length; } while (Pos[Length] != '\0' && Pos[Length] != '\n');