Closed Bug 746285 Opened 8 years ago Closed 8 years ago

pack nsXBLBinding more carefully for 64-bit

Categories

(Core :: XBL, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
nsXBLBinding is laid out like so:

  int mRefCnt;
  nsIContent *mBoundElement;
  nsClassHashtable* mInsertionPointTable;
  bool mIsStyleBinding;
  bool mMarkedForDeath;

which introduces padding after mRefCnt and at the end of the structure.  Moving the booleans up into the padding required after mRefCnt saves a word at the end of the structure.
Attachment #615828 - Flags: review?(bzbarsky)
Comment on attachment 615828 [details] [diff] [review]
patch

r=me
Attachment #615828 - Flags: review?(bzbarsky) → review+
http://hg.mozilla.org/projects/birch/rev/25ed7d0b8433
Assignee: nobody → nfroyd
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/25ed7d0b8433
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.