Closed Bug 701584 Opened 12 years ago Closed 12 years ago

Mini-abstract some offsetofs noted in the initial patch for bug 699185

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla11

People

(Reporter: Waldo, Assigned: Waldo)

Details

Attachments

(1 file)

Patch
7.68 KB, patch
cdleary
: review+
Details | Diff | Splinter Review
Attached patch Patch — — Splinter Review 
Style for deep offsetofs in classes seems to be heading toward static inline methods that return them.  Continue this trend in a few more places.

Naming the JSFunction::U::n struct isn't really necessary, but it makes it consistent with Scripted right next to it.
Attachment #573702 - Flags: review?(cdleary)
Attachment #573702 - Flags: review?(cdleary) → review+
https://hg.mozilla.org/mozilla-central/rev/f4183f93f844
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.