Re: Printing

[Kenichi Handa]

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.

> *************** ftfont_text_extents (font, code, nglyphs
> *** 1255,1261 ****
>                 metrics->lbearing = m->horiBearingX >> 6;
>                 metrics->rbearing = (m->horiBearingX + m->width) >> 6;
>                 metrics->ascent = m->horiBearingY >> 6;
> !               metrics->descent = (m->horiBearingY + m->height) >> 6;
>               }
>             first = 0;
>           }
> --- 1255,1261 ----
>                 metrics->lbearing = m->horiBearingX >> 6;
>                 metrics->rbearing = (m->horiBearingX + m->width) >> 6;
>                 metrics->ascent = m->horiBearingY >> 6;
> !               metrics->descent = (m->height - m->horiBearingY) >> 6;
>               }
>             first = 0;
>           }
> *************** 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;
>       }

---
Kenichi Handa
address@hidden