Clean up nsFontMetricsOS2 array usage

RESOLVED FIXED

Status

RESOLVED FIXED
15 years ago
10 years ago

People

(Reporter: jhpedemonte, Assigned: jhpedemonte)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

15 years ago
 
(Assignee)

Comment 1

15 years ago
Created attachment 142865 [details] [diff] [review]
patch

Switched from nsVoidArray to nsTHashtable.  Performance has stayed the same,
but the code is cleaner, and there are minor size savings in code size and
memory usage.
(Assignee)

Updated

15 years ago
Attachment #142865 - Flags: review?(mkaply)
Comment on attachment 142865 [details] [diff] [review]
patch

r=mkaply, sr=blizzard (platform specific code)
Attachment #142865 - Flags: superreview+
Attachment #142865 - Flags: review?(mkaply)
Attachment #142865 - Flags: review+
(Assignee)

Comment 3

15 years ago
Created attachment 143328 [details] [diff] [review]
patch v1.1

This patch fixes a few bugs in the old patch: (1) need to compare fonts case
insensitive, and (2) need to set the correct codepage for symbol fonts.
Attachment #142865 - Attachment is obsolete: true
(Assignee)

Comment 4

15 years ago
Last patch checked in on verbal approval from mkaply.  Marking fixed.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.