>>>> Please keep FT_Get_Glyph light.
>>> 
>>> Why?
>>
>> Because it is meant to be used once followed by as many FT_Glyph_Copy 
>> as you want.
>
> OK.  Armin, can you change your usage pattern to something like the
following?
>
>  FT_Load_Glyph( face, 0, 0 );
>  FT_Get_Glyph( face->glyph, &reference );
>
>  FT_Glyph_Copy( reference, &glyph1 );
>  ...
>  FT_Done_Glyph( glyph1 );
>
>  FT_Glyph_Copy( reference, &glyph2 );
>  ...
>  FT_Done_Glyph( glyph2 );
>
>  FT_Done_Glyph( reference );
>
> Together with an update of the documentation this doesn't look too bad
IMHO.

Mhm;  I mean I get it -- changing to deep copy here would also introduce
semantic changes which is never anything you really want in an API.

Just another proposal (feel free to decline!):  how about coming back to (3)
and introduce something like `FT_Get_Glyph_Deep'?  Internally, the same
logic could be used;  I would simply add a switch (bool var) that gets
propagated down to `ft_bitmap_glyph_init' to skip the lazy copy?  I could
even prepare the patch if you like, no work for anyone ;)


_______________________________________________
Freetype-devel mailing list
Freetype-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/freetype-devel

Reply via email to