Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread George Williams
On Fri, 2007-10-19 at 04:04, David Turner wrote: By the way, where can I find the spec for cmap format 14 and the variant selector things ?, I could not find anything on the Internet.(yes, I've seen the description in src/sfnt/ttcmap.c) This was posted on the opentype list:

Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread Werner LEMBERG
Another point is that the arrays returned are 0-terminated. this assumes that 0 is not a valid selector variant, but I don't see code to check for this in the cmap 14_validate function. will add these George, what do you think of undefining TT_CONFIG_CMAP_FORMAT_14 until the specs are

Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread David Turner
Another point is that the arrays returned are 0-terminated. this assumes that 0 is not a valid selector variant, but I don't see code to check for this in the cmap 14_validate function. will add these 2007/10/19, David Turner [EMAIL PROTECTED]: ok, I have several issues with this patch (and I'm

Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread David Turner
ok, I have several issues with this patch (and I'm writing a fix right now): -naming: I prefer all new APIs to use the FT_Object_ActionName template, this means I'd favor renaming the functions to things like: FT_Face_GetCharsOfVariant FT_Face_GetVariantsOfChar FT_Face_GetVariantSelectors

Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread Werner LEMBERG
ok, I have several issues with this patch I've expected this, but you have been silent :-) (and I'm writing a fix right now): Great. -naming: I prefer all new APIs to use the FT_Object_ActionName template, OK. - documentation: the API documentation doesn't make the purpose of these

Re: [ft-devel] About the Cmap format 14

2007-10-19 Thread David Turner
ok, this is done. the functions still return arrays, but these are owned by the FT_Face but may be reused/deallocated on the next FreeType call. I've also renamed the functions, and updated their documentation. I'm still interested by the CMAP 14 spec by the way :-) - David