Closed
Bug 1065716
Opened 11 years ago
Closed 9 years ago
Move jsonparser to vm/JSONParser
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: evilpies, Assigned: evilpies)
References
Details
Attachments
(1 file)
4.65 KB,
patch
|
Waldo
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8487556 -
Flags: review?(jwalden+bmo)
Updated•11 years ago
|
Attachment #8487556 -
Flags: review?(jwalden+bmo) → review+
![]() |
||
Comment 2•11 years ago
|
||
One plan that we have informally talked about a while ago is that, while we should eventually move everything from js/src/js* files into subdirs, we shouldn't just move them en masse but, rather, try to split them up into logical units of .h/.cpp files as we go. In that way, js/src/js* is a TODO list. The real targets here are the fatties like jsobj, jsscript, jsfun. Perhaps you could initially only move the files that are relatively self-contained and then incrementally surgically eviscerate the remaining files?
Assignee | ||
Updated•11 years ago
|
Assignee | ||
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Assignee | ||
Updated•9 years ago
|
Summary: Move all js*.{h,cpp} files into subdirectories → Move jsonparser to vm/JSONParser
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → evilpies
Keywords: leave-open
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•