Closed Bug 695549 Opened 13 years ago Closed 13 years ago

Rename everything in the front end

Categories

(Core :: JavaScript Engine, defect)

Other Branch
x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: jorendorff, Assigned: jorendorff)

Details

Attachments

(4 files)

The front end is old. It has a lot of names like JSParseNode and JSStmtInfo that should be js::ParseNode and js::StmtInfo.
Assignee: general → jorendorff
Attachment #567919 - Flags: review?(luke)
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)
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)
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 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+
Attachment #567925 - Flags: review?(luke) → review+
Attachment #567926 - Flags: review?(luke) → review+
Attachment #567929 - Flags: review?(luke) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: