summaryrefslogtreecommitdiffstats
path: root/js/src/frontend/Parser.h
diff options
context:
space:
mode:
authorGaming4JC <g4jc@hyperbola.info>2019-07-13 23:20:19 -0400
committerGaming4JC <g4jc@hyperbola.info>2019-07-18 22:38:43 -0400
commitfe80b3d80d20241ff03338de4351d580796fd2c7 (patch)
tree169e4c59f59a972db0fd8d3d54780ec63406fae9 /js/src/frontend/Parser.h
parent7e510ee9b4dee7c2d15005baac89a2017f5673ef (diff)
downloadUXP-fe80b3d80d20241ff03338de4351d580796fd2c7.tar
UXP-fe80b3d80d20241ff03338de4351d580796fd2c7.tar.gz
UXP-fe80b3d80d20241ff03338de4351d580796fd2c7.tar.lz
UXP-fe80b3d80d20241ff03338de4351d580796fd2c7.tar.xz
UXP-fe80b3d80d20241ff03338de4351d580796fd2c7.zip
1357506 - Remove assert that constructorBox can only be set once when parsing classes.
Both asm.js and syntax parsing can abort and rewind parsing of an inner function. The bookkeeping to make sure that a class's constructor FunctionBox is only set once is not worth it -- duplicate constructor definitions already throw an early error.
Diffstat (limited to 'js/src/frontend/Parser.h')
-rw-r--r--js/src/frontend/Parser.h21
1 files changed, 3 insertions, 18 deletions
diff --git a/js/src/frontend/Parser.h b/js/src/frontend/Parser.h
index 65e46a65e..7866bc4fd 100644
--- a/js/src/frontend/Parser.h
+++ b/js/src/frontend/Parser.h
@@ -85,29 +85,14 @@ class ParseContext : public Nestable<ParseContext>
}
};
- class ClassStatement : public Statement
+ struct ClassStatement : public Statement
{
- FunctionBox* constructorBox_;
+ FunctionBox* constructorBox;
- public:
explicit ClassStatement(ParseContext* pc)
: Statement(pc, StatementKind::Class),
- constructorBox_(nullptr)
+ constructorBox(nullptr)
{ }
-
- void clearConstructorBoxForAbortedSyntaxParse(FunctionBox* funbox) {
- MOZ_ASSERT(constructorBox_ == funbox);
- constructorBox_ = nullptr;
- }
-
- void setConstructorBox(FunctionBox* funbox) {
- MOZ_ASSERT(!constructorBox_);
- constructorBox_ = funbox;
- }
-
- FunctionBox* constructorBox() const {
- return constructorBox_;
- }
};
// The intra-function scope stack.