Closed
Bug 1224048
Opened 10 years ago
Closed 10 years ago
Use stable hashing for JSON parsing stack structures
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla45
| Tracking | Status | |
|---|---|---|
| firefox45 | --- | fixed |
People
(Reporter: terrence, Assigned: terrence)
References
Details
Attachments
(1 file)
|
1.69 KB,
patch
|
jonco
:
review+
|
Details | Diff | Splinter Review |
There are only 3 sites we need to change, so on one hand this is trivial. On the other hand, I'd much prefer to just let these be handled by default when we make the global switch over to DefaultHasher<JSObject*>:public MovableCellHasher<JSObject*>.
|
Assignee | |
Comment 1•10 years ago
|
||
Trivial.
|
|
Assignee | |
Comment 2•10 years ago
|
||
|
||
Updated•10 years ago
|
Attachment #8686801 -
Flags: review?(jcoppeard) → review+
|
|
Assignee | |
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8fc43eda8c2a5ca6d3561d8f600c4218c888640a
Bug 1224048 - Use stable hashing for the temporary tables in the JSON parser; r=jonco
|
||
Comment 4•10 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
You need to log in
before you can comment on or make changes to this bug.
Description
•