Re: [Fontforge-devel] Invalid 2nd order spline

[Andrey V. P. <pa...@ca...>]

Your patch does not fix the bug.  (I also checked cvs tree.) The
attachement
contains the result of flip transformation of the "C" glyph.

On Thu, Aug 30, 2007 at 01:54:56PM -0700, George Williams wrote:
> On Wed, 2007-08-29 at 21:40, Andrey V. Panov wrote:
> > Open the glyph "C" from the attachment. Select all its points with
> > Ctrl+A,
> > then move them with, e.g., arrow right key. Fontforge will produce error
> > messages:
> > 
> > Internal Error:
> > Invalid 2nd order spline in SplineRefigure2

> --- splineutil.c~	2007-08-30 13:49:34.000000000 -0700
> +++ splineutil.c	2007-08-30 13:49:34.000000000 -0700
> @@ -1497,7 +1497,7 @@
>  	    if ( allpoints || spt->selected ) {
>  		TransformPoint(spt,transform);
>  		if ( !allpoints ) {
> -		    if ( spt->next!=NULL && spt->next->order2 && spt->next->to->ttfindex==0xffff ) {
> +		    if ( spt->next!=NULL && spt->next->order2 && !spt->next->to->selected && spt->next->to->ttfindex==0xffff ) {
>  			SplinePoint *to = spt->next->to;
>  			to->prevcp = spt->nextcp;
>  			to->me.x = (to->prevcp.x+to->nextcp.x)/2;

> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems?  Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >>  http://get.splunk.com/
> _______________________________________________
> Fontforge-devel mailing list
> Fon...@li...
> https://lists.sourceforge.net/lists/listinfo/fontforge-devel


-- 
Andrey V. Panov
panov /AT/ iacp.dvo.ru
http://canopus.iacp.dvo.ru/~panov/