Closed
Bug 645947
Opened 14 years ago
Closed 14 years ago
Remove unneeded copy constructors
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla5
People
(Reporter: jrmuizel, Assigned: jrmuizel)
Details
Attachments
(1 file)
|
1.26 KB,
patch
|
bjacob
:
review+
|
Details | Diff | Splinter Review |
This removes the redundant copy constructors on some gfx types. This will let the be POD which helps generate better code. Overall this saves 1224 bytes of text and 54465 bytes of total size in libxul.so with gcc-4.4
|
Assignee | |
Updated•14 years ago
|
Attachment #522584 -
Flags: review?(bjacob)
|
||
Updated•14 years ago
|
Assignee: nobody → jmuizelaar
|
||
Comment 1•14 years ago
|
||
Comment on attachment 522584 [details] [diff] [review] Remove unneeded copy constructors And you might want to apply the same treatment to gfxSize too, in gfxPoint.h
Attachment #522584 -
Flags: review?(bjacob) → review+
|
|
||
Comment 2•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/8466671a0fc1
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.2
You need to log in
before you can comment on or make changes to this bug.
Description
•