Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category User Interface → Font/charset
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Release 3.4
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by mitk - 2010-03-07
Last edited by fml2 - 2010-03-14

FS#11082 - Polish character set for 08-Atadore font

Polish character set for 08-Atadore font.

Closed by  fml2
2010-03-14 19:46
Reason for closing:  Accepted
Additional comments about closing:  

Committed in r25179 and r25180. Thanks!

fml2 commented on 2010-03-10 22:27

Tomasz, without having tested the patch, I’d like to note that IMO it would be better when it contained only the parts than have really been changed. IIUC you added 16 new glyphs. To find them and leave only them in the patch (and the corrected comments of course) is really not hard. That way we would better see what has been changed. BDF is a really simple text based file format.

Thank you for you work!

mitk commented on 2010-03-11 15:00

Alexander,

Interesting link: http://www.adobe.com/devnet/opentype/archives/aglfn.txt

Please take a look at glyph 00C1;Aacute;LATIN CAPITAL LETTER A WITH ACUTE
Aacute is Glyph Name
LATIN CAPITAL LETTER A WITH ACUTE is Unicode Glyph Name

In my patch it is:

-STARTCHAR capital A, acute accent
+STARTCHAR Aacute
ENCODING 193

from fonts directory:
[88] tomekk@tyfus:~/workspace/rockbox/fonts$ ls -Al *.bdf | wc -l
73
[89] tomekk@tyfus:~/workspace/rockbox/fonts$ grep -c Aacute *.bdf | grep “:0”| wc -l
16
[90] tomekk@tyfus:~/workspace/rockbox/fonts$ grep -c Aacute *.bdf | grep “:1”| wc -l
57

We have 73 fonts in rockbox
16 uses something similiar to Unicode Glyph Name
57 uses Glyph Names exactly

FIXME:
1. It seems to be that standard for Rockbox fonts is Glyph Name not Unicode Glyph Name
2. STARTCHAR field in glyph record is not used by rockbox at all. It’s needed for PostScript mapping or something (don’t know exactly).
3. Leaving only new glyphs in my patch is the right way to mix Glyph Names with Unicode Glyph Names in one font file.
4. FontForge during export font to bdf uses Glyph Names, so every subsequent add to mixed font file will need more and more manual work on patch file

What I should do with this patch finally?
Please take into account that knowledge of fonts is 1 hour with FontForge and 2 hours with google
so please fixme if I’m wrong in any point.

fml2 commented on 2010-03-11 17:36

What I mean is that each patch should be as atomic as possible, i.e. do only one thing. Your patch does two at the same time: adding new glyphs and changing the glyph names (which of course don’t matter at all). Hence I propose that we split this into two patches for better traceability of changes.

1. Adding new glyphs (cleaned patch attached, but I haven’t tested it, just edited your patch; please test whether it applies cleanly)
2. Changing glyph names. This should be done by re-exporting the BDF font from FontForge after applying the first patch (I can’t do it now)

I’m not against renaming glyphs at all; I’d just do one thing at a time.

Some more notes:

1. STARTCHAR records are of course used, just not by Rockbox itself, but rather by the program that converts BDF to the Rockbox font format (.fnt). How else woult it detect the beginning of a glyph?
2. You also changed the SIZE record. I think it’s not used by Rockbox (and the generating program) but still I wouldn’to do it. I think it was automatically changed by FontForge because some other property is set to 9 instead of 8.
3. Rockbox fonts don’t follow any standards in naming glyphs. It’s more or less by chance that one or the other name is used! :-)

I’ll be able to test and commit the patch, but not before Sunday evening GMT. So if you need it before, please ask in the IRC.

mitk commented on 2010-03-11 20:08

Tested your patch. No problem with it, it applies cleanly and working good on sim.
I attached tested glyph rename patch which needs to be applied after yours 08-Atadore-pl-charset-patch.2.txt. This patch is from diff, not from svn.

Notes:
Ad.1 I agree, STARTCHAR label itself is used by font converter, but not it content I think.
Ad.2 Not me. FontForge changed it because in original bdf PIXEL_SIZE is 9. With pixel size set to 8 FontForge doesn’t export font to bdf.
Ad.3 Sure, but I think that it’s better to keep one standard in one file even when standard changes between files. It should avoid confusions like that in the future.

Thanks for your help

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing