gfxTextRun constructor has unused parameter 'aText'

RESOLVED FIXED in mozilla14

Status

()

Core
Graphics
P5
normal
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jwir3, Assigned: jfkthame)

Tracking

Trunk
mozilla14
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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

6 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

6 years ago
Created attachment 605669 [details] [diff] [review]
eliminate unused parameter in the gfxTextRun constructor
Attachment #605669 - Flags: review?(jdaggett)

Updated

6 years ago
Attachment #605669 - Flags: review?(jdaggett) → review+
(Assignee)

Comment 3

6 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/507417303ad2
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/507417303ad2
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.