Closed
Bug 695549
Opened 14 years ago
Closed 14 years ago
Rename everything in the front end
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: jorendorff, Assigned: jorendorff)
Details
Attachments
(4 files)
465.00 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
208.96 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
14.06 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
37.86 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
The front end is old. It has a lot of names like JSParseNode and JSStmtInfo that should be js::ParseNode and js::StmtInfo.
![]() |
Assignee | |
Comment 1•14 years ago
|
||
Assignee: general → jorendorff
Attachment #567919 -
Flags: review?(luke)
![]() |
Assignee | |
Comment 2•14 years ago
|
||
js_PopStatement became js::frontend::PopStatementTC rather than js::frontend::PopStatement, because there is already a function named PopStatement, and that one is actually called rather more than this one.
Attachment #567925 -
Flags: review?(luke)
![]() |
Assignee | |
Comment 3•14 years ago
|
||
It is subjective whether js or js::frontend is better. These functions (at least js_IsIdentifier) actually are used outside of the frontend/ subdirectory.
Attachment #567926 -
Flags: review?(luke)
![]() |
Assignee | |
Comment 4•14 years ago
|
||
Elective surgery. I want to rename it to match the filename. (methodjit has a class js::mjit::Compiler. No collision.)
While I was in here I grepped for places where the old filenames jsscan, jsparse, jsemit were still used, so this patch fixes several of those (and deletes some bafflingly ancient super-dead code).
Attachment #567929 -
Flags: review?(luke)
![]() |
||
Comment 5•14 years ago
|
||
Comment on attachment 567919 [details] [diff] [review]
Part 1, rename types and put enums into namespace js - v1
I like it all.
Attachment #567919 -
Flags: review?(luke) → review+
![]() |
||
Updated•14 years ago
|
Attachment #567925 -
Flags: review?(luke) → review+
![]() |
||
Updated•14 years ago
|
Attachment #567926 -
Flags: review?(luke) → review+
![]() |
||
Updated•14 years ago
|
Attachment #567929 -
Flags: review?(luke) → review+
![]() |
Assignee | |
Comment 6•14 years ago
|
||
Comment 7•14 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/318083d40e1c
https://hg.mozilla.org/mozilla-central/rev/39cbf5aa4c4c
https://hg.mozilla.org/mozilla-central/rev/1c151e4cc979
https://hg.mozilla.org/mozilla-central/rev/a4bd037eb252
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in
before you can comment on or make changes to this bug.
Description
•