Consider removing Reflect.parse's "builder" API

NEW
Unassigned

Status

()

Core
JavaScript Engine
P5
normal
15 days ago
15 days ago

People

(Reporter: jorendorff, Unassigned)

Tracking

unspecified
Points:
---

Firefox Tracking Flags

(firefox58 fix-optional)

Details

(Reporter)

Description

15 days ago
Reflect.parse takes an optional `builder` argument. If present, it's an object with a ton of methods, and it's used to build the resulting AST. (The default, if `builder` is missing, is to build a straightforward JSON AST.)

https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Parser_API#Builder_objects

This API is brittle. Every time we add anything to JS syntax--or remove an obsolete feature--all code using the builder API breaks. Therefore I doubt anyone is using it and we should remove it.
(Reporter)

Updated

15 days ago
status-firefox58: --- → fix-optional
You need to log in before you can comment on or make changes to this bug.