On 1/12/2014 11:51 PM, Khaled Hosny wrote:
On Sat, Jan 11, 2014 at 04:30:19PM +0100, Cron Daemon wrote:
------------------------------------------------------------------------ r4742 | luigi | 2014-01-11 12:26:06 +0100 (Sat, 11 Jan 2014) | 1 line Changed paths: M /trunk/source/texk/web2c/luatexdir/luafontloader/fontforge/fontforge/parsettfatt.c
unwanted side effects: font loading takes way longer due to linear checks and it makes no sense to do that for a bugged font; this can better be doen at the lua end if needed at all (hans)
Then you are better dropping glyph name guessing code completely because it is known to produce duplicate glyph names, having generic "glyphXXXX" names is better than having more than one glyph with the same name (the fonts are not bugged, but the original was).
we'll have a look at it (using some bianry search instead); the old patch was quitting and not assigning a generic indexXXXX name either which was also wrong i think; in that case a_b_c_1 (first available subname) is better Hans ----------------------------------------------------------------- Hans Hagen | PRAGMA ADE Ridderstraat 27 | 8061 GH Hasselt | The Netherlands tel: 038 477 53 69 | voip: 087 875 68 74 | www.pragma-ade.com | www.pragma-pod.nl -----------------------------------------------------------------