From 87c3a0cfa60c55541a4c602b0967eb4267e466b2 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Fri, 27 Dec 2013 04:44:35 +0000 Subject: 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. llvm-svn: 198069 --- llvm/lib/Support/LineIterator.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) (limited to 'llvm/lib/Support/LineIterator.cpp') diff --git a/llvm/lib/Support/LineIterator.cpp b/llvm/lib/Support/LineIterator.cpp index c14f96bf714..056d817526c 100644 --- a/llvm/lib/Support/LineIterator.cpp +++ b/llvm/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'); -- cgit v1.2.3