Closed Bug 964016 Opened 6 years ago Closed 6 years ago

Replace some JS macros with their MFBT MOZ equivalents (e.g. s/JS_ALWAYS_INLINE/MOZ_ALWAYS_INLINE/)

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla29

People

(Reporter: cpeterson, Assigned: cpeterson)

Details

Attachments

(5 files)

Part 1: Replace JS_LIKELY with MOZ_LIKELY.
Attachment #8365637 - Flags: review?(luke)
Part 2: Replace JS_UNLIKELY with MOZ_UNLIKELY.
Attachment #8365638 - Flags: review?(luke)
Part 3: Replace JS_NEVER_INLINE with MOZ_NEVER_INLINE.
Attachment #8365639 - Flags: review?(luke)
Part 4: Replace JS_ALWAYS_INLINE with MOZ_ALWAYS_INLINE.
Attachment #8365640 - Flags: review?(luke)
Attachment #8365640 - Attachment description: R?_964016_part-4-JS_ALWAYS_INLINE.patch → part-4-JS_ALWAYS_INLINE.patch
Part 5: Remove unused JS_BEGIN_EXTERN_C and JS_END_EXTERN_C macros.
Attachment #8365641 - Flags: review?(luke)
This all seems fine to me, but let's check with Waldo whether this is part of the official plan.
Flags: needinfo?(jwalden+bmo)
Can't imagine why it wouldn't be!  This same-in-name-only stuff needs to die sooner or later, this is a good day to die.  (Hard.)
Flags: needinfo?(jwalden+bmo)
Attachment #8365637 - Flags: review?(luke) → review+
Attachment #8365638 - Flags: review?(luke) → review+
Attachment #8365639 - Flags: review?(luke) → review+
Attachment #8365640 - Flags: review?(luke) → review+
Attachment #8365641 - Flags: review?(luke) → review+
You need to log in before you can comment on or make changes to this bug.