Closed
Bug 236403
Opened 21 years ago
Closed 21 years ago
Clean up nsFontMetricsOS2 array usage
Categories
(Core Graveyard :: GFX: OS/2, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jhpedemonte, Assigned: jhpedemonte)
Details
Attachments
(1 file, 1 obsolete file)
|
37.22 KB,
patch
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•21 years ago
|
||
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•21 years ago
|
Attachment #142865 -
Flags: review?(mkaply)
|
||
Comment 2•21 years ago
|
||
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•21 years ago
|
||
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•21 years ago
|
||
Last patch checked in on verbal approval from mkaply. Marking fixed.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•16 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•