Remove guard and guardedHandlers properties from try/catch nodes

Closes acornjs#323
deathcap · Oct 9, 2015 · 1ecae1a · 1ecae1a
1 parent d956017
commit 1ecae1a
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 11 deletions.
diff --git a/src/loose/statement.js b/src/loose/statement.js
@@ -125,7 +125,6 @@ lp.parseStatement = function() {
       this.expect(tt.parenL)
       clause.param = this.toAssignable(this.parseExprAtom(), true)
       this.expect(tt.parenR)
-      clause.guard = null
       clause.body = this.parseBlock()
       node.handler = this.finishNode(clause, "CatchClause")
     }

diff --git a/src/statement.js b/src/statement.js
@@ -256,11 +256,9 @@ pp.parseTryStatement = function(node) {
     clause.param = this.parseBindingAtom()
     this.checkLVal(clause.param, true)
     this.expect(tt.parenR)
-    clause.guard = null
     clause.body = this.parseBlock()
     node.handler = this.finishNode(clause, "CatchClause")
   }
-  node.guardedHandlers = empty
   node.finalizer = this.eat(tt._finally) ? this.parseBlock() : null
   if (!node.handler && !node.finalizer)
     this.raise(node.start, "Missing catch or finally clause")

diff --git a/test/compare/esprima.js b/test/compare/esprima.js
@@ -1937,7 +1937,6 @@
         finishTryStatement: function (block, handler, finalizer) {
             this.type = Syntax.TryStatement;
             this.block = block;
-            this.guardedHandlers = [];
             this.handlers = handler ? [ handler ] : [];
             this.handler = handler;
             this.finalizer = finalizer;

diff --git a/test/tests-harmony.js b/test/tests-harmony.js
@@ -14769,7 +14769,6 @@ test("try {} catch ({message}) {}", {
           }
         }]
       },
-      guard: null,
       body: {
         type: "BlockStatement",
         range: [25, 27],

diff --git a/test/tests.js b/test/tests.js
@@ -22066,7 +22066,6 @@ test("try { } catch (e) { }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [],
@@ -22154,7 +22153,6 @@ test("try { } catch (eval) { }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [],
@@ -22242,7 +22240,6 @@ test("try { } catch (arguments) { }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [],
@@ -22330,7 +22327,6 @@ test("try { } catch (e) { say(e) }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [
@@ -22632,7 +22628,6 @@ test("try { doThat(); } catch (e) { say(e) }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [
@@ -22819,7 +22814,6 @@ test("try { doThat(); } catch (e) { say(e) } finally { cleanup(stuff) }", {
               }
             }
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             body: [
-Original file line number
+Diff line change
@@ Expand Up / @@ -14769,7 +14769,6 @@ test("try {} catch ({message}) {}", { @@
               }
             }]
           },
-          guard: null,
           body: {
             type: "BlockStatement",
             range: [25, 27],
@@ Expand Down @@