Closed Bug 1858803 Opened 2 years ago Closed 2 years ago

Add non-GC JSON parsing API

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

RESOLVED FIXED
124 Branch
Tracking Status
firefox124 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

In bug 1858692, L10nArgsHelperDict need to create a global in order to parse JSON, which is inefficient.
Given the underlying code is already templated and introducing another variant is easy,
we can look into adding a new non-GC API for JSON parse.

it could be either:

  • (a) returns a non-GC structure
  • (b) receive callbacks and call them for object start, object end, etc, like SAX

and the option (b) seems to work better in bug 1858692 case.

Assignee: nobody → arai.unmht
Status: NEW → ASSIGNED
Attachment #9358222 - Attachment description: WIP: Bug 1858803 - Add JS::ParseJSONWithHandler. → WIP: Bug 1858803 - Add JS::ParseJSONWithHandler. r=bthrall!
Severity: -- → S3
Priority: -- → P3
Blocks: sm-meta
Severity: S3 → N/A
Summary: Add non-GC JSON API → Add non-GC JSON parsing API
See Also: → 1860460
Blocks: 1861819
Attachment #9358222 - Attachment description: WIP: Bug 1858803 - Add JS::ParseJSONWithHandler. r=bthrall! → Bug 1858803 - Add JS::ParseJSONWithHandler. r=bthrall!
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/ff3451e8c9b2 Add JS::ParseJSONWithHandler. r=bthrall
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 124 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: