Closed
Bug 735419
Opened 12 years ago
Closed 12 years ago
gfxTextRun constructor has unused parameter 'aText'
Categories
(Core :: Graphics, defect, P5)
Core
Graphics
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: jwir3, Assigned: jfkthame)
Details
Attachments
(1 file)
|
10.62 KB,
patch
|
jtd
:
review+
|
Details | Diff | Splinter Review |
The aText parameter in this function: http://mxr.mozilla.org/mozilla-central/source/gfx/thebes/gfxFont.cpp#4040 Is unused in the body of the function, and should probably be removed.
|
Assignee | |
Comment 1•12 years ago
|
||
Yup, it can go - thanks for catching that. It was made redundant by bug 703100, but I forgot to remove it there.
Assignee: nobody → jfkthame
|
|
Assignee | |
Comment 2•12 years ago
|
||
Attachment #605669 -
Flags: review?(jdaggett)
|
||
Updated•12 years ago
|
Attachment #605669 -
Flags: review?(jdaggett) → review+
|
|
Assignee | |
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/507417303ad2
Target Milestone: --- → mozilla14
|
||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/507417303ad2
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•