Closed
Bug 132660
Opened 24 years ago
Closed 24 years ago
Implement OS/2 fontmetrics using locale based collation
Categories
(Core :: XUL, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: jhpedemonte, Assigned: mkaply)
Details
Attachments
(1 file, 2 obsolete files)
|
38.93 KB,
patch
|
mkaply
:
review+
mkaply
:
superreview+
jesup
:
approval+
|
Details | Diff | Splinter Review |
When sorting the font list in EnumerateAllFonts, use locale bases nsCollation.
| Reporter | ||
Comment 1•24 years ago
|
||
This implements using nsCollation for sorting. The patch also includes many
changes, mostly performance changes taking advantage of the nsCollation code
for comparing strings.
| Reporter | ||
Comment 2•24 years ago
|
||
This patch includes more performance tuning, memory leak fixes, and a better
implementation of DBCS font handling.
Attachment #75476 -
Attachment is obsolete: true
Comment 3•24 years ago
|
||
mkaply, could you take ownership of this bug?
| Assignee | ||
Comment 5•24 years ago
|
||
Add special code to handle hosed Chinese font names.
Attachment #77104 -
Attachment is obsolete: true
| Assignee | ||
Comment 6•24 years ago
|
||
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 7•24 years ago
|
||
Comment on attachment 77926 [details] [diff] [review]
another updated diff
a=rjesup@wgate.com for drivers
Attachment #77926 -
Flags: approval+
| Assignee | ||
Comment 8•24 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 9•24 years ago
|
||
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.
Description
•