Changeset 88724 in webkit for trunk/Source/JavaScriptCore/parser/Lexer.cpp

Timestamp:

Jun 13, 2011, 3:46:21 PM (14 years ago)

Author:

[email protected]

Message:

2011-06-13 Oliver Hunt <​[email protected]>

Reviewed by Gavin Barraclough.

Fix llocp and lvalp names in the lexer to something more meaningful
​https://bugs.webkit.org/show_bug.cgi?id=62605

A simple rename

• parser/Lexer.cpp: (JSC::Lexer::parseIdentifier): (JSC::Lexer::parseString): (JSC::Lexer::lex):
• parser/Lexer.h: (JSC::Lexer::lexExpectIdentifier):

File:

• trunk/Source/JavaScriptCore/parser/Lexer.cpp (modified) (12 diffs)

trunk/Source/JavaScriptCore/parser/Lexer.cpp

@@ -406 +406 @@
 }
 
-template <bool shouldCreateIdentifier> ALWAYS_INLINE JSTokenType Lexer::parseIdentifier(JSTokenData* lvalp, unsigned lexType)
+template <bool shouldCreateIdentifier> ALWAYS_INLINE JSTokenType Lexer::parseIdentifier(JSTokenData* tokenData, unsigned lexType)
 {
     const ptrdiff_t remaining = m_codeEnd - m_code;
     if ((remaining >= maxTokenLength) && !(lexType & IgnoreReservedWords)) {
-        JSTokenType keyword = parseKeyword<shouldCreateIdentifier>(lvalp);
+        JSTokenType keyword = parseKeyword<shouldCreateIdentifier>(tokenData);
         if (keyword != IDENT) {
-            ASSERT((!shouldCreateIdentifier) || lvalp->ident);
+            ASSERT((!shouldCreateIdentifier) || tokenData->ident);
             return keyword;
         }
@@ -458 +458 @@
 
         ident = makeIdentifier(identifierStart, identifierLength);
-        lvalp->ident = ident;
+        tokenData->ident = ident;
     } else
-        lvalp->ident = 0;
+        tokenData->ident = 0;
 
     m_delimited = false;
@@ -479 +479 @@
 }
 
-template <bool shouldBuildStrings> ALWAYS_INLINE bool Lexer::parseString(JSTokenData* lvalp, bool strictMode)
+template <bool shouldBuildStrings> ALWAYS_INLINE bool Lexer::parseString(JSTokenData* tokenData, bool strictMode)
 {
     int stringQuoteCharacter = m_current;
@@ -575 +575 @@
         m_buffer16.append(stringStart, currentCharacter() - stringStart);
     if (shouldBuildStrings)
-        lvalp->ident = makeIdentifier(m_buffer16.data(), m_buffer16.size());
+        tokenData->ident = makeIdentifier(m_buffer16.data(), m_buffer16.size());
     else
-        lvalp->ident = 0;
+        tokenData->ident = 0;
 
     m_buffer16.resize(0);
@@ -753 +753 @@
 }
 
-JSTokenType Lexer::lex(JSTokenData* lvalp, JSTokenInfo* llocp, unsigned lexType, bool strictMode)
+JSTokenType Lexer::lex(JSTokenData* tokenData, JSTokenInfo* tokenInfo, unsigned lexType, bool strictMode)
 {
     ASSERT(!m_error);
@@ -1008 +1008 @@
         break;
     case CharacterOpenBrace:
-        lvalp->intValue = currentOffset();
+        tokenData->intValue = currentOffset();
         shift();
         token = OPENBRACE;
         break;
     case CharacterCloseBrace:
-        lvalp->intValue = currentOffset();
+        tokenData->intValue = currentOffset();
         m_delimited = true;
         shift();
@@ -1028 +1028 @@
         shift();
         if ((m_current | 0x20) == 'x' && isASCIIHexDigit(peek(1))) {
-            parseHex(lvalp->doubleValue);
+            parseHex(tokenData->doubleValue);
             token = NUMBER;
         } else {
             record8('0');
             if (isASCIIOctalDigit(m_current)) {
-                if (parseOctal(lvalp->doubleValue)) {
+                if (parseOctal(tokenData->doubleValue)) {
                     if (strictMode)
                         goto returnError;
@@ -1043 +1043 @@
     case CharacterNumber:
         if (LIKELY(token != NUMBER)) {
-            if (!parseDecimal(lvalp->doubleValue)) {
+            if (!parseDecimal(tokenData->doubleValue)) {
                 if (m_current == '.') {
                     shift();
@@ -1054 +1054 @@
                 // Null-terminate string for strtod.
                 m_buffer8.append('\0');
-                lvalp->doubleValue = WTF::strtod(m_buffer8.data(), 0);
+                tokenData->doubleValue = WTF::strtod(m_buffer8.data(), 0);
             }
             token = NUMBER;
@@ -1067 +1067 @@
     case CharacterQuote:
         if (lexType & DontBuildStrings) {
-            if (UNLIKELY(!parseString<false>(lvalp, strictMode)))
+            if (UNLIKELY(!parseString<false>(tokenData, strictMode)))
                 goto returnError;
         } else {
-            if (UNLIKELY(!parseString<true>(lvalp, strictMode)))
+            if (UNLIKELY(!parseString<true>(tokenData, strictMode)))
                 goto returnError;
         }
@@ -1082 +1082 @@
     case CharacterBackSlash:
         if (lexType & DontBuildKeywords)
-            token = parseIdentifier<false>(lvalp, lexType);
+            token = parseIdentifier<false>(tokenData, lexType);
         else
-            token = parseIdentifier<true>(lvalp, lexType);
+            token = parseIdentifier<true>(tokenData, lexType);
         break;
     case CharacterLineTerminator:
@@ -1119 +1119 @@
 
 returnToken:
-    llocp->line = m_lineNumber;
-    llocp->startOffset = startOffset;
-    llocp->endOffset = currentOffset();
+    tokenInfo->line = m_lineNumber;
+    tokenInfo->startOffset = startOffset;
+    tokenInfo->endOffset = currentOffset();
     m_lastToken = token;
     return token;