Re: [ft-devel] otvalid modules

2007-08-17 Thread Werner LEMBERG
Applied. However, I think there is a mistake for the SingleSubstFormat1 case: idx = otv_Coverage_get_first( Coverage ) + DeltaGlyphID; if ( idx 0 || idx + DeltaGlyphID 0||

Re: [ft-devel] otvalid modules

2007-08-17 Thread George Williams
On Fri, 2007-08-17 at 08:56, Werner LEMBERG wrote: Applied. However, I think there is a mistake for the SingleSubstFormat1 case: idx = otv_Coverage_get_first( Coverage ) + DeltaGlyphID; if ( idx 0 || idx +

Re: [ft-devel] otvalid modules

2007-08-17 Thread Werner LEMBERG
That entire patch chunk was wrong (and unneeded in the first place). This should back it out. Applied, thanks. Werner ___ Freetype-devel mailing list Freetype-devel@nongnu.org http://lists.nongnu.org/mailman/listinfo/freetype-devel

Re: [ft-devel] otvalid modules

2007-08-11 Thread Werner LEMBERG
Please apply the otvmathconstant patch Applied. The patch coverageCount passes an extra argument to otv_Coverage_validate (the expected number of glyphs, or -1 if this is not known -- it isn't always) and then checks that these counts match. Applied. I may be wrong, and perhaps the

Re: [ft-devel] otvalid modules

2007-08-10 Thread George Williams
I blush to admit that I miscounted the number of constants in one of the tables (there are 56 not 57). Please apply the otvmathconstant patch On Wed, 2007-08-08 at 11:24, Werner LEMBERG wrote: Suzuki-san, what do you think and/or suggest? I have a couple of general questions. Probably for

Re: [ft-devel] otvalid modules

2007-08-08 Thread George Williams
On Tue, 2007-08-07 at 15:35, Behdad Esfahbod wrote: On Tue, 2007-08-07 at 18:06 -0400, George Williams wrote: I am working on a validator for the MATH table to slot it in to otvalid. Hi George, Any spec on the MATH table available online? Thanks No, MS hasn't made it public yet.