Fix a typo in Parser error message
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 22 Mar 2015 04:08:23 +0000 (04:08 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 22 Mar 2015 04:08:23 +0000 (04:08 +0000)
https://bugs.webkit.org/show_bug.cgi?id=142942

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2015-03-21
Reviewed by Alexey Proskuryakov.

Source/JavaScriptCore:

* jit/JITPropertyAccess.cpp:
(JSC::JIT::emitSlow_op_resolve_scope):
* jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::emitSlow_op_resolve_scope):
* parser/Parser.cpp:
(JSC::Parser<LexerType>::parseClass):
Fix a common identifier typo.

LayoutTests:

* js/class-syntax-declaration-expected.txt:
* js/class-syntax-expression-expected.txt:
* js/class-syntax-super-expected.txt:
* js/script-tests/class-syntax-declaration.js:
* js/script-tests/class-syntax-expression.js:
* js/script-tests/class-syntax-super.js:

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

LayoutTests/ChangeLog
LayoutTests/js/class-syntax-declaration-expected.txt
LayoutTests/js/class-syntax-expression-expected.txt
LayoutTests/js/class-syntax-super-expected.txt
LayoutTests/js/script-tests/class-syntax-declaration.js
LayoutTests/js/script-tests/class-syntax-expression.js
LayoutTests/js/script-tests/class-syntax-super.js
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/jit/JITPropertyAccess.cpp
Source/JavaScriptCore/jit/JITPropertyAccess32_64.cpp
Source/JavaScriptCore/parser/Parser.cpp

index 2f12c241a03cd2cb83bcc56e248d772c353e7e7e..60266835c165e25ca2cca7d31c192f33aee4f782 100644 (file)
@@ -1,3 +1,17 @@
+2015-03-21  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Fix a typo in Parser error message
+        https://bugs.webkit.org/show_bug.cgi?id=142942
+
+        Reviewed by Alexey Proskuryakov.
+
+        * js/class-syntax-declaration-expected.txt:
+        * js/class-syntax-expression-expected.txt:
+        * js/class-syntax-super-expected.txt:
+        * js/script-tests/class-syntax-declaration.js:
+        * js/script-tests/class-syntax-expression.js:
+        * js/script-tests/class-syntax-super.js:
+
 2015-03-21  Brent Fulgham  <bfulgham@apple.com>
 
         [Win] Try to get debug bots green.
index 93c5e4a02f527a393c4fa876f859e3a52dbf6829..e197ea6775c9a2490b4433058d06318530094ac8 100644 (file)
@@ -17,7 +17,7 @@ PASS (new A).__proto__ is A.prototype
 PASS A.prototype.constructor is A
 PASS class threw exception SyntaxError: Unexpected end of script.
 PASS class X { threw exception SyntaxError: Unexpected end of script.
-PASS class X { ( } threw exception SyntaxError: Unexpected token '('. Expected an indentifier..
+PASS class X { ( } threw exception SyntaxError: Unexpected token '('. Expected an identifier..
 PASS class X {} did not throw exception.
 PASS class X { constructor() {} constructor() {} } threw exception SyntaxError: Cannot declare multiple constructors in a single class..
 PASS class X { constructor() {} static constructor() { return staticMethodValue; } } did not throw exception.
index 38074617a151aa99a8729e9e6535462298bf3efa..b2725d591421b6a9b6d6a0a8402c0448d45c0019 100644 (file)
@@ -17,7 +17,7 @@ PASS (new A).__proto__ is A.prototype
 PASS A.prototype.constructor is A
 PASS x = class threw exception SyntaxError: Unexpected end of script.
 PASS x = class { threw exception SyntaxError: Unexpected end of script.
-PASS x = class { ( } threw exception SyntaxError: Unexpected token '('. Expected an indentifier..
+PASS x = class { ( } threw exception SyntaxError: Unexpected token '('. Expected an identifier..
 PASS x = class {} did not throw exception.
 PASS x = class { constructor() {} constructor() {} } threw exception SyntaxError: Cannot declare multiple constructors in a single class..
 PASS x = class { constructor() {} static constructor() { return staticMethodValue; } } did not throw exception.
index e636c88c3b9a672c394f4c36c2c4f4362b3d00f2..234799f21e7fd6c90150005b64f08520f6b28dfc 100644 (file)
@@ -13,7 +13,7 @@ PASS (new Derived).baseMethodInGetterSetter is (new Base).baseMethod
 PASS (new Derived).baseMethodInGetterSetter = 1; valueInSetter is (new Base).baseMethod
 PASS Derived.staticMethod() is "base3"
 PASS (new SecondDerived).chainMethod() is ["base", "derived", "secondDerived"]
-PASS x = class extends Base { constructor() { super(); } super() {} } threw exception SyntaxError: Unexpected keyword 'super'. Expected an indentifier..
+PASS x = class extends Base { constructor() { super(); } super() {} } threw exception SyntaxError: Unexpected keyword 'super'. Expected an identifier..
 PASS x = class extends Base { constructor() { super(); } method() { super() } } threw exception SyntaxError: Cannot call super() outside of a class constructor..
 PASS x = class extends Base { constructor() { super(); } method() { super } } threw exception SyntaxError: Cannot reference super..
 PASS x = class extends Base { constructor() { super(); } method() { return new super } } did not throw exception.
index 035321e783a5c0e8bbb72438ac7d1b1cc1bfa9be..0289819ab9f0380fbc678d05852bd0e728efa7a6 100644 (file)
@@ -29,7 +29,7 @@ shouldBe("A.prototype.constructor", "A");
 
 shouldThrow("class", "'SyntaxError: Unexpected end of script'");
 shouldThrow("class X {", "'SyntaxError: Unexpected end of script'");
-shouldThrow("class X { ( }", "'SyntaxError: Unexpected token \\'(\\'. Expected an indentifier.'");
+shouldThrow("class X { ( }", "'SyntaxError: Unexpected token \\'(\\'. Expected an identifier.'");
 shouldNotThrow("class X {}");
 shouldThrow("class X { constructor() {} constructor() {} }", "'SyntaxError: Cannot declare multiple constructors in a single class.'");
 shouldNotThrow("class X { constructor() {} static constructor() { return staticMethodValue; } }");
index 6eca6ffe50594ad5980c5f4ea514fd275f3f8c2b..b581111231000765626b22c1d463b9cfacd75396 100644 (file)
@@ -29,7 +29,7 @@ shouldBe("A.prototype.constructor", "A");
 
 shouldThrow("x = class", "'SyntaxError: Unexpected end of script'");
 shouldThrow("x = class {", "'SyntaxError: Unexpected end of script'");
-shouldThrow("x = class { ( }", "'SyntaxError: Unexpected token \\'(\\'. Expected an indentifier.'");
+shouldThrow("x = class { ( }", "'SyntaxError: Unexpected token \\'(\\'. Expected an identifier.'");
 shouldNotThrow("x = class {}");
 shouldThrow("x = class { constructor() {} constructor() {} }", "'SyntaxError: Cannot declare multiple constructors in a single class.'");
 shouldNotThrow("x = class { constructor() {} static constructor() { return staticMethodValue; } }");
index 3c29ac4aad636f486c080410de6780c9ae63e6a6..b27adbe06700ac1615bd7477ee684e50b23c6553 100644 (file)
@@ -37,7 +37,7 @@ shouldBe('(new Derived).baseMethodInGetterSetter', '(new Base).baseMethod');
 shouldBe('(new Derived).baseMethodInGetterSetter = 1; valueInSetter', '(new Base).baseMethod');
 shouldBe('Derived.staticMethod()', '"base3"');
 shouldBe('(new SecondDerived).chainMethod()', '["base", "derived", "secondDerived"]');
-shouldThrow('x = class extends Base { constructor() { super(); } super() {} }', '"SyntaxError: Unexpected keyword \'super\'. Expected an indentifier."');
+shouldThrow('x = class extends Base { constructor() { super(); } super() {} }', '"SyntaxError: Unexpected keyword \'super\'. Expected an identifier."');
 shouldThrow('x = class extends Base { constructor() { super(); } method() { super() } }',
     '"SyntaxError: Cannot call super() outside of a class constructor."');
 shouldThrow('x = class extends Base { constructor() { super(); } method() { super } }', '"SyntaxError: Cannot reference super."');
index 3b983d9bacce178eb398dba2788f653885bb8ab3..526d0fbd5b22b624de1fd8131938641d14fb8d89 100644 (file)
@@ -1,3 +1,18 @@
+2015-03-21  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Fix a typo in Parser error message
+        https://bugs.webkit.org/show_bug.cgi?id=142942
+
+        Reviewed by Alexey Proskuryakov.
+
+        * jit/JITPropertyAccess.cpp:
+        (JSC::JIT::emitSlow_op_resolve_scope):
+        * jit/JITPropertyAccess32_64.cpp:
+        (JSC::JIT::emitSlow_op_resolve_scope):
+        * parser/Parser.cpp:
+        (JSC::Parser<LexerType>::parseClass):
+        Fix a common identifier typo.
+
 2015-03-21  Joseph Pecoraro  <pecoraro@apple.com>
 
         Computed Property names should allow only AssignmentExpressions not any Expression
index cab4da1337b4c39d009a33ab03c730ca23156e62..8cdb070104854b8609e9c6210ce7a51712dfe8ea 100644 (file)
@@ -637,8 +637,8 @@ void JIT::emitSlow_op_resolve_scope(Instruction* currentInstruction, Vector<Slow
 
     linkSlowCase(iter);
     int32_t scope = currentInstruction[2].u.operand;
-    int32_t indentifierIndex = currentInstruction[3].u.operand;
-    callOperation(operationResolveScope, dst, scope, indentifierIndex);
+    int32_t identifierIndex = currentInstruction[3].u.operand;
+    callOperation(operationResolveScope, dst, scope, identifierIndex);
 }
 
 void JIT::emitLoadWithStructureCheck(int scope, Structure** structureSlot)
index ab54c8434af9deb992ce98f830273739db4ec5ee..3171d747e6e1e7477d05c5527d4ef96b65b9ee97 100644 (file)
@@ -661,8 +661,8 @@ void JIT::emitSlow_op_resolve_scope(Instruction* currentInstruction, Vector<Slow
 
     linkSlowCase(iter);
     int32_t scope = currentInstruction[2].u.operand;
-    int32_t indentifierIndex = currentInstruction[3].u.operand;
-    callOperation(operationResolveScope, dst, scope, indentifierIndex);
+    int32_t identifierIndex = currentInstruction[3].u.operand;
+    callOperation(operationResolveScope, dst, scope, identifierIndex);
 }
 
 void JIT::emitLoadWithStructureCheck(int scope, Structure** structureSlot)
index a1a283ae51a45b37924fe53d587b6476d2e5f161..35ee302d7191d0de404e825972be2f4b7775e13c 100644 (file)
@@ -1513,7 +1513,7 @@ template <class TreeBuilder> TreeClassExpression Parser<LexerType>::parseClass(T
         if (isStaticMethod)
             next();
 
-        matchOrFail(IDENT, "Expected an indentifier");
+        matchOrFail(IDENT, "Expected an identifier");
 
         const CommonIdentifiers& propertyNames = *m_vm->propertyNames;
         const Identifier& ident = *m_token.m_data.ident;