Re: [ft-devel] GF + TFM

[Parth Wazurkar]

>> >> Yes, it has been already done.
>>
>> Hmm.  Which commit is this?  I don't see anything recent change in
>> the `parthw-cleanup' branch related to this issue.
>
> Actually this was done from the start itself, I misunderstood
> somethings and got confused myself and then I figured out that it
> was fine.  You can check in the `GF_Glyph_Load' function in
> `gfdrivr.c', in the latest source tree.

No, no, you are mistaken.  In `gfdrivr.c', line 398 (function
`GF_Glyph_Load') there is

  bm = gf->gf_glyph->bm_table[glyph_index];

which means that you don't load the glyph on demand from the font file
stream.  Instead, you have already loaded *all* bitmaps into
`bm_table'[*] while creating the GF face object, and you simply get a
pointer into this array and copy the bitmap into the glyph slot.

This wastes a lot of memory; it also makes `GF_Init_Face'
unnecessarily slow.  I asked you to change this code so that
`GF_Init_Face' only loads file stream *offsets* to an array.  The
actual bitmap (namely a *single* one for a given glyph index, and not
all together) should then be loaded by `GF_Glyph_Load', seeking to the
stored offset and starting the parsing of the GF data.

Oh, I see. I'll change it. Thanks.

BTW, BDF driver too does the same, this might have to be changed too.

Thank you

Parth