Vincent Povirk : gdiplus: Don' t position characters by glyph metrics in GdipAddPathString.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 3cde67e1444e794d220f6344f04774ff88f5e23f
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=3cde67e1444e794d220f6344f0...

Author: Vincent Povirk 
Date:   Thu Jun 21 15:49:39 2012 -0500

gdiplus: Don't position characters by glyph metrics in GdipAddPathString.

---

dlls/gdiplus/graphicspath.c |   15 ++++++++-------
 1 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c
index 3095c08..df9613d 100644
--- a/dlls/gdiplus/graphicspath.c
+++ b/dlls/gdiplus/graphicspath.c
@@ -842,6 +842,7 @@ struct format_string_args
     GpPath *path;
     float maxY;
     float scale;
+    float ascent;
 };

static GpStatus format_string_callback(HDC dc,
@@ -861,13 +862,15 @@ static GpStatus format_string_callback(HDC dc,
     if (underlined_index_count)
         FIXME("hotkey underlines not drawn yet\n");

+    if (y + bounds->Height * args->scale > args->maxY)
+        args->maxY = y + bounds->Height * args->scale;
+
     for (i = index; i < length; ++i)
     {
         GLYPHMETRICS gm;
         TTPOLYGONHEADER *ph = NULL;
         char *start;
         DWORD len, ofs = 0;
-        float bb_end;
         len = GetGlyphOutlineW(dc, string[i], GGO_BEZIER, &gm, 0, NULL, &identity);
         if (len == GDI_ERROR)
         {
@@ -882,9 +885,6 @@ static GpStatus format_string_callback(HDC dc,
             break;
         }
         GetGlyphOutlineW(dc, string[i], GGO_BEZIER, &gm, len, start, &identity);
-        bb_end = (gm.gmBlackBoxY + gm.gmptGlyphOrigin.y) * args->scale;
-        if (bb_end + y > args->maxY)
-            args->maxY = bb_end + y;

ofs = 0;
         while (ofs < len)
@@ -893,7 +893,7 @@ static GpStatus format_string_callback(HDC dc,
             ph = (TTPOLYGONHEADER*)&start[ofs];
             path->pathdata.Types[path->pathdata.Count] = PathPointTypeStart;
             path->pathdata.Points[path->pathdata.Count].X = x + fromfixedpoint(ph->pfxStart.x) * args->scale;
-            path->pathdata.Points[path->pathdata.Count++].Y = y + bb_end - fromfixedpoint(ph->pfxStart.y) * args->scale;
+            path->pathdata.Points[path->pathdata.Count++].Y = y + args->ascent - fromfixedpoint(ph->pfxStart.y) * args->scale;
             TRACE("Starting at count %i with pos %f, %f)\n", path->pathdata.Count, x, y);
             ofs += sizeof(*ph);
             while (ofs - ofs_start < ph->cb)
@@ -909,7 +909,7 @@ static GpStatus format_string_callback(HDC dc,
                     {
                         path->pathdata.Types[path->pathdata.Count] = PathPointTypeLine;
                         path->pathdata.Points[path->pathdata.Count].X = x + fromfixedpoint(curve->apfx[j].x) * args->scale;
-                        path->pathdata.Points[path->pathdata.Count++].Y = y + bb_end - fromfixedpoint(curve->apfx[j].y) * args->scale;
+                        path->pathdata.Points[path->pathdata.Count++].Y = y + args->ascent - fromfixedpoint(curve->apfx[j].y) * args->scale;
                     }
                     break;
                 case TT_PRIM_CSPLINE:
@@ -917,7 +917,7 @@ static GpStatus format_string_callback(HDC dc,
                     {
                         path->pathdata.Types[path->pathdata.Count] = PathPointTypeBezier;
                         path->pathdata.Points[path->pathdata.Count].X = x + fromfixedpoint(curve->apfx[j].x) * args->scale;
-                        path->pathdata.Points[path->pathdata.Count++].Y = y + bb_end - fromfixedpoint(curve->apfx[j].y) * args->scale;
+                        path->pathdata.Points[path->pathdata.Count++].Y = y + args->ascent - fromfixedpoint(curve->apfx[j].y) * args->scale;
                     }
                     break;
                 default:
@@ -992,6 +992,7 @@ GpStatus WINGDIPAPI GdipAddPathString(GpPath* path, GDIPCONST WCHAR* string, INT
     args.path = path;
     args.maxY = 0;
     args.scale = emSize / native_height;
+    args.ascent = textmetric.tmAscent * args.scale;
     status = gdip_format_string(dc, string, length, NULL, &scaled_layout_rect, format, format_string_callback, &args);

DeleteDC(dc);