Re: Font Problems

[Owen Taylor <otaylor redhat com>]

Looks good to commit, except that:

>         cur_offset = FILE_Pos();
> -      if ( FILE_Seek( new_offset ) ||
> -           ( error = Load_Script( &sr[n].Script, stream ) ) != TT_Err_Ok )
> -        goto Fail;
> +
> +      if ( FILE_Seek( new_offset ) )
> +       goto Fail;
> +
> +      error = Load_Script( &sr[sl->ScriptCount].Script, stream );
> +      if ( error != TT_Err_Ok &&  error != TTO_Err_Empty_Script )
> +       goto Fail;
> +
> +      sl->ScriptCount += 1;
> +
>         (void)FILE_Seek( cur_offset );
>       }

Don't you only want to increment sl->ScriptCount if 
error != TT_Err_Empty_Script?

Thanks,
                                        Owen