Closed
Bug 1416155
Opened 8 years ago
Closed 1 year 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•8 years ago
|
||
Assignee: nobody → amarchesini
Attachment #8927245 -
Flags: review?(evilpies)
|
||
Comment 2•8 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•8 years ago
|
Attachment #8927245 -
Flags: review?(evilpies)
|
||
Updated•8 years ago
|
status-firefox58:
--- → fix-optional
Priority: -- → P5
|
||
Comment 4•8 years ago
|
||
status-firefox59:
--- → ?
|
||
Updated•3 years ago
|
Severity: normal → S3
|
||
Updated•1 year ago
|
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•