Changeset 43361 in webkit for trunk/JavaScriptCore/parser/Lexer.cpp

Timestamp:

May 7, 2009, 1:17:55 PM (16 years ago)

Author:

Darin Adler

Message:

2009-05-07 Darin Adler <Darin Adler>

Reviewed by Geoff Garen.

• parser/Lexer.cpp: (JSC::Lexer::lex): Fix missing braces. This would make us always take the slower case for string parsing and Visual Studio correctly noticed unreachable code.

File:

• trunk/JavaScriptCore/parser/Lexer.cpp (modified) (1 diff)

trunk/JavaScriptCore/parser/Lexer.cpp

@@ -576 +576 @@
         // Catches -1, \n, \r, \, 0x2028, and 0x2029 as efficiently
         // as possible, and lets through all common ASCII characters.
-        if (UNLIKELY(m_current == '\\') || UNLIKELY(((static_cast<unsigned>(m_current) - 0xE) & 0x2000)))
+        if (UNLIKELY(m_current == '\\') || UNLIKELY(((static_cast<unsigned>(m_current) - 0xE) & 0x2000))) {
             m_buffer16.append(stringStart, currentCharacter() - stringStart);
             goto inString;
+        }
         shift1();
     }