2010-11-30 James Robinson <jamesr@chromium.org>
authorjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Nov 2010 23:17:48 +0000 (23:17 +0000)
committerjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 30 Nov 2010 23:17:48 +0000 (23:17 +0000)
        Reviewed by Tony Chang.

        Build fix - add parens in foo || bar && baz to make gcc happy
        https://bugs.webkit.org/show_bug.cgi?id=50276

        gcc 4.4 warns that statements of the form "foo || bar && baz".  It wants
        parens around the "bar && baz" part to make it clear which way the logical
        operators bind.

        * editing/TextIterator.cpp:
        (WebCore::SearchBuffer::search):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@72968 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/editing/TextIterator.cpp

index e79174a..894910a 100644 (file)
@@ -1,3 +1,17 @@
+2010-11-30  James Robinson  <jamesr@chromium.org>
+
+        Reviewed by Tony Chang.
+
+        Build fix - add parens in foo || bar && baz to make gcc happy
+        https://bugs.webkit.org/show_bug.cgi?id=50276
+
+        gcc 4.4 warns that statements of the form "foo || bar && baz".  It wants
+        parens around the "bar && baz" part to make it clear which way the logical
+        operators bind.
+
+        * editing/TextIterator.cpp:
+        (WebCore::SearchBuffer::search):
+
 2010-11-30  Daniel Bates  <dbates@rim.com>
 
         Reviewed by David Hyatt.
index 1bc558a..8313c9b 100644 (file)
@@ -2142,7 +2142,7 @@ nextMatch:
     ASSERT(matchStart + matchedLength <= size);
 
     // If this match is "bad", move on to the next match.
-    if (isBadMatch(m_buffer.data() + matchStart, matchedLength) || m_options & AtWordStarts && !isWordStartMatch(matchStart, matchedLength)) {
+    if (isBadMatch(m_buffer.data() + matchStart, matchedLength) || ((m_options & AtWordStarts) && !isWordStartMatch(matchStart, matchedLength))) {
         matchStart = usearch_next(searcher, &status);
         ASSERT(status == U_ZERO_ERROR);
         goto nextMatch;