Closed Bug 1551489 Opened 7 months ago Closed 7 months ago

[BinAST] Accept "context" as format in parseBin shell built-in

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox68 --- wontfix
firefox69 --- fixed

People

(Reporter: arai, Assigned: arai)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

parseBin shell built-in has options parameter, that has format field.
currently it accepts only multipart, and invokes multipart parser.

We should add context format and invoke context parser.

Pushed by arai_a@mac.com:
https://hg.mozilla.org/integration/autoland/rev/d68c95ef507d
Part 1: Fix BinAST build script to invoke mach clang-format properly. r=Yoric
https://hg.mozilla.org/integration/autoland/rev/43dd49397f14
Part 2: Instantiate BinASTParser<BinASTTokenReaderContext>. r=Yoric
https://hg.mozilla.org/integration/autoland/rev/678d5b056849
Part 3: Support { format: "context" } in parseBin shell built-in. r=Yoric
Status: ASSIGNED → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
No longer blocks: 1549952
Blocks: 1557668
You need to log in before you can comment on or make changes to this bug.