remove unused function gfxTextRun::Clone

RESOLVED FIXED

Status

()

Core
Graphics
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: jfkthame, Assigned: jfkthame)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 487536 [details] [diff] [review]
remove the gfxTextRun::Clone function

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)

Updated

7 years ago
Attachment #487536 - Flags: review?(jdaggett) → review+
(Assignee)

Comment 1

6 years ago
http://hg.mozilla.org/mozilla-central/rev/60ea7d2f4289
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.