Bug 691959: Remove unused variable 'bool done' from nsCSSFrameConstructor.cpp. r=dbaron
authorDaniel Holbert <dholbert@cs.stanford.edu>
Sat, 08 Oct 2011 14:44:26 -0700
changeset 79751 d323beedf560c47993c37054ac741acaf0cf2279
parent 79750 cf33518ba341619044089ee6ba28c295acccff76
child 79752 be0b8f246876271fea16a7f2d9f6c774d404e1ca
push id434
push userclegnitto@mozilla.com
push dateWed, 21 Dec 2011 12:10:54 +0000
treeherdermozilla-beta@bddb6ed8dd47 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdbaron
bugs691959
milestone10.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 691959: Remove unused variable 'bool done' from nsCSSFrameConstructor.cpp. r=dbaron
layout/base/nsCSSFrameConstructor.cpp
--- a/layout/base/nsCSSFrameConstructor.cpp
+++ b/layout/base/nsCSSFrameConstructor.cpp
@@ -9926,41 +9926,37 @@ nsCSSFrameConstructor::InsertFirstLineFr
 
 // Determine how many characters in the text fragment apply to the
 // first letter
 static PRInt32
 FirstLetterCount(const nsTextFragment* aFragment)
 {
   PRInt32 count = 0;
   PRInt32 firstLetterLength = 0;
-  bool done = false;
 
   PRInt32 i, n = aFragment->GetLength();
   for (i = 0; i < n; i++) {
     PRUnichar ch = aFragment->CharAt(i);
     if (XP_IS_SPACE(ch)) {
       if (firstLetterLength) {
-        done = PR_TRUE;
         break;
       }
       count++;
       continue;
     }
     // XXX I18n
     if ((ch == '\'') || (ch == '\"')) {
       if (firstLetterLength) {
-        done = PR_TRUE;
         break;
       }
       // keep looping
       firstLetterLength = 1;
     }
     else {
       count++;
-      done = PR_TRUE;
       break;
     }
   }
 
   return count;
 }
 
 static bool