Re: Printing

[YAMAMOTO Mitsuharu]

>>>>> On Wed, 08 Apr 2009 10:33:59 +0900, Kenichi Handa <address@hidden> said:

> In article <address@hidden>, YAMAMOTO Mitsuharu <address@hidden> writes:
>> Handa-san, could you check if the following change to ftfont.c is
>> correct?  It is also included in the revised patch below.

> Thank you.  Those changes are correct.  I've just committed them.

I've just found the following part was partially applied.  Could you
check the direction of the inequality in the if-condition?

> *************** ftfont_text_extents (font, code, nglyphs
> *** 1269,1276 ****
>                 = width + ((m->horiBearingX + m->width) >> 6);
>             if (metrics->ascent < (m->horiBearingY >> 6))
>               metrics->ascent = m->horiBearingY >> 6;
> !           if (metrics->descent > ((m->horiBearingY + m->height) >> 6))
> !             metrics->descent = (m->horiBearingY + m->height) >> 6;
>           }
>         width += m->horiAdvance >> 6;
>       }
> --- 1269,1276 ----
>                 = width + ((m->horiBearingX + m->width) >> 6);
>             if (metrics->ascent < (m->horiBearingY >> 6))
>               metrics->ascent = m->horiBearingY >> 6;
> !           if (metrics->descent < ((m->height - m->horiBearingY) >> 6))
> !             metrics->descent = (m->height - m->horiBearingY) >> 6;
>           }
>         width += m->horiAdvance >> 6;
>       }

                                     YAMAMOTO Mitsuharu
                                address@hidden