freetype-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Devel] Suspected error in cff_parse_font_matrix


From: Blinn, Eric W
Subject: RE: [Devel] Suspected error in cff_parse_font_matrix
Date: Fri, 11 May 2001 09:38:07 -0500

Tom,

It's funny you should mention the likelihood of the same mistake in the T1
driver.  I just upgraded to FT 2.0.2 a few days ago and found the same
problem you had fixed before with the Distiller produced PFAs (roughly
double-sized glyphs).  

In t1load.c function parse_font_matrix() the following should be done...

      root->units_per_EM = (FT_UShort)( FT_DivFix( 0x10000L,
  -                                     FT_DivFix( temp_scale, 1000 ) ) >>
16 );
  +                                     FT_DivFix( temp_scale, 1000 ) ) );

Thanks,
Eric

> -----Original Message-----
> From: Tom Kacvinsky [mailto:address@hidden
> Sent: Friday, May 11, 2001 9:29 AM
> To: address@hidden
> Subject: Re: [Devel] Suspected error in cff_parse_font_matrix
> 
> 
> Sure -- I have been busy here and haven't gotten around to 
> this.  You might want
> to check the code in the Type 1 and CID drivers, too.  I 
> think the same mistake
> might be there...
> 
> On Fri, 11 May 2001, David Turner wrote:
> 
> >
> >
> > Sander van der Wal a écrit :
> > >
> > > Hi Tom
> > >
> > > > I took a closer look at what was going on.  Try the 
> following and
> > > see if it
> > > > fixes your problem:
> > > >
> > > >       *upm = (FT_UShort)( FT_DivFix( 0x10000L,
> > > > -                          FT_DivFix( temp, 1000 ) ) >> 16 );
> > > > +                          FT_DivFix( temp, 1000 ) ) );
> > > >
> > > > Let me know how you fare...
> > >
> > > Hi Tom
> > >
> > > This does indeed fixes the bug.
> > >
> >
> > Tom, you don't seem to have commited this fix, should I do
> > this now ??
> >
> > Regards,
> >
> > - David
> >
> > _______________________________________________
> > Devel mailing list
> > address@hidden
> > http://www.freetype.org/mailman/listinfo/devel
> >
> 
> 
> _______________________________________________
> Devel mailing list
> address@hidden
> http://www.freetype.org/mailman/listinfo/devel
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]