Update data files to Unicode 5.1

RESOLVED FIXED in mozilla1.9.1

Status

()

defect
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: smontagu, Assigned: smontagu)

Tracking

({fixed1.9.1})

unspecified
mozilla1.9.1
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

()

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

11 years ago
Unicode 5.1 was released today. See http://wiki.mozilla.org/I18n:Updating_Unicode_version for instructions on updating data files.
Assignee

Updated

11 years ago
Blocks: 438571
Assignee

Comment 1

11 years ago
Posted patch Patch (obsolete) — Splinter Review
Assignee

Updated

11 years ago
Attachment #328251 - Flags: review?(jdaggett)
Assignee

Comment 2

11 years ago
The patch needs to be regenerated with the fix to bug 445626.
Depends on: 445626
Assignee

Updated

11 years ago
Target Milestone: mozilla2.0 → mozilla1.9.1
Simon, do you have an updated version of this?
Assignee

Comment 4

11 years ago
Posted patch Updated patchSplinter Review
Attachment #328251 - Attachment is obsolete: true
Attachment #347156 - Flags: review?
Attachment #328251 - Flags: review?(jdaggett)
Assignee

Updated

11 years ago
Attachment #347156 - Flags: review? → review?(jdaggett)
Comment on attachment 347156 [details] [diff] [review]
Updated patch

Looks good.  Just curious, is bug 210501 really still an open issue?
Attachment #347156 - Flags: review?(jdaggett) → review+
Assignee

Comment 6

11 years ago
(In reply to comment #5)
> (From update of attachment 347156 [details] [diff] [review])
> Just curious, is bug 210501 really still an open issue?

Yes. Fixing it would require either making the data tables significantly larger or rewriting nsICaseConversion to be stateful, and neither one seems justified when only one rare script that requires case conversions (the Deseret Alphabet) is encoded outside the BMP.
Assignee

Comment 7

11 years ago
Comment on attachment 347156 [details] [diff] [review]
Updated patch

I'd like to get this in for 1.9.1 to maintain our policy of supporting the most recent Unicode version in our releases.
Attachment #347156 - Flags: approval1.9.1?
Assignee

Comment 8

11 years ago
Pushed to trunk as http://hg.mozilla.org/mozilla-central/rev/b7d98dc8d246
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee

Updated

11 years ago
Flags: in-testsuite+
Comment on attachment 347156 [details] [diff] [review]
Updated patch

a191=beltzner
Attachment #347156 - Flags: approval1.9.1? → approval1.9.1+
You need to log in before you can comment on or make changes to this bug.