799913 - Remove support for the Zapf Dingbats encoding and other dead-code encoder-only encodings

Status: RESOLVED FIXED

(Core :: Internationalization, defect)

Description

[Henri Sivonen (:hsivonen)]

We have support for x-zapf-dingbats but it does not appear to be exposed to the Web or used in font code:
http://mxr.mozilla.org/mozilla-central/ident?i=zapf&tree=mozilla-central&filter=

It was added by Frank Tang in 1999 with the helpful comment “add unicode encoders”.

Let’s remove it.

Comment 1

[Henri Sivonen (:hsivonen)]

Attachment: Remove dead code

Comment 2

[Henri Sivonen (:hsivonen)]

https://tbpl.mozilla.org/?tree=Try&rev=7f9558ee41f8

Comment 3

[Henri Sivonen (:hsivonen)]

Attachment: Remove dead code, fix property file comments

Awesome. We've had bogus comments in a property file for a long time, but the entry after the comments was useless, so no one noticed.

https://tbpl.mozilla.org/?tree=Try&rev=484564b87ce1

Comment 4

[Henri Sivonen (:hsivonen)]

Comment on attachment 8355210 [details] [diff] [review]
Remove dead code, fix property file comments

It seems to me that these encodings exist, because year ago the X11 gfx code knew about font-specific encodings and this stuff wasn't fully removed when X11 gfx moved to FreeType2, etc.

Comment 5

[Henri Sivonen (:hsivonen)]

Thanks.

https://hg.mozilla.org/integration/mozilla-inbound/rev/89b66dd9847a

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/89b66dd9847a