Some nsFontLangGroup entries missing in X11 font code

RESOLVED FIXED

Status

()

Core
Internationalization
--
critical
RESOLVED FIXED
15 years ago
15 years ago

People

(Reporter: Roland Mainz, Assigned: Roland Mainz)

Tracking

({intl})

Trunk
All
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

15 years ago
[Per bug 78201 comment 51]:
Brian Stell wrote:
> would you add a font-lang group for arabic?
>
>  nsFontLangGroup FLG_JA =      { "ja", nsnull };
>  nsFontLangGroup FLG_KO =      { "ko", nsnull };
> +nsFontLangGroup FLG_AR =      { "ar", nsnull };
>  nsFontLangGroup FLG_NONE =    { nsnull , nsnull };

We'll miss a bunch of other nsFontLangGroup entries, too... ;-(
(Assignee)

Comment 1

15 years ago
Suffering is all mine... ;-(
Status: NEW → ASSIGNED
(Assignee)

Comment 2

15 years ago
Created attachment 101645 [details] [diff] [review]
Prototype patch for 2002-09-29-08-trunk

Prototype patch for getting from comments.
I am sure I am missing some things here - therefore I need the feedback from
the exports, e.g. {bstell, katakai, prabhat, smontagu, rbs, shanjian, etc.} ...
(Assignee)

Comment 3

15 years ago
s/the exports/the experts/
(Assignee)

Comment 4

15 years ago
bstell:
What about mapping "*-iso10646-1" fonts to the "x-unicode" langgroup ?

Comment 5

15 years ago
code issue? QA to yokoyama@netscape.com for now.
Keywords: intl
QA Contact: ruixu → yokoyama

Comment 6

15 years ago
The patch looks good to me. Do you have specific ideas on what else might be
needed?

> What about mapping "*-iso10646-1" fonts to the "x-unicode" langgroup

Unicode doesn't have a language associated with it so it would not helpful.
(Assignee)

Comment 7

15 years ago
Brian Stell wrote:
> The patch looks good to me. Do you have specific ideas on what else might be
> needed?

The question is whether I catched all entries in |gCharSetMap[]|.
At least the following are still mapped to |FLG_NONE|:
1. "cp1251-1" is |FLG_WESTERN|, right ?
2. "microsoft-cp1251" is |FLG_WESTERN| too, right ?
3.
http://lxr.mozilla.org/seamonkey/source/intl/uconv/src/charsetData.properties#56
maps (adobe) sybol fonts to "greek", e.g.
symbol-fontspecific"&&"adobe-fontspecific" would be mapped to |FLG_GREEK| (IMHO
this is completely wrong, but "charsetData.properties" has a different meaning
here - who is right ? :)

> > What about mapping "*-iso10646-1" fonts to the "x-unicode" langgroup
>
> Unicode doesn't have a language associated with it so it would not helpful.

I know ... that's the reason why it is not in the patch, but someone added
"x-unicode" to "charsetData.properties" (see 
http://lxr.mozilla.org/seamonkey/source/intl/uconv/src/charsetData.properties#102
for example). I was wondering if I was missing something here...
(Assignee)

Comment 8

15 years ago
bstell:
Offtopic:
Looking at
http://lxr.mozilla.org/seamonkey/source/gfx/src/gtk/nsFontMetricsGTK.cpp#466 ...
-- snip --
466   { "hpbig5-",    &FLG_ZHTW,    &Big5          },
467   { "hproc16-",   &FLG_NONE,    &Unknown       },
-- snip --
... the '-' chars at the end don't look right for me...
... should I fix that with this patch, too ?

Comment 9

15 years ago
I thought cp1251 was Russian so but you will need to verify this with ftang.

I would not map symbol fonts to Greek.

What would you change the hpbig5- / hproc16- values to?
(Assignee)

Comment 10

15 years ago
bstell wrote:
> I thought cp1251 was Russian so but you will need to verify this with ftang.

ftang ?

> I would not map symbol fonts to Greek.

That's my opinion, too... :)
Any idea why "charsetData.properties" maps them to greek ?

> What would you change the hpbig5- / hproc16- values to?

I would remove the '-' (e.g. s/"hpbig5-"/"hpbig5"/ and s/"hproc16-"/"hproc16"/).
Both entries are the only examples in the list which have such a trailing dash
and I wonder if this was a mistake/typo/etc. ...

Comment 11

15 years ago
> Any idea why "charsetData.properties" maps them to greek ?

Sorry, no. You would need to ask Erik.

> > What would you change the hpbig5- / hproc16- values to?
> 
> I would remove the '-' (e.g. s/"hpbig5-"/"hpbig5"/ and /"hproc16-"/"hproc16"/).
> Both entries are the only examples in the list which have such a trailing dash
> and I wonder if this was a mistake/typo/etc. ...

The strings have 3 parts: registry, separater, and encoding: "registry-encoding"
These appear to have a blank encoding. Its my guess that these do not have a
"encoding" hence it is blank. I recommend not removing the dash.


(Assignee)

Comment 12

15 years ago
I just checked it, cp1251 is cyrillc - new patch follows...
(Assignee)

Comment 13

15 years ago
Created attachment 101966 [details] [diff] [review]
Patch for 2002-09-29-08-trunk
Attachment #101645 - Attachment is obsolete: true
(Assignee)

Comment 14

15 years ago
Requesting r=/sr= ...
Keywords: patch, review

Comment 15

15 years ago
r=bstell@ix.netcom.com
Comment on attachment 101966 [details] [diff] [review]
Patch for 2002-09-29-08-trunk

The code looks fine. Changing to "static" was a good idea. sr=roc+moz
Attachment #101966 - Flags: superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.