July 2007 - Wine-Commits - list.winehq.org

Evan Stade : gdiplus: Added GdipTransformPath.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 1f1ecfb2b51bb0cab790c9c83cd8cbd8fb20d013 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1f1ecfb2b51bb0cab790c9c83… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:08:05 2007 -0700 gdiplus: Added GdipTransformPath. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphicspath.c | 12 ++++++++++++ include/gdiplusflat.h | 1 + 3 files changed, 14 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index 5879360..c1d6725 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -584,7 +584,7 @@ @ stub GdipTestControl @ stdcall GdipTransformMatrixPoints(ptr ptr long) @ stub GdipTransformMatrixPointsI -@ stub GdipTransformPath +@ stdcall GdipTransformPath(ptr ptr) @ stub GdipTransformPoints @ stub GdipTransformPointsI @ stub GdipTransformRegion diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index 394b9f6..4665fca 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -222,3 +222,15 @@ GpStatus WINGDIPAPI GdipStartPathFigure(GpPath *path) return Ok; } + +GpStatus WINGDIPAPI GdipTransformPath(GpPath *path, GpMatrix *matrix) +{ + if(!path) + return InvalidParameter; + + if(path->pathdata.Count == 0) + return Ok; + + return GdipTransformMatrixPoints(matrix, (GpPointF*) path->pathdata.Points, + path->pathdata.Count); +} diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index 9185fd5..95dafde 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -59,6 +59,7 @@ GpStatus WINGDIPAPI GdipGetPathPoints(GpPath*,GpPointF*,INT); GpStatus WINGDIPAPI GdipGetPathTypes(GpPath*,BYTE*,INT); GpStatus WINGDIPAPI GdipGetPointCount(GpPath*,INT*); GpStatus WINGDIPAPI GdipStartPathFigure(GpPath*); +GpStatus WINGDIPAPI GdipTransformPath(GpPath*,GpMatrix*); GpStatus WINGDIPAPI GdipCreateMatrix2(REAL,REAL,REAL,REAL,REAL,REAL,GpMatrix**); GpStatus WINGDIPAPI GdipDeleteMatrix(GpMatrix*);

1 0

Evan Stade : gdiplus: Added GdipCreateMatrix2 test.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 05450b036611a8b7c5b5430bf89389dd397dda41 URL: http://source.winehq.org/git/wine.git/?a=commit;h=05450b036611a8b7c5b5430bf… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:56 2007 -0700 gdiplus: Added GdipCreateMatrix2 test. --- dlls/gdiplus/tests/Makefile.in | 1 + dlls/gdiplus/tests/matrix.c | 60 ++++++++++++++++++++++++++++++++++++++++ 2 files changed, 61 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/tests/Makefile.in b/dlls/gdiplus/tests/Makefile.in index 4d8662a..9eb9e51 100644 --- a/dlls/gdiplus/tests/Makefile.in +++ b/dlls/gdiplus/tests/Makefile.in @@ -8,6 +8,7 @@ IMPORTS = gdiplus kernel32 CTESTS = \ brush.c \ graphicspath.c \ + matrix.c \ pen.c @MAKE_TEST_RULES@ diff --git a/dlls/gdiplus/tests/matrix.c b/dlls/gdiplus/tests/matrix.c new file mode 100644 index 0000000..d6fc0ed --- /dev/null +++ b/dlls/gdiplus/tests/matrix.c @@ -0,0 +1,60 @@ +/* + * Unit test suite for matrices + * + * Copyright (C) 2007 Google (Evan Stade) + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#include "windef.h" +#include "gdiplus.h" +#include "wine/test.h" + +#define expect(expected, got) ok(got == expected, "Expected %.8x, got %.8x\n", expected, got) + +static void test_constructor_destructor(void) +{ + GpStatus status; + GpMatrix *matrix = NULL; + + status = GdipCreateMatrix2(0.0, 0.0, 0.0, 0.0, 0.0, 0.0, &matrix); + expect(Ok, status); + ok(matrix != NULL, "Expected matrix to be initialized\n"); + + status = GdipDeleteMatrix(NULL); + expect(InvalidParameter, status); + + status = GdipDeleteMatrix(matrix); + expect(Ok, status); +} + +START_TEST(matrix) +{ + struct GdiplusStartupInput gdiplusStartupInput; + ULONG_PTR gdiplusToken; + + gdiplusStartupInput.GdiplusVersion = 1; + gdiplusStartupInput.DebugEventCallback = NULL; + gdiplusStartupInput.SuppressBackgroundThread = 0; + gdiplusStartupInput.SuppressExternalCodecs = 0; + + GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL); + + test_constructor_destructor(); + + GdiplusShutdown(gdiplusToken); +}

1 0

Evan Stade : gdiplus: Added GdipTransformMatrixPoints.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 47a605ef9bd2bae60ba74d5f60a2bd1792dfab9e URL: http://source.winehq.org/git/wine.git/?a=commit;h=47a605ef9bd2bae60ba74d5f6… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:52 2007 -0700 gdiplus: Added GdipTransformMatrixPoints. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/matrix.c | 21 +++++++++++++++++++++ include/gdiplusflat.h | 1 + 3 files changed, 23 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index b349e01..5879360 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -582,7 +582,7 @@ @ stub GdipStringFormatGetGenericDefault @ stub GdipStringFormatGetGenericTypographic @ stub GdipTestControl -@ stub GdipTransformMatrixPoints +@ stdcall GdipTransformMatrixPoints(ptr ptr long) @ stub GdipTransformMatrixPointsI @ stub GdipTransformPath @ stub GdipTransformPoints diff --git a/dlls/gdiplus/matrix.c b/dlls/gdiplus/matrix.c index 919e1d4..909bfee 100644 --- a/dlls/gdiplus/matrix.c +++ b/dlls/gdiplus/matrix.c @@ -56,3 +56,24 @@ GpStatus WINGDIPAPI GdipDeleteMatrix(GpMatrix *matrix) return Ok; } + +GpStatus WINGDIPAPI GdipTransformMatrixPoints(GpMatrix *matrix, GpPointF *pts, + INT count) +{ + REAL x, y; + INT i; + + if(!matrix || !pts) + return InvalidParameter; + + for(i = 0; i < count; i++) + { + x = pts[i].X; + y = pts[i].Y; + + pts[i].X = x * matrix->matrix[0] + y * matrix->matrix[2] + matrix->matrix[4]; + pts[i].Y = x * matrix->matrix[1] + y * matrix->matrix[3] + matrix->matrix[5]; + } + + return Ok; +} diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index 5a48fdc..9185fd5 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -62,6 +62,7 @@ GpStatus WINGDIPAPI GdipStartPathFigure(GpPath*); GpStatus WINGDIPAPI GdipCreateMatrix2(REAL,REAL,REAL,REAL,REAL,REAL,GpMatrix**); GpStatus WINGDIPAPI GdipDeleteMatrix(GpMatrix*); +GpStatus WINGDIPAPI GdipTransformMatrixPoints(GpMatrix*,GpPointF*,INT); #ifdef __cplusplus }

1 0

Evan Stade : gdiplus: Added basic matrix implementation.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: f28262b47da30f5d660bdc37244da81d4c3e1240 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f28262b47da30f5d660bdc372… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:48 2007 -0700 gdiplus: Added basic matrix implementation. --- dlls/gdiplus/Makefile.in | 1 + dlls/gdiplus/gdiplus.spec | 4 +- dlls/gdiplus/gdiplus_private.h | 4 +++ dlls/gdiplus/matrix.c | 58 ++++++++++++++++++++++++++++++++++++++++ include/gdiplusflat.h | 3 ++ include/gdiplusgpstubs.h | 2 + 6 files changed, 70 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/Makefile.in b/dlls/gdiplus/Makefile.in index 1194fc0..85c1f27 100644 --- a/dlls/gdiplus/Makefile.in +++ b/dlls/gdiplus/Makefile.in @@ -11,6 +11,7 @@ C_SRCS = \ gdiplus.c \ graphics.c \ graphicspath.c \ + matrix.c \ pen.c @MAKE_DLL_RULES@ diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index a805bd8..b349e01 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -95,7 +95,7 @@ @ stub GdipCreateLineBrushFromRectWithAngle @ stub GdipCreateLineBrushFromRectWithAngleI @ stub GdipCreateLineBrushI -@ stub GdipCreateMatrix2 +@ stdcall GdipCreateMatrix2(long long long long long long ptr) @ stub GdipCreateMatrix3 @ stub GdipCreateMatrix3I @ stub GdipCreateMatrix @@ -133,7 +133,7 @@ @ stub GdipDeleteFont @ stub GdipDeleteFontFamily @ stdcall GdipDeleteGraphics(ptr) -@ stub GdipDeleteMatrix +@ stdcall GdipDeleteMatrix(ptr) @ stdcall GdipDeletePath(ptr) @ stub GdipDeletePathIter @ stdcall GdipDeletePen(ptr) diff --git a/dlls/gdiplus/gdiplus_private.h b/dlls/gdiplus/gdiplus_private.h index e37c20e..7b7ec62 100644 --- a/dlls/gdiplus/gdiplus_private.h +++ b/dlls/gdiplus/gdiplus_private.h @@ -71,4 +71,8 @@ struct GpPath{ INT datalen; /* size of the arrays in pathdata */ }; +struct GpMatrix{ + REAL matrix[6]; +}; + #endif diff --git a/dlls/gdiplus/matrix.c b/dlls/gdiplus/matrix.c new file mode 100644 index 0000000..919e1d4 --- /dev/null +++ b/dlls/gdiplus/matrix.c @@ -0,0 +1,58 @@ +/* + * Copyright (C) 2007 Google (Evan Stade) + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> + +#include "windef.h" +#include "winbase.h" +#include "wingdi.h" + +#include "gdiplus.h" +#include "gdiplus_private.h" + +GpStatus WINGDIPAPI GdipCreateMatrix2(REAL m11, REAL m12, REAL m21, REAL m22, + REAL dx, REAL dy, GpMatrix **matrix) +{ + if(!matrix) + return InvalidParameter; + + *matrix = GdipAlloc(sizeof(GpMatrix)); + if(!*matrix) return OutOfMemory; + + /* first row */ + (*matrix)->matrix[0] = m11; + (*matrix)->matrix[1] = m12; + /* second row */ + (*matrix)->matrix[2] = m21; + (*matrix)->matrix[3] = m22; + /* third row */ + (*matrix)->matrix[4] = dx; + (*matrix)->matrix[5] = dy; + + return Ok; +} + +GpStatus WINGDIPAPI GdipDeleteMatrix(GpMatrix *matrix) +{ + if(!matrix) + return InvalidParameter; + + GdipFree(matrix); + + return Ok; +} diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index b5743eb..5a48fdc 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -60,6 +60,9 @@ GpStatus WINGDIPAPI GdipGetPathTypes(GpPath*,BYTE*,INT); GpStatus WINGDIPAPI GdipGetPointCount(GpPath*,INT*); GpStatus WINGDIPAPI GdipStartPathFigure(GpPath*); +GpStatus WINGDIPAPI GdipCreateMatrix2(REAL,REAL,REAL,REAL,REAL,REAL,GpMatrix**); +GpStatus WINGDIPAPI GdipDeleteMatrix(GpMatrix*); + #ifdef __cplusplus } #endif diff --git a/include/gdiplusgpstubs.h b/include/gdiplusgpstubs.h index 8be4e9c..39bca1f 100644 --- a/include/gdiplusgpstubs.h +++ b/include/gdiplusgpstubs.h @@ -26,6 +26,7 @@ class GpGraphics {}; class GpBrush {}; class GpSolidFill {}; class GpPath {}; +class GpMatrix {}; #else /* end of c++ declarations */ @@ -34,6 +35,7 @@ typedef struct GpPen GpPen; typedef struct GpBrush GpBrush; typedef struct GpSolidFill GpSolidFill; typedef struct GpPath GpPath; +typedef struct GpMatrix GpMatrix; #endif /* end of c declarations */

1 0

Evan Stade : gdiplus: Added draw_polyline error checking.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 6f4ab5282411cde3298fa1c036004f2894350d73 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6f4ab5282411cde3298fa1c03… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:44 2007 -0700 gdiplus: Added draw_polyline error checking. --- dlls/gdiplus/graphics.c | 29 +++++++++++++++++++++++------ 1 files changed, 23 insertions(+), 6 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 2a3ea33..93cbcd1 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -282,12 +282,23 @@ static void shorten_line_amt(REAL x1, REAL y1, REAL *x2, REAL *y2, REAL amt) /* Draws lines between the given points, and if caps is true then draws an endcap * at the end of the last line. FIXME: Startcaps not implemented. */ -static void draw_polyline(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, +static GpStatus draw_polyline(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, INT count, BOOL caps) { - POINT *pti = GdipAlloc(count * sizeof(POINT)); + POINT *pti; REAL x = pt[count - 1].X, y = pt[count - 1].Y; INT i; + GpStatus status = GenericError; + + if(!count) + return Ok; + + pti = GdipAlloc(count * sizeof(POINT)); + + if(!pti){ + status = OutOfMemory; + goto end; + } if(caps){ if(pen->endcap == LineCapArrowAnchor) @@ -306,7 +317,11 @@ static void draw_polyline(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, pti[i].y = roundr(y); Polyline(hdc, pti, count); + +end: GdipFree(pti); + + return status; } /* Conducts a linear search to find the bezier points that will back off @@ -660,6 +675,7 @@ GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics *graphics, GpPen *pen, INT x1, { INT save_state; GpPointF pt[2]; + GpStatus retval; if(!pen || !graphics) return InvalidParameter; @@ -673,17 +689,18 @@ GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics *graphics, GpPen *pen, INT x1, EndPath(graphics->hdc); SelectObject(graphics->hdc, pen->gdipen); - draw_polyline(graphics->hdc, pen, pt, 2, TRUE); + retval = draw_polyline(graphics->hdc, pen, pt, 2, TRUE); RestoreDC(graphics->hdc, save_state); - return Ok; + return retval; } GpStatus WINGDIPAPI GdipDrawLines(GpGraphics *graphics, GpPen *pen, GDIPCONST GpPointF *points, INT count) { INT save_state; + GpStatus retval; if(!pen || !graphics || (count < 2)) return InvalidParameter; @@ -692,11 +709,11 @@ GpStatus WINGDIPAPI GdipDrawLines(GpGraphics *graphics, GpPen *pen, GDIPCONST EndPath(graphics->hdc); SelectObject(graphics->hdc, pen->gdipen); - draw_polyline(graphics->hdc, pen, points, count, TRUE); + retval = draw_polyline(graphics->hdc, pen, points, count, TRUE); RestoreDC(graphics->hdc, save_state); - return Ok; + return retval; } GpStatus WINGDIPAPI GdipDrawPath(GpGraphics *graphics, GpPen *pen, GpPath *path)

1 0

Evan Stade : gdiplus: Added draw_polybezier error checking.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: fa31217d6e831bf7b04f05d42e0c3dd58f496979 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fa31217d6e831bf7b04f05d42… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:39 2007 -0700 gdiplus: Added draw_polybezier error checking. --- dlls/gdiplus/graphics.c | 39 ++++++++++++++++++++++++++++++--------- 1 files changed, 30 insertions(+), 9 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index c16a744..2a3ea33 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -344,12 +344,24 @@ static void shorten_bezier_amt(GpPointF * pt, REAL amt) /* Draws bezier curves between given points, and if caps is true then draws an * endcap at the end of the last line. FIXME: Startcaps not implemented. */ -static void draw_polybezier(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, +static GpStatus draw_polybezier(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, INT count, BOOL caps) { - POINT *pti = GdipAlloc(count * sizeof(POINT)); - GpPointF *ptf = GdipAlloc(4 * sizeof(GpPointF)); + POINT *pti; + GpPointF *ptf; INT i; + GpStatus status = GenericError; + + if(!count) + return Ok; + + pti = GdipAlloc(count * sizeof(POINT)); + ptf = GdipAlloc(4 * sizeof(GpPointF)); + + if(!pti || !ptf){ + status = OutOfMemory; + goto end; + } memcpy(ptf, &pt[count-4], 4 * sizeof(GpPointF)); @@ -371,8 +383,14 @@ static void draw_polybezier(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, } PolyBezier(hdc, pti, count); + + status = Ok; + +end: GdipFree(pti); GdipFree(ptf); + + return status; } /* Converts from gdiplus path point type to gdi path point type. */ @@ -534,6 +552,7 @@ GpStatus WINGDIPAPI GdipDrawArc(GpGraphics *graphics, GpPen *pen, REAL x, { INT save_state, num_pts; GpPointF points[MAX_ARC_PTS]; + GpStatus retval; if(!graphics || !pen) return InvalidParameter; @@ -544,11 +563,11 @@ GpStatus WINGDIPAPI GdipDrawArc(GpGraphics *graphics, GpPen *pen, REAL x, EndPath(graphics->hdc); SelectObject(graphics->hdc, pen->gdipen); - draw_polybezier(graphics->hdc, pen, points, num_pts, TRUE); + retval = draw_polybezier(graphics->hdc, pen, points, num_pts, TRUE); RestoreDC(graphics->hdc, save_state); - return Ok; + return retval; } GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics *graphics, GpPen *pen, REAL x1, @@ -556,6 +575,7 @@ GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics *graphics, GpPen *pen, REAL x1, { INT save_state; GpPointF pt[4]; + GpStatus retval; if(!graphics || !pen) return InvalidParameter; @@ -573,11 +593,11 @@ GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics *graphics, GpPen *pen, REAL x1, EndPath(graphics->hdc); SelectObject(graphics->hdc, pen->gdipen); - draw_polybezier(graphics->hdc, pen, pt, 4, TRUE); + retval = draw_polybezier(graphics->hdc, pen, pt, 4, TRUE); RestoreDC(graphics->hdc, save_state); - return Ok; + return retval; } /* Approximates cardinal spline with Bezier curves. */ @@ -588,6 +608,7 @@ GpStatus WINGDIPAPI GdipDrawCurve2(GpGraphics *graphics, GpPen *pen, INT i, len_pt = count*3-2, save_state; GpPointF *pt; REAL x1, x2, y1, y2; + GpStatus retval; if(!graphics || !pen) return InvalidParameter; @@ -626,12 +647,12 @@ GpStatus WINGDIPAPI GdipDrawCurve2(GpGraphics *graphics, GpPen *pen, EndPath(graphics->hdc); SelectObject(graphics->hdc, pen->gdipen); - draw_polybezier(graphics->hdc, pen, pt, len_pt, TRUE); + retval = draw_polybezier(graphics->hdc, pen, pt, len_pt, TRUE); GdipFree(pt); RestoreDC(graphics->hdc, save_state); - return Ok; + return retval; } GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics *graphics, GpPen *pen, INT x1,

1 0

Evan Stade : gdiplus: Added GdipDrawPath.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 9d5f568183340e53e82fb623fed0d04490f66112 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9d5f568183340e53e82fb623f… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:34 2007 -0700 gdiplus: Added GdipDrawPath. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphics.c | 148 +++++++++++++++++++++++++++++++++++++++++++++ include/gdiplusflat.h | 1 + 3 files changed, 150 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index d18af7f..a805bd8 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -178,7 +178,7 @@ @ stdcall GdipDrawLineI(ptr ptr long long long long) @ stdcall GdipDrawLines(ptr ptr ptr long) @ stub GdipDrawLinesI -@ stub GdipDrawPath +@ stdcall GdipDrawPath(ptr ptr ptr) @ stdcall GdipDrawPie(ptr ptr long long long long long long) @ stub GdipDrawPieI @ stub GdipDrawPolygon diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 82260ec..c16a744 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -375,6 +375,120 @@ static void draw_polybezier(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, GdipFree(ptf); } +/* Converts from gdiplus path point type to gdi path point type. */ +static BYTE convert_path_point_type(BYTE type) +{ + BYTE ret; + + switch(type & PathPointTypePathTypeMask){ + case PathPointTypeBezier: + ret = PT_BEZIERTO; + break; + case PathPointTypeLine: + ret = PT_LINETO; + break; + case PathPointTypeStart: + ret = PT_MOVETO; + break; + default: + ERR("Bad point type\n"); + return 0; + } + + if(type & PathPointTypeCloseSubpath) + ret |= PT_CLOSEFIGURE; + + return ret; +} + +/* Draws a combination of bezier curves and lines between points. */ +static GpStatus draw_poly(HDC hdc, GpPen *pen, GDIPCONST GpPointF * pt, + GDIPCONST BYTE * types, INT count, BOOL caps) +{ + POINT *pti = GdipAlloc(count * sizeof(POINT)); + BYTE *tp = GdipAlloc(count); + GpPointF *ptf = NULL; + REAL x = pt[count - 1].X, y = pt[count - 1].Y; + INT i; + GpStatus status = GenericError; + + if(!count){ + status = Ok; + goto end; + } + if(!pti || !tp){ + status = OutOfMemory; + goto end; + } + + for(i = 0; i < count; i++){ + if((types[i] & PathPointTypePathTypeMask) == PathPointTypeBezier){ + if((i + 2 >= count) || !(types[i + 1] & PathPointTypeBezier) + || !(types[i + 1] & PathPointTypeBezier)){ + ERR("Bad bezier points\n"); + goto end; + } + + i += 2; + } + } + + if((types[count - 1] & PathPointTypePathTypeMask) == PathPointTypeBezier){ + ptf = GdipAlloc(4 * sizeof(GpPointF)); + memcpy(ptf, &pt[count-4], 4 * sizeof(GpPointF)); + + if(caps){ + if(pen->endcap == LineCapArrowAnchor) + shorten_bezier_amt(ptf, pen->width); + + draw_cap(hdc, pen->color, pen->endcap, pen->width, ptf[3].X, + ptf[3].Y, pt[count - 1].X, pt[count - 1].Y); + } + for(i = 0; i < 4; i ++){ + pti[i + count - 4].x = roundr(ptf[i].X); + pti[i + count - 4].y = roundr(ptf[i].Y); + } + for(i = 0; i < count - 4; i ++){ + pti[i].x = roundr(pt[i].X); + pti[i].y = roundr(pt[i].Y); + } + } + else if((types[count - 1] & PathPointTypePathTypeMask) == PathPointTypeLine){ + if(caps){ + if(pen->endcap == LineCapArrowAnchor) + shorten_line_amt(pt[count-2].X, pt[count-2].Y, &x, &y, pen->width); + + draw_cap(hdc, pen->color, pen->endcap, pen->width, pt[count-2].X, + pt[count-2].Y, pt[count - 1].X, pt[count - 1].Y); + } + pti[count - 1].x = roundr(x); + pti[count - 1].y = roundr(y); + for(i = 0; i < count - 1; i ++){ + pti[i].x = roundr(pt[i].X); + pti[i].y = roundr(pt[i].Y); + } + } + else{ + ERR("Bad path last point\n"); + goto end; + } + + for(i = 0; i < count; i++){ + tp[i] = convert_path_point_type(types[i]); + } + + PolyDraw(hdc, pti, tp, count); + + status = Ok; + +end: + GdipFree(pti); + GdipFree(ptf); + GdipFree(tp); + + return status; +} + GpStatus WINGDIPAPI GdipCreateFromHDC(HDC hdc, GpGraphics **graphics) { if(hdc == NULL) @@ -564,6 +678,40 @@ GpStatus WINGDIPAPI GdipDrawLines(GpGraphics *graphics, GpPen *pen, GDIPCONST return Ok; } +GpStatus WINGDIPAPI GdipDrawPath(GpGraphics *graphics, GpPen *pen, GpPath *path) +{ + INT save_state, i, this_fig = 0; + GpStatus retval; + + if(!pen || !graphics) + return InvalidParameter; + + save_state = SaveDC(graphics->hdc); + EndPath(graphics->hdc); + SelectObject(graphics->hdc, pen->gdipen); + + for(i = 0; i < path->pathdata.Count; i++){ + if(path->pathdata.Types[i] == PathPointTypeStart){ + retval = draw_poly(graphics->hdc, pen, + &path->pathdata.Points[this_fig], + &path->pathdata.Types[this_fig], i - this_fig, TRUE); + this_fig = i; + + if(retval != Ok) + goto end; + } + } + + retval = draw_poly(graphics->hdc, pen, &path->pathdata.Points[this_fig], + &path->pathdata.Types[this_fig], path->pathdata.Count - this_fig, + TRUE); + +end: + RestoreDC(graphics->hdc, save_state); + + return retval; +} + GpStatus WINGDIPAPI GdipDrawPie(GpGraphics *graphics, GpPen *pen, REAL x, REAL y, REAL width, REAL height, REAL startAngle, REAL sweepAngle) { diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index 04d71a9..b5743eb 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -40,6 +40,7 @@ GpStatus WINGDIPAPI GdipDrawBezier(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL, GpStatus WINGDIPAPI GdipDrawCurve2(GpGraphics*,GpPen*,GDIPCONST GpPointF*,INT,REAL); GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics*,GpPen*,INT,INT,INT,INT); GpStatus WINGDIPAPI GdipDrawLines(GpGraphics*,GpPen*,GDIPCONST GpPointF*,INT); +GpStatus WINGDIPAPI GdipDrawPath(GpGraphics*,GpPen*,GpPath*); GpStatus WINGDIPAPI GdipDrawPie(GpGraphics*,GpPen*,REAL,REAL,REAL,REAL,REAL,REAL); GpStatus WINGDIPAPI GdipDrawRectangleI(GpGraphics*,GpPen*,INT,INT,INT,INT); GpStatus WINGDIPAPI GdipFillPie(GpGraphics*,GpBrush*,REAL,REAL,REAL,REAL,REAL,REAL);

1 0

Evan Stade : gdiplus: Added GdipStartPathFigure.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 8a76d1e82c6c39ca946f42d01b6875cceb737002 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8a76d1e82c6c39ca946f42d01… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:28 2007 -0700 gdiplus: Added GdipStartPathFigure. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphicspath.c | 10 ++++++++++ include/gdiplusflat.h | 1 + 3 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index 22b085e..d18af7f 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -578,7 +578,7 @@ @ stub GdipSetTextureWrapMode @ stub GdipSetWorldTransform @ stub GdipShearMatrix -@ stub GdipStartPathFigure +@ stdcall GdipStartPathFigure(ptr) @ stub GdipStringFormatGetGenericDefault @ stub GdipStringFormatGetGenericTypographic @ stub GdipTestControl diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index e65dabd..394b9f6 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -212,3 +212,13 @@ GpStatus WINGDIPAPI GdipGetPointCount(GpPath *path, INT *count) return Ok; } + +GpStatus WINGDIPAPI GdipStartPathFigure(GpPath *path) +{ + if(!path) + return InvalidParameter; + + path->newfigure = TRUE; + + return Ok; +} diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index e49ecc3..04d71a9 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -57,6 +57,7 @@ GpStatus WINGDIPAPI GdipDeletePath(GpPath*); GpStatus WINGDIPAPI GdipGetPathPoints(GpPath*,GpPointF*,INT); GpStatus WINGDIPAPI GdipGetPathTypes(GpPath*,BYTE*,INT); GpStatus WINGDIPAPI GdipGetPointCount(GpPath*,INT*); +GpStatus WINGDIPAPI GdipStartPathFigure(GpPath*); #ifdef __cplusplus }

1 0

Evan Stade : gdiplus: Updated GdipDrawLines to use SaveDC()/RestoreDC()/ end caps.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 852aac8b54b12812187d5c4dc558b5fb52cbcc07 URL: http://source.winehq.org/git/wine.git/?a=commit;h=852aac8b54b12812187d5c4dc… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:16 2007 -0700 gdiplus: Updated GdipDrawLines to use SaveDC()/RestoreDC()/end caps. --- dlls/gdiplus/graphics.c | 14 ++++++-------- 1 files changed, 6 insertions(+), 8 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 6e3bf1d..82260ec 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -548,20 +548,18 @@ GpStatus WINGDIPAPI GdipDrawLineI(GpGraphics *graphics, GpPen *pen, INT x1, GpStatus WINGDIPAPI GdipDrawLines(GpGraphics *graphics, GpPen *pen, GDIPCONST GpPointF *points, INT count) { - HGDIOBJ old_obj; - INT i; + INT save_state; if(!pen || !graphics || (count < 2)) return InvalidParameter; - old_obj = SelectObject(graphics->hdc, pen->gdipen); - MoveToEx(graphics->hdc, roundr(points[0].X), roundr(points[0].Y), NULL); + save_state = SaveDC(graphics->hdc); + EndPath(graphics->hdc); + SelectObject(graphics->hdc, pen->gdipen); - for(i = 1; i < count; i++){ - LineTo(graphics->hdc, roundr(points[i].X), roundr(points[i].Y)); - } + draw_polyline(graphics->hdc, pen, points, count, TRUE); - SelectObject(graphics->hdc, old_obj); + RestoreDC(graphics->hdc, save_state); return Ok; }

1 0

Evan Stade : gdiplus: Updated GdipDrawArc to use SaveDC()/RestoreDC()/ line caps.
by Alexandre Julliard July 12, 2007

July 12, 2007

Module: wine Branch: master Commit: 40f2273f0354ce403684623449379a0da388e98a URL: http://source.winehq.org/git/wine.git/?a=commit;h=40f2273f0354ce40368462344… Author: Evan Stade <estade(a)gmail.com> Date: Wed Jul 11 18:07:09 2007 -0700 gdiplus: Updated GdipDrawArc to use SaveDC()/RestoreDC()/line caps. --- dlls/gdiplus/graphics.c | 19 ++++++++----------- 1 files changed, 8 insertions(+), 11 deletions(-) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index c32a412..6e3bf1d 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -418,24 +418,21 @@ GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *graphics) GpStatus WINGDIPAPI GdipDrawArc(GpGraphics *graphics, GpPen *pen, REAL x, REAL y, REAL width, REAL height, REAL startAngle, REAL sweepAngle) { - HGDIOBJ old_pen; - REAL x_0, y_0, x_1, y_1, x_2, y_2; + INT save_state, num_pts; + GpPointF points[MAX_ARC_PTS]; if(!graphics || !pen) return InvalidParameter; - old_pen = SelectObject(graphics->hdc, pen->gdipen); - - x_0 = x + (width/2.0); - y_0 = y + (height/2.0); + num_pts = arc2polybezier(points, x, y, width, height, startAngle, sweepAngle); - deg2xy(startAngle+sweepAngle, x_0, y_0, &x_1, &y_1); - deg2xy(startAngle, x_0, y_0, &x_2, &y_2); + save_state = SaveDC(graphics->hdc); + EndPath(graphics->hdc); + SelectObject(graphics->hdc, pen->gdipen); - Arc(graphics->hdc, roundr(x), roundr(y), roundr(x+width), roundr(y+height), - roundr(x_1), roundr(y_1), roundr(x_2), roundr(y_2)); + draw_polybezier(graphics->hdc, pen, points, num_pts, TRUE); - SelectObject(graphics->hdc, old_pen); + RestoreDC(graphics->hdc, save_state); return Ok; }

1 0