Closed
Bug 1065716
Opened 10 years ago
Closed 8 years ago
Move jsonparser to vm/JSONParser
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: evilpie, Assigned: evilpie)
References
Details
Attachments
(1 file)
4.65 KB,
patch
|
Waldo
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8487556 -
Flags: review?(jwalden+bmo)
Updated•10 years ago
|
Attachment #8487556 -
Flags: review?(jwalden+bmo) → review+
Comment 2•10 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•10 years ago
|
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/cd26af446244
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cd26af446244
Assignee | ||
Updated•8 years ago
|
Summary: Move all js*.{h,cpp} files into subdirectories → Move jsonparser to vm/JSONParser
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → evilpies
Keywords: leave-open
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•