Transition to TryStatement single handler #1031

ariya · 2015-02-13T15:21:16Z

To facilitate the transition of TryStatement from the old handlers to the new handler (see #1030), Esprima should produce both properties. Once all the major downstream tools have been ported to use the handler only, handlers can be safely removed in the next major version.

In the current master branch:

> require('./esprima').parse('try{}catch(e){}')
{ type: 'Program',
  body: 
   [ { type: 'TryStatement',
       block: [Object],
       guardedHandlers: [],
       handlers: [Object],
       finalizer: null } ] }

Once this transition step is implemented, it will look like:

> require('./esprima').parse('try{}catch(e){}')
{ type: 'Program',
  body: 
   [ { type: 'TryStatement',
       block: [Object],
       guardedHandlers: [],
       handlers: [Object],
       handler: [Object],
       finalizer: null } ] }

Refs jquery#1031

Refs #1031

mikesherov modified the milestone: 2.1 Feb 13, 2015

ariya added a commit to ariya/esprima that referenced this issue Feb 14, 2015

TryStatement needs to have handler property.

bd84131

Refs jquery#1031

ariya mentioned this issue Feb 14, 2015

Trystatement handler #1034

Closed

ariya added a commit that referenced this issue Feb 14, 2015

TryStatement needs to have handler property.

6577760

Refs #1031

ariya closed this in 5372ada Feb 14, 2015

btmills mentioned this issue Mar 3, 2015

Use TryStatement handler property eslint/eslint#1930

Closed

millermedeiros mentioned this issue Mar 11, 2015

Try / Catch statements are not indented properly millermedeiros/esformatter#264

Closed

jquery / esprima

Transition to TryStatement single handler #1031

Transition to TryStatement single handler #1031

ariya commented Feb 13, 2015

jquery / esprima

Join GitHub today

GitHub is where the world builds software

Transition to TryStatement single handler #1031

Transition to TryStatement single handler #1031

Comments

ariya commented Feb 13, 2015

Essential cookies

Always active

Analytics cookies