Closed
Bug 695549
Opened 13 years ago
Closed 13 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•13 years ago
|
||
Assignee: general → jorendorff
Attachment #567919 -
Flags: review?(luke)
Assignee | ||
Comment 2•13 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•13 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•13 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•13 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•13 years ago
|
Attachment #567925 -
Flags: review?(luke) → review+
Updated•13 years ago
|
Attachment #567926 -
Flags: review?(luke) → review+
Updated•13 years ago
|
Attachment #567929 -
Flags: review?(luke) → review+
Assignee | ||
Comment 6•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/318083d40e1c https://hg.mozilla.org/integration/mozilla-inbound/rev/39cbf5aa4c4c https://hg.mozilla.org/integration/mozilla-inbound/rev/1c151e4cc979 https://hg.mozilla.org/integration/mozilla-inbound/rev/a4bd037eb252
Comment 7•13 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: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in
before you can comment on or make changes to this bug.
Description
•