475538 - Remove obsolete files in intl/uconv

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Simon Montagu :smontagu]

The following files in intl/uconv can be removed:

ucvibm/cp1046.uf, cp1046.ut -- conversion tables for IBM1046 encoding, which was removed in bug 413613

ucvlatin/8859-11.ut, tis620.ut -- these were replaced by cp874.ut in bug 127755.

ucvlatin/macukrai.ut, macukrai.uf, nsUnicodeToMacUkrainian.* nsMacUkrainianToUnicode.* -- MacUkrainian is identical with MacCyrillic, see http://www.unicode.org/Public/MAPPINGS/VENDORS/APPLE/UKRAINE.TXT

Comment 1

[Simon Montagu :smontagu]

Attachment: Patch

Comment 2

[Masatoshi Kimura [:emk]]

Is MacUkrainian still requird in charset menu whilst it's exactly the same as MacCyrillic?

Comment 3

[Simon Montagu :smontagu]

(In reply to comment #2)
> Is MacUkrainian still requird in charset menu whilst it's exactly the same as
> MacCyrillic?

We're in string freeze right now, so can we handle that in a follow-up bug?

Comment 4

[Masatoshi Kimura [:emk]]

(In reply to comment #3)
> We're in string freeze right now, so can we handle that in a follow-up bug?
Are you planning to land this on the branch?
If you aren't, string freeze is not an issue because l10n has a 1.9.1 branch.
For example, bug 474655 was landed only on the trunk. Am I missing something?

Comment 5

[Simon Montagu :smontagu]

Attachment: Updated patch including removing unused lines from charsetData.properties

OK, but I still want to make a separate bug for changing the charset menu. Because the menu is in a localizable file (toolkit/locales/en-US/chrome/global/intl.properties), localizations will have to include the patch, so it will be easier to have a separate bug with only that change.

Also, if we are already going to change all localizations, I think that we could actually remove all the Mac- charsets from the menus. I had a reviewed patch to do this some years ago in bug 203838, but never checked it in because it caused a regression that I didn't understand. I now know how to prevent that.

Comment 6

[Masatoshi Kimura [:emk]]

Comment on attachment 360679 [details] [diff] [review]
Updated patch including removing unused lines from charsetData.properties

OK, let's separate this bug from a localizable part.

Comment 7

[Simon Montagu :smontagu]

http://hg.mozilla.org/mozilla-central/rev/29f5a8085435