Closed Bug 132660 Opened 20 years ago Closed 20 years ago

Implement OS/2 fontmetrics using locale based collation

Categories

(Core :: XUL, defect)

x86
OS/2
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: jhpedemonte, Assigned: mkaply)

Details

Attachments

(1 file, 2 obsolete files)

When sorting the font list in EnumerateAllFonts, use locale bases nsCollation.
Attached patch patch (obsolete) — Splinter Review
This implements using nsCollation for sorting.	The patch also includes many
changes, mostly performance changes taking advantage of the nsCollation code
for comparing strings.
Attached patch updated patch (obsolete) — Splinter Review
This patch includes more performance tuning, memory leak fixes, and a better
implementation of DBCS font handling.
Attachment #75476 - Attachment is obsolete: true
mkaply, could you take ownership of this bug?
Taking
Assignee: jaggernaut → mkaply
Add special code to handle hosed Chinese font names.
Attachment #77104 - Attachment is obsolete: true
Comment on attachment 77926 [details] [diff] [review]
another updated diff

r=mkaply for JAviers original diff
r=pedemont for my change to Javiers diff
sr=blizzard (platform specific code)
Attachment #77926 - Flags: superreview+
Attachment #77926 - Flags: review+
Comment on attachment 77926 [details] [diff] [review]
another updated diff

a=rjesup@wgate.com for drivers
Attachment #77926 - Flags: approval+
Fix checked in.
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Verified - Javier's code is working fine.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.