Title: [138227] trunk/Source/WebCore
Revision
138227
Author
tha...@chromium.org
Date
2012-12-19 22:08:11 -0800 (Wed, 19 Dec 2012)

Log Message

Remove slighly confusing "} if" pattern
https://bugs.webkit.org/show_bug.cgi?id=105492

Reviewed by Andreas Kling.

ADVANCE_TO ends in a goto statement, so this doesn't change behavior.

* html/parser/HTMLTokenizer.cpp:
(WebCore::HTMLTokenizer::nextToken):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (138226 => 138227)


--- trunk/Source/WebCore/ChangeLog	2012-12-20 05:57:47 UTC (rev 138226)
+++ trunk/Source/WebCore/ChangeLog	2012-12-20 06:08:11 UTC (rev 138227)
@@ -1,3 +1,15 @@
+2012-12-19  Nico Weber  <tha...@chromium.org>
+
+        Remove slighly confusing "} if" pattern
+        https://bugs.webkit.org/show_bug.cgi?id=105492
+
+        Reviewed by Andreas Kling.
+
+        ADVANCE_TO ends in a goto statement, so this doesn't change behavior.
+
+        * html/parser/HTMLTokenizer.cpp:
+        (WebCore::HTMLTokenizer::nextToken):
+
 2012-12-19  Mark Pilgrim  <pilg...@chromium.org>
 
         [Chromium] Remove idbFactory from PlatformSupport

Modified: trunk/Source/WebCore/html/parser/HTMLTokenizer.cpp (138226 => 138227)


--- trunk/Source/WebCore/html/parser/HTMLTokenizer.cpp	2012-12-20 05:57:47 UTC (rev 138226)
+++ trunk/Source/WebCore/html/parser/HTMLTokenizer.cpp	2012-12-20 06:08:11 UTC (rev 138227)
@@ -348,7 +348,7 @@
         else if (isASCIIUpper(cc)) {
             m_token->appendToName(toLowerCase(cc));
             HTML_ADVANCE_TO(TagNameState);
-        } if (cc == InputStreamPreprocessor::endOfFileMarker) {
+        } else if (cc == InputStreamPreprocessor::endOfFileMarker) {
             parseError();
             HTML_RECONSUME_IN(DataState);
         } else {
@@ -616,7 +616,7 @@
         else if (cc == '>') {
             bufferCharacter(cc);
             HTML_ADVANCE_TO(ScriptDataState);
-        } if (cc == InputStreamPreprocessor::endOfFileMarker) {
+        } else if (cc == InputStreamPreprocessor::endOfFileMarker) {
             parseError();
             HTML_RECONSUME_IN(DataState);
         } else {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to