Title: [240686] trunk/Source/_javascript_Core
- Revision
- 240686
- Author
- [email protected]
- Date
- 2019-01-29 15:10:46 -0800 (Tue, 29 Jan 2019)
Log Message
Remove FIXME for Annex B.3.5's "for-of var" subcase.
Rubber-stamped by Yusuke Suzuki.
This subcase is removed from the spec in https://github.com/tc39/ecma262/pull/1393.
* parser/Parser.h:
(JSC::Parser::declareHoistedVariable):
Modified Paths
Diff
Modified: trunk/Source/_javascript_Core/ChangeLog (240685 => 240686)
--- trunk/Source/_javascript_Core/ChangeLog 2019-01-29 22:40:44 UTC (rev 240685)
+++ trunk/Source/_javascript_Core/ChangeLog 2019-01-29 23:10:46 UTC (rev 240686)
@@ -1,3 +1,14 @@
+2019-01-29 Ross Kirsling <[email protected]>
+
+ Remove FIXME for Annex B.3.5's "for-of var" subcase.
+
+ Rubber-stamped by Yusuke Suzuki.
+
+ This subcase is removed from the spec in https://github.com/tc39/ecma262/pull/1393.
+
+ * parser/Parser.h:
+ (JSC::Parser::declareHoistedVariable):
+
2019-01-29 Mark Lam <[email protected]>
Remove unneeded CPU(BIG_ENDIAN) handling in LLInt after new bytecode format.
Modified: trunk/Source/_javascript_Core/parser/Parser.h (240685 => 240686)
--- trunk/Source/_javascript_Core/parser/Parser.h 2019-01-29 22:40:44 UTC (rev 240685)
+++ trunk/Source/_javascript_Core/parser/Parser.h 2019-01-29 23:10:46 UTC (rev 240686)
@@ -1230,7 +1230,6 @@
ASSERT(i < m_scopeStack.size());
while (true) {
// Annex B.3.5 exempts `try {} catch (e) { var e; }` from being a syntax error.
- // FIXME: This exemption should not apply if the var declaration is a for-of initializer.
if (m_scopeStack[i].hasLexicallyDeclaredVariable(*ident) && !m_scopeStack[i].isSimpleCatchParameterScope())
return DeclarationResult::InvalidDuplicateDeclaration;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
