Closed Bug 608940 Opened 11 years ago Closed 10 years ago

remove unused function gfxTextRun::Clone

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jfkthame, Assigned: jfkthame)

Details

Attachments

(1 file)

Noticed while checking textrun allocation safety issues.... We have a (virtual) function Clone() on gfxTextRun, but as far as I can tell it's not used anywhere; I think we should remove it.

(If we *did* need a function to clone textruns, we should design it differently anyhow - there shouldn't be any need to pass the original text as a parameter to Clone(), as the textrun already either owns or refers to a copy.)
Attachment #487536 - Flags: review?(jdaggett)
Attachment #487536 - Flags: review?(jdaggett) → review+
http://hg.mozilla.org/mozilla-central/rev/60ea7d2f4289
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.