Closed
Bug 1416155
Opened 7 years ago
Closed 3 months ago
Get rid of JS_ParseJSONWithReviver
Categories
(Core :: JavaScript Engine, enhancement, P5)
Core
JavaScript Engine
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: baku, Unassigned)
References
Details
Attachments
(1 file)
4.19 KB,
patch
|
Details | Diff | Splinter Review |
This function is not needed anymore because of bug 1347515
Reporter | ||
Comment 1•7 years ago
|
||
Assignee: nobody → amarchesini
Attachment #8927245 -
Flags: review?(evilpies)
Comment 2•7 years ago
|
||
I don't think we need to remove this. I was probably thinking about this https://searchfox.org/mozilla-central/rev/30ead7d1ae5bf95b8bc0fd67b950cd46ca05e32c/js/src/json.cpp#967 when I mentioned we could simplify something. I also thought we had special error handling for the old JSON code, but apparently that is just for eval. Maybe somebody else knows something we could simplify now that nsJSON was removed.
Updated•7 years ago
|
Attachment #8927245 -
Flags: review?(evilpies)
Updated•7 years ago
|
status-firefox58:
--- → fix-optional
Priority: -- → P5
Reporter | ||
Comment 3•7 years ago
|
||
Better if JS folks work on this bug.
Assignee: amarchesini → nobody
Comment 4•7 years ago
|
||
https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Move_fix-optionals
status-firefox59:
--- → ?
Updated•2 years ago
|
Severity: normal → S3
Updated•3 months ago
|
Status: NEW → RESOLVED
Closed: 3 months ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•