In article <wl1vs4iyp2.wl%mituharu@math...>, YAMAMOTO Mitsuharu <mituharu@math...> 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
handa@m17n...