Closed
Bug 498871
Opened 15 years ago
Closed 15 years ago
Remove use of unnecessary QBrush pointer
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: tobias.hunger, Assigned: tobias.hunger)
References
Details
Attachments
(1 file)
4.05 KB,
patch
|
pavlov
:
review+
pavlov
:
superreview+
|
Details | Diff | Splinter Review |
QBrush is a implizitly shared datastructure. There is on need to layer "explicit pointer based sharing" atop of it.
Attachment #383685 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 1•15 years ago
|
||
This depends on #498843 for the whitespace cleanup.
Depends on: 498843
Assignee | ||
Comment 2•15 years ago
|
||
Ping?
Updated•15 years ago
|
Attachment #383685 -
Flags: superreview+
Attachment #383685 -
Flags: review?(mark.finkle)
Attachment #383685 -
Flags: review+
Updated•15 years ago
|
Keywords: checkin-needed
Updated•15 years ago
|
Assignee: nobody → tobias.hunger
Assignee | ||
Comment 3•15 years ago
|
||
Ping? Would be nice to see this in mozilla before the code changes again.
Assignee | ||
Comment 4•15 years ago
|
||
Could somebody please check this in? Thanks!
Assignee | ||
Updated•15 years ago
|
Whiteboard: checkin-needed
Comment 5•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/54eb4dc88c9b
Status: NEW → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: checkin-needed
Target Milestone: --- → mozilla1.9.2a1
You need to log in
before you can comment on or make changes to this bug.
Description
•