Closed Bug 1808216 Opened 1 year ago Closed 1 year ago

Improve StringBuffer API a bit

Categories

(Core :: JavaScript Engine, task, P1)

task

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox110 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(3 files)

We can improve StringBuffer a bit by taking either a JSContext* or a FrontendContext* as argument. This also lets us remove the JSContext* argument from FoldConstants.

In StringBuffer::inflateChars we now copy the alloc policy from the Latin1 buffer.
This lets us remove the fc_ and arenaId_ fields from StringBuffer.

Severity: -- → N/A
Priority: -- → P1
Pushed by jdemooij@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fc7c37faf9b7
part 1 - Change StringBuffer constructor to take either JSContext* or FrontendContext* but not both. r=arai
https://hg.mozilla.org/integration/autoland/rev/0e45a51c5b25
part 2 - Simplify JSStringBuilder API. r=arai
https://hg.mozilla.org/integration/autoland/rev/97d3a9e5a867
part 3 - Remove JSContext* argument from FoldConstants. r=arai
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: