<div dir="ltr"><div><div><div>Here's a patch for this issue. <br></div><br>The old code is apparent wrong, since extents->height is always 0<br></div>AFAIK, the order of fontBBox is [x1 y1 x2 y2], and the patch fixes it accordingly.<br>

<br><br>regards,<br></div><div>- Lu Wang<br></div><div><br></div><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 19, 2013 at 4:19 PM, Lu Wang <span dir="ltr"><<a href="mailto:coolwanglu@gmail.com" target="_blank">coolwanglu@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div>Hi all,<br><br><br></div>   While reading CairoFontEngine.cc, I found the following lines around line 600:<br>

<br>   extents->ascent = mat[3]; /* y2 */<br>  extents->descent = -mat[3]; /* -y1 */<br>
  extents->height = extents->ascent + extents->descent;<br>  extents->max_x_advance = mat[2] - mat[1]; /* x2 - x1 */<br>  extents->max_y_advance = 0;<br><br>  <br></div>  The indices of mat[] looks suspicious to me, seems that they are not matching the comments nor the fields of extents.<br>


<br></div>  Can any one please verify this?<br><br></div>  Thanks!<br><br><br></div>  regards,<br></div>  - Lu Wang<br></div>
</blockquote></div><br></div>