July 2016 - Wine-commits - list.winehq.org

Huw Davies : gdi32: Use a buffer on the stack if the number of points is small.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 22202eae59f95bc370c37da8bb385e019f712a6f URL: http://source.winehq.org/git/wine.git/?a=commit;h=22202eae59f95bc370c37da8b… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 13:13:05 2016 +0100 gdi32: Use a buffer on the stack if the number of points is small. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/dibdrv/graphics.c | 35 +++++++++++++++++++++++------------ 1 file changed, 23 insertions(+), 12 deletions(-) diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 08e777d..3ea39a1 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -1239,7 +1239,8 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD DC *dc = get_physdev_dc( dev ); DWORD total, i, pos; BOOL ret = TRUE; - POINT *points; + POINT pt_buf[32]; + POINT *points = pt_buf; HRGN outline = 0, interior = 0; for (i = total = 0; i < polygons; i++) @@ -1248,16 +1249,19 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD total += counts[i]; } - points = HeapAlloc( GetProcessHeap(), 0, total * sizeof(*pt) ); - if (!points) return FALSE; + if (total > sizeof(pt_buf) / sizeof(pt_buf[0])) + { + points = HeapAlloc( GetProcessHeap(), 0, total * sizeof(*pt) ); + if (!points) return FALSE; + } memcpy( points, pt, total * sizeof(*pt) ); lp_to_dp( dc, points, total ); if (pdev->brush.style != BS_NULL && !(interior = CreatePolyPolygonRgn( points, counts, polygons, dc->polyFillMode ))) { - HeapFree( GetProcessHeap(), 0, points ); - return FALSE; + ret = FALSE; + goto done; } if (pdev->pen_uses_region) outline = CreateRectRgn( 0, 0, 0, 0 ); @@ -1289,7 +1293,9 @@ BOOL dibdrv_PolyPolygon( PHYSDEV dev, const POINT *pt, const INT *counts, DWORD if (ret) ret = pen_region( pdev, outline ); DeleteObject( outline ); } - HeapFree( GetProcessHeap(), 0, points ); + +done: + if (points != pt_buf) HeapFree( GetProcessHeap(), 0, points ); return ret; } @@ -1301,7 +1307,8 @@ BOOL dibdrv_PolyPolyline( PHYSDEV dev, const POINT* pt, const DWORD* counts, DWO dibdrv_physdev *pdev = get_dibdrv_pdev(dev); DC *dc = get_physdev_dc( dev ); DWORD total, pos, i; - POINT *points; + POINT pt_buf[32]; + POINT *points = pt_buf; BOOL ret = TRUE; HRGN outline = 0; @@ -1311,15 +1318,18 @@ BOOL dibdrv_PolyPolyline( PHYSDEV dev, const POINT* pt, const DWORD* counts, DWO total += counts[i]; } - points = HeapAlloc( GetProcessHeap(), 0, total * sizeof(*pt) ); - if (!points) return FALSE; + if (total > sizeof(pt_buf) / sizeof(pt_buf[0])) + { + points = HeapAlloc( GetProcessHeap(), 0, total * sizeof(*pt) ); + if (!points) return FALSE; + } memcpy( points, pt, total * sizeof(*pt) ); lp_to_dp( dc, points, total ); if (pdev->pen_uses_region && !(outline = CreateRectRgn( 0, 0, 0, 0 ))) { - HeapFree( GetProcessHeap(), 0, points ); - return FALSE; + ret = FALSE; + goto done; } for (i = pos = 0; i < polylines; i++) @@ -1336,7 +1346,8 @@ BOOL dibdrv_PolyPolyline( PHYSDEV dev, const POINT* pt, const DWORD* counts, DWO DeleteObject( outline ); } - HeapFree( GetProcessHeap(), 0, points ); +done: + if (points != pt_buf) HeapFree( GetProcessHeap(), 0, points ); return ret; }

1 0

Huw Davies : gdi32: Add an internal version of GetTextCharsetInfo() that takes a DC pointer.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 41f763ea879045464b02d45c048852e7e13cea93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=41f763ea879045464b02d45c0… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 13:13:06 2016 +0100 gdi32: Add an internal version of GetTextCharsetInfo() that takes a DC pointer. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/font.c | 57 ++++++++++++++++++++++++++++++++++--------------------- 1 file changed, 35 insertions(+), 22 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index b42aaa6..2faefe8 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -407,6 +407,40 @@ DWORD WINAPI GdiGetCodePage( HDC hdc ) } /*********************************************************************** + * get_text_charset_info + * + * Internal version of GetTextCharsetInfo() that takes a DC pointer. + */ +static UINT get_text_charset_info(DC *dc, FONTSIGNATURE *fs, DWORD flags) +{ + UINT ret = DEFAULT_CHARSET; + PHYSDEV dev; + + dev = GET_DC_PHYSDEV( dc, pGetTextCharsetInfo ); + ret = dev->funcs->pGetTextCharsetInfo( dev, fs, flags ); + + if (ret == DEFAULT_CHARSET && fs) + memset(fs, 0, sizeof(FONTSIGNATURE)); + return ret; +} + +/*********************************************************************** + * GetTextCharsetInfo (GDI32.@) + */ +UINT WINAPI GetTextCharsetInfo(HDC hdc, FONTSIGNATURE *fs, DWORD flags) +{ + UINT ret = DEFAULT_CHARSET; + DC *dc = get_dc_ptr(hdc); + + if (dc) + { + ret = get_text_charset_info( dc, fs, flags ); + release_dc_ptr( dc ); + } + return ret; +} + +/*********************************************************************** * FONT_mbtowc * * Returns a Unicode translation of str using the charset of the @@ -646,7 +680,7 @@ static DWORD get_associated_charset_info(void) static void update_font_code_page( DC *dc, HANDLE font ) { CHARSETINFO csi; - int charset = GetTextCharsetInfo( dc->hSelf, NULL, 0 ); + int charset = get_text_charset_info( dc, NULL, 0 ); LOGFONTW lf; GetObjectW( font, sizeof(lf), &lf ); @@ -3689,27 +3723,6 @@ UINT WINAPI GetTextCharset(HDC hdc) } /*********************************************************************** - * GetTextCharsetInfo (GDI32.@) - */ -UINT WINAPI GetTextCharsetInfo(HDC hdc, LPFONTSIGNATURE fs, DWORD flags) -{ - UINT ret = DEFAULT_CHARSET; - DC *dc = get_dc_ptr(hdc); - PHYSDEV dev; - - if (dc) - { - dev = GET_DC_PHYSDEV( dc, pGetTextCharsetInfo ); - ret = dev->funcs->pGetTextCharsetInfo( dev, fs, flags ); - release_dc_ptr( dc ); - } - - if (ret == DEFAULT_CHARSET && fs) - memset(fs, 0, sizeof(FONTSIGNATURE)); - return ret; -} - -/*********************************************************************** * GdiGetCharDimensions (GDI32.@) * * Gets the average width of the characters in the English alphabet.

1 0

Huw Davies : gdi32: Don't call GetObjectW() unless necessary.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 0c1e42235aa8c6e2df939e8d7d673d16469d78d4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0c1e42235aa8c6e2df939e8d7… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 13:13:07 2016 +0100 gdi32: Don't call GetObjectW() unless necessary. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/font.c | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 2faefe8..02efc8a 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -681,13 +681,15 @@ static void update_font_code_page( DC *dc, HANDLE font ) { CHARSETINFO csi; int charset = get_text_charset_info( dc, NULL, 0 ); - LOGFONTW lf; - GetObjectW( font, sizeof(lf), &lf ); + if (charset == ANSI_CHARSET && get_associated_charset_info() & ASSOC_CHARSET_ANSI) + { + LOGFONTW lf; - if (charset == ANSI_CHARSET && !(lf.lfClipPrecision & CLIP_DFA_DISABLE) && - get_associated_charset_info() & ASSOC_CHARSET_ANSI) - charset = DEFAULT_CHARSET; + GetObjectW( font, sizeof(lf), &lf ); + if (!(lf.lfClipPrecision & CLIP_DFA_DISABLE)) + charset = DEFAULT_CHARSET; + } /* Hmm, nicely designed api this one! */ if (TranslateCharsetInfo( ULongToPtr(charset), &csi, TCI_SRCCHARSET) )

1 0

Huw Davies : gdi32: Pass a DC pointer to get_gdi_flat_path().
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 67520ebaf57c76a6b69820a93c902c804843ea7b URL: http://source.winehq.org/git/wine.git/?a=commit;h=67520ebaf57c76a6b69820a93… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:26 2016 +0100 gdi32: Pass a DC pointer to get_gdi_flat_path(). Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/dibdrv/graphics.c | 3 ++- dlls/gdi32/enhmfdrv/graphics.c | 3 ++- dlls/gdi32/gdi_private.h | 2 +- dlls/gdi32/path.c | 8 ++------ 4 files changed, 7 insertions(+), 9 deletions(-) diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 3ea39a1..e6d65e1 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -408,6 +408,7 @@ static BOOL draw_arc( PHYSDEV dev, INT left, INT top, INT right, INT bottom, /* helper for path stroking and filling functions */ static BOOL stroke_and_fill_path( dibdrv_physdev *dev, BOOL stroke, BOOL fill ) { + DC *dc = get_physdev_dc( &dev->dev ); struct gdi_path *path; POINT *points; BYTE *types; @@ -417,7 +418,7 @@ static BOOL stroke_and_fill_path( dibdrv_physdev *dev, BOOL stroke, BOOL fill ) if (dev->brush.style == BS_NULL) fill = FALSE; - if (!(path = get_gdi_flat_path( dev->dev.hdc, fill ? &interior : NULL ))) return FALSE; + if (!(path = get_gdi_flat_path( dc, fill ? &interior : NULL ))) return FALSE; if (!(total = get_gdi_path_data( path, &points, &types ))) goto done; if (stroke && dev->pen_uses_region) outline = CreateRectRgn( 0, 0, 0, 0 ); diff --git a/dlls/gdi32/enhmfdrv/graphics.c b/dlls/gdi32/enhmfdrv/graphics.c index f74e86b..7aaff9d 100644 --- a/dlls/gdi32/enhmfdrv/graphics.c +++ b/dlls/gdi32/enhmfdrv/graphics.c @@ -99,6 +99,7 @@ static void get_points_bounds( RECTL *bounds, const POINT *pts, UINT count, HDC /* helper for path stroke and fill functions */ static BOOL emfdrv_stroke_and_fill_path( PHYSDEV dev, INT type ) { + DC *dc = get_physdev_dc( dev ); EMRSTROKEANDFILLPATH emr; struct gdi_path *path; POINT *points; @@ -107,7 +108,7 @@ static BOOL emfdrv_stroke_and_fill_path( PHYSDEV dev, INT type ) emr.emr.iType = type; emr.emr.nSize = sizeof(emr); - if ((path = get_gdi_flat_path( dev->hdc, NULL ))) + if ((path = get_gdi_flat_path( dc, NULL ))) { int count = get_gdi_path_data( path, &points, &flags ); get_points_bounds( &emr.rclBounds, points, count, 0 ); diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 5bad7b7..3df3a2c 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -334,7 +334,7 @@ typedef struct /* path.c */ extern void free_gdi_path( struct gdi_path *path ) DECLSPEC_HIDDEN; -extern struct gdi_path *get_gdi_flat_path( HDC hdc, HRGN *rgn ) DECLSPEC_HIDDEN; +extern struct gdi_path *get_gdi_flat_path( DC *dc, HRGN *rgn ) DECLSPEC_HIDDEN; extern int get_gdi_path_data( struct gdi_path *path, POINT **points, BYTE **flags ) DECLSPEC_HIDDEN; extern BOOL PATH_SavePath( DC *dst, DC *src ) DECLSPEC_HIDDEN; extern BOOL PATH_RestorePath( DC *dst, DC *src ) DECLSPEC_HIDDEN; diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index 2126dab..c03208f 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -536,24 +536,20 @@ static BOOL PATH_DoArcPart(struct gdi_path *pPath, FLOAT_POINT corners[], /* retrieve a flattened path in device coordinates, and optionally its region */ /* the DC path is deleted; the returned data must be freed by caller using free_gdi_path() */ /* helper for stroke_and_fill_path in the DIB driver */ -struct gdi_path *get_gdi_flat_path( HDC hdc, HRGN *rgn ) +struct gdi_path *get_gdi_flat_path( DC *dc, HRGN *rgn ) { - DC *dc = get_dc_ptr( hdc ); struct gdi_path *ret = NULL; - if (!dc) return NULL; - if (dc->path) { ret = PATH_FlattenPath( dc->path ); free_gdi_path( dc->path ); dc->path = NULL; - if (ret && rgn) *rgn = path_to_region( ret, GetPolyFillMode( hdc ) ); + if (ret && rgn) *rgn = path_to_region( ret, dc->polyFillMode ); } else SetLastError( ERROR_CAN_NOT_COMPLETE ); - release_dc_ptr( dc ); return ret; }

1 0

Huw Davies : gdi32: Pass the path directly to start_new_stroke().
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 1b4f0f8a07c81363aa4946de382e1852675cbec6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b4f0f8a07c81363aa4946de3… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:27 2016 +0100 gdi32: Pass the path directly to start_new_stroke(). Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/path.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index c03208f..a5d01bc 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -298,10 +298,8 @@ static void reverse_points( POINT *points, UINT count ) } /* start a new path stroke if necessary */ -static BOOL start_new_stroke( struct path_physdev *physdev ) +static BOOL start_new_stroke( struct gdi_path *path ) { - struct gdi_path *path = physdev->path; - if (!path->newStroke && path->count && !(path->flags[path->count - 1] & PT_CLOSEFIGURE) && path->points[path->count - 1].x == path->pos.x && @@ -309,7 +307,7 @@ static BOOL start_new_stroke( struct path_physdev *physdev ) return TRUE; path->newStroke = FALSE; - return add_points( physdev->path, &path->pos, 1, PT_MOVETO ) != NULL; + return add_points( path, &path->pos, 1, PT_MOVETO ) != NULL; } /* set current position to the last point that was added to the path */ @@ -330,7 +328,7 @@ static void close_figure( struct gdi_path *path ) static BOOL add_log_points_new_stroke( struct path_physdev *physdev, const POINT *points, DWORD count, BYTE type ) { - if (!start_new_stroke( physdev )) return FALSE; + if (!start_new_stroke( physdev->path )) return FALSE; if (!add_log_points( physdev, points, count, type )) return FALSE; update_current_pos( physdev->path ); return TRUE; @@ -1143,7 +1141,7 @@ static BOOL PATH_Arc( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, } /* arcto: Add a PT_MOVETO only if this is the first entry in a stroke */ - if (lines==-1 && !start_new_stroke( physdev )) return FALSE; + if (lines == -1 && !start_new_stroke( physdev->path )) return FALSE; /* Add the arc to the path with one Bezier spline per quadrant that the * arc spans */

1 0

Huw Davies : gdi32: Pass a DC pointer to the point adding routines.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: f1aaf0efb0344bc7f8d75606dcc02109e38e4443 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f1aaf0efb0344bc7f8d75606d… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:28 2016 +0100 gdi32: Pass a DC pointer to the point adding routines. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/path.c | 54 +++++++++++++++++++++++++++++++----------------------- 1 file changed, 31 insertions(+), 23 deletions(-) diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index a5d01bc..24a9436 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -162,9 +162,8 @@ static struct gdi_path *copy_gdi_path( const struct gdi_path *src_path ) /* Performs a world-to-viewport transformation on the specified point (which * is in floating point format). */ -static inline void INTERNAL_LPTODP_FLOAT( HDC hdc, FLOAT_POINT *point, int count ) +static inline void INTERNAL_LPTODP_FLOAT( DC *dc, FLOAT_POINT *point, int count ) { - DC *dc = get_dc_ptr( hdc ); double x, y; while (count--) @@ -175,7 +174,6 @@ static inline void INTERNAL_LPTODP_FLOAT( HDC hdc, FLOAT_POINT *point, int count point->y = x * dc->xformWorld2Vport.eM12 + y * dc->xformWorld2Vport.eM22 + dc->xformWorld2Vport.eDy; point++; } - release_dc_ptr( dc ); } static inline INT int_from_fixed(FIXED f) @@ -255,16 +253,16 @@ static BOOL PATH_AddEntry(struct gdi_path *pPath, const POINT *pPoint, BYTE flag /* add a number of points, converting them to device coords */ /* return a pointer to the first type byte so it can be fixed up if necessary */ -static BYTE *add_log_points( struct path_physdev *physdev, const POINT *points, DWORD count, BYTE type ) +static BYTE *add_log_points( DC *dc, struct gdi_path *path, const POINT *points, + DWORD count, BYTE type ) { BYTE *ret; - struct gdi_path *path = physdev->path; if (!PATH_ReserveEntries( path, path->count + count )) return NULL; ret = &path->flags[path->count]; memcpy( &path->points[path->count], points, count * sizeof(*points) ); - LPtoDP( physdev->dev.hdc, &path->points[path->count], count ); + lp_to_dp( dc, &path->points[path->count], count ); memset( ret, type, count ); path->count += count; return ret; @@ -325,12 +323,12 @@ static void close_figure( struct gdi_path *path ) } /* add a number of points, starting a new stroke if necessary */ -static BOOL add_log_points_new_stroke( struct path_physdev *physdev, const POINT *points, +static BOOL add_log_points_new_stroke( DC *dc, struct gdi_path *path, const POINT *points, DWORD count, BYTE type ) { - if (!start_new_stroke( physdev->path )) return FALSE; - if (!add_log_points( physdev, points, count, type )) return FALSE; - update_current_pos( physdev->path ); + if (!start_new_stroke( path )) return FALSE; + if (!add_log_points( dc, path, points, count, type )) return FALSE; + update_current_pos( path ); return TRUE; } @@ -889,11 +887,12 @@ static BOOL pathdrv_MoveTo( PHYSDEV dev, INT x, INT y ) static BOOL pathdrv_LineTo( PHYSDEV dev, INT x, INT y ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); POINT point; point.x = x; point.y = y; - return add_log_points_new_stroke( physdev, &point, 1, PT_LINETO ); + return add_log_points_new_stroke( dc, physdev->path, &point, 1, PT_LINETO ); } @@ -1066,6 +1065,7 @@ static BOOL pathdrv_Ellipse( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2 ) static BOOL PATH_Arc( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, INT xStart, INT yStart, INT xEnd, INT yEnd, int direction, int lines ) { + DC *dc = get_physdev_dc( dev ); struct path_physdev *physdev = get_path_physdev( dev ); double angleStart, angleEnd, angleStartQuadrant, angleEndQuadrant=0.0; /* Initialize angleEndQuadrant to silence gcc's warning */ @@ -1091,9 +1091,9 @@ static BOOL PATH_Arc( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, pointStart.y = yStart; pointEnd.x = xEnd; pointEnd.y = yEnd; - INTERNAL_LPTODP_FLOAT(dev->hdc, corners, 2); - INTERNAL_LPTODP_FLOAT(dev->hdc, &pointStart, 1); - INTERNAL_LPTODP_FLOAT(dev->hdc, &pointEnd, 1); + INTERNAL_LPTODP_FLOAT(dc, corners, 2); + INTERNAL_LPTODP_FLOAT(dc, &pointStart, 1); + INTERNAL_LPTODP_FLOAT(dc, &pointEnd, 1); /* Make sure first corner is top left and second corner is bottom right */ if(corners[0].x>corners[1].x) @@ -1266,8 +1266,9 @@ static BOOL pathdrv_Pie( PHYSDEV dev, INT left, INT top, INT right, INT bottom, static BOOL pathdrv_PolyBezierTo( PHYSDEV dev, const POINT *pts, DWORD cbPoints ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); - return add_log_points_new_stroke( physdev, pts, cbPoints, PT_BEZIERTO ); + return add_log_points_new_stroke( dc, physdev->path, pts, cbPoints, PT_BEZIERTO ); } @@ -1277,7 +1278,8 @@ static BOOL pathdrv_PolyBezierTo( PHYSDEV dev, const POINT *pts, DWORD cbPoints static BOOL pathdrv_PolyBezier( PHYSDEV dev, const POINT *pts, DWORD cbPoints ) { struct path_physdev *physdev = get_path_physdev( dev ); - BYTE *type = add_log_points( physdev, pts, cbPoints, PT_BEZIERTO ); + DC *dc = get_physdev_dc( dev ); + BYTE *type = add_log_points( dc, physdev->path, pts, cbPoints, PT_BEZIERTO ); if (!type) return FALSE; type[0] = PT_MOVETO; @@ -1292,6 +1294,7 @@ static BOOL pathdrv_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, { struct path_physdev *physdev = get_path_physdev( dev ); struct gdi_path *path = physdev->path; + DC *dc = get_physdev_dc( dev ); POINT orig_pos; INT i, lastmove = 0; @@ -1310,13 +1313,13 @@ static BOOL pathdrv_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, break; case PT_LINETO: case PT_LINETO | PT_CLOSEFIGURE: - if (!add_log_points_new_stroke( physdev, &pts[i], 1, PT_LINETO )) return FALSE; + if (!add_log_points_new_stroke( dc, path, &pts[i], 1, PT_LINETO )) return FALSE; break; case PT_BEZIERTO: if ((i + 2 < cbPoints) && (types[i + 1] == PT_BEZIERTO) && (types[i + 2] & ~PT_CLOSEFIGURE) == PT_BEZIERTO) { - if (!add_log_points_new_stroke( physdev, &pts[i], 3, PT_BEZIERTO )) return FALSE; + if (!add_log_points_new_stroke( dc, path, &pts[i], 3, PT_BEZIERTO )) return FALSE; i += 2; break; } @@ -1343,10 +1346,11 @@ static BOOL pathdrv_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, static BOOL pathdrv_Polyline( PHYSDEV dev, const POINT *pts, INT count ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); BYTE *type; if (count < 2) return FALSE; - if (!(type = add_log_points( physdev, pts, count, PT_LINETO ))) return FALSE; + if (!(type = add_log_points( dc, physdev->path, pts, count, PT_LINETO ))) return FALSE; type[0] = PT_MOVETO; return TRUE; } @@ -1358,9 +1362,10 @@ static BOOL pathdrv_Polyline( PHYSDEV dev, const POINT *pts, INT count ) static BOOL pathdrv_PolylineTo( PHYSDEV dev, const POINT *pts, INT count ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); if (count < 1) return FALSE; - return add_log_points_new_stroke( physdev, pts, count, PT_LINETO ); + return add_log_points_new_stroke( dc, physdev->path, pts, count, PT_LINETO ); } @@ -1370,10 +1375,11 @@ static BOOL pathdrv_PolylineTo( PHYSDEV dev, const POINT *pts, INT count ) static BOOL pathdrv_Polygon( PHYSDEV dev, const POINT *pts, INT count ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); BYTE *type; if (count < 2) return FALSE; - if (!(type = add_log_points( physdev, pts, count, PT_LINETO ))) return FALSE; + if (!(type = add_log_points( dc, physdev->path, pts, count, PT_LINETO ))) return FALSE; type[0] = PT_MOVETO; type[count - 1] = PT_LINETO | PT_CLOSEFIGURE; return TRUE; @@ -1386,6 +1392,7 @@ static BOOL pathdrv_Polygon( PHYSDEV dev, const POINT *pts, INT count ) static BOOL pathdrv_PolyPolygon( PHYSDEV dev, const POINT* pts, const INT* counts, UINT polygons ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); UINT poly, count; BYTE *type; @@ -1396,7 +1403,7 @@ static BOOL pathdrv_PolyPolygon( PHYSDEV dev, const POINT* pts, const INT* count count += counts[poly]; } - type = add_log_points( physdev, pts, count, PT_LINETO ); + type = add_log_points( dc, physdev->path, pts, count, PT_LINETO ); if (!type) return FALSE; /* make the first point of each polyline a PT_MOVETO, and close the last one */ @@ -1415,6 +1422,7 @@ static BOOL pathdrv_PolyPolygon( PHYSDEV dev, const POINT* pts, const INT* count static BOOL pathdrv_PolyPolyline( PHYSDEV dev, const POINT* pts, const DWORD* counts, DWORD polylines ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); UINT poly, count; BYTE *type; @@ -1425,7 +1433,7 @@ static BOOL pathdrv_PolyPolyline( PHYSDEV dev, const POINT* pts, const DWORD* co count += counts[poly]; } - type = add_log_points( physdev, pts, count, PT_LINETO ); + type = add_log_points( dc, physdev->path, pts, count, PT_LINETO ); if (!type) return FALSE; /* make the first point of each polyline a PT_MOVETO */

1 0

Huw Davies : gdi32: Pass a DC pointer to PATH_CheckCorners().
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 377010dcfce629d52472261e0c3c18252f638abe URL: http://source.winehq.org/git/wine.git/?a=commit;h=377010dcfce629d52472261e0… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:29 2016 +0100 gdi32: Pass a DC pointer to PATH_CheckCorners(). Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/path.c | 23 +++++++++++++---------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index 24a9436..015a059 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -362,7 +362,7 @@ static HRGN path_to_region( const struct gdi_path *path, int mode ) * * Helper function for RoundRect() and Rectangle() */ -static BOOL PATH_CheckCorners( HDC hdc, POINT corners[], INT x1, INT y1, INT x2, INT y2 ) +static BOOL PATH_CheckCorners( DC *dc, POINT corners[], INT x1, INT y1, INT x2, INT y2 ) { INT temp; @@ -371,7 +371,7 @@ static BOOL PATH_CheckCorners( HDC hdc, POINT corners[], INT x1, INT y1, INT x2, corners[0].y=y1; corners[1].x=x2; corners[1].y=y2; - LPtoDP( hdc, corners, 2 ); + lp_to_dp( dc, corners, 2 ); /* Make sure first corner is top left and second corner is bottom right */ if(corners[0].x>corners[1].x) @@ -388,7 +388,7 @@ static BOOL PATH_CheckCorners( HDC hdc, POINT corners[], INT x1, INT y1, INT x2, } /* In GM_COMPATIBLE, don't include bottom and right edges */ - if (GetGraphicsMode( hdc ) == GM_COMPATIBLE) + if (dc->GraphicsMode == GM_COMPATIBLE) { if (corners[0].x == corners[1].x) return FALSE; if (corners[0].y == corners[1].y) return FALSE; @@ -902,10 +902,11 @@ static BOOL pathdrv_LineTo( PHYSDEV dev, INT x, INT y ) static BOOL pathdrv_Rectangle( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2 ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); POINT corners[2], points[4]; BYTE *type; - if (!PATH_CheckCorners( dev->hdc, corners, x1, y1, x2, y2 )) return TRUE; + if (!PATH_CheckCorners( dc, corners, x1, y1, x2, y2 )) return TRUE; points[0].x = corners[1].x; points[0].y = corners[0].y; @@ -913,7 +914,7 @@ static BOOL pathdrv_Rectangle( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2 ) points[2].x = corners[0].x; points[2].y = corners[1].y; points[3] = corners[1]; - if (GetArcDirection( dev->hdc ) == AD_CLOCKWISE) reverse_points( points, 4 ); + if (dc->ArcDirection == AD_CLOCKWISE) reverse_points( points, 4 ); if (!(type = add_points( physdev->path, points, 4, PT_LINETO ))) return FALSE; type[0] = PT_MOVETO; @@ -929,18 +930,19 @@ static BOOL pathdrv_RoundRect( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, INT { const double factor = 0.55428475; /* 4 / 3 * (sqrt(2) - 1) */ struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); POINT corners[2], ellipse[2], points[16]; BYTE *type; double width, height; if (!ell_width || !ell_height) return pathdrv_Rectangle( dev, x1, y1, x2, y2 ); - if (!PATH_CheckCorners( dev->hdc, corners, x1, y1, x2, y2 )) return TRUE; + if (!PATH_CheckCorners( dc, corners, x1, y1, x2, y2 )) return TRUE; ellipse[0].x = ellipse[0].y = 0; ellipse[1].x = ell_width; ellipse[1].y = ell_height; - LPtoDP( dev->hdc, (POINT *)&ellipse, 2 ); + lp_to_dp( dc, (POINT *)&ellipse, 2 ); ell_width = min( abs( ellipse[1].x - ellipse[0].x ), corners[1].x - corners[0].x ); ell_height = min( abs( ellipse[1].y - ellipse[0].y ), corners[1].y - corners[0].y ); width = ell_width / 2.0; @@ -987,7 +989,7 @@ static BOOL pathdrv_RoundRect( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, INT points[15].x = corners[1].x; points[15].y = corners[1].y - GDI_ROUND( height ); - if (GetArcDirection( dev->hdc ) == AD_CLOCKWISE) reverse_points( points, 16 ); + if (dc->ArcDirection == AD_CLOCKWISE) reverse_points( points, 16 ); if (!(type = add_points( physdev->path, points, 16, PT_BEZIERTO ))) return FALSE; type[0] = PT_MOVETO; type[4] = type[8] = type[12] = PT_LINETO; @@ -1003,11 +1005,12 @@ static BOOL pathdrv_Ellipse( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2 ) { const double factor = 0.55428475; /* 4 / 3 * (sqrt(2) - 1) */ struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); POINT corners[2], points[13]; BYTE *type; double width, height; - if (!PATH_CheckCorners( dev->hdc, corners, x1, y1, x2, y2 )) return TRUE; + if (!PATH_CheckCorners( dc, corners, x1, y1, x2, y2 )) return TRUE; width = (corners[1].x - corners[0].x) / 2.0; height = (corners[1].y - corners[0].y) / 2.0; @@ -1044,7 +1047,7 @@ static BOOL pathdrv_Ellipse( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2 ) points[12].x = corners[1].x; points[12].y = corners[1].y - GDI_ROUND( height ); - if (GetArcDirection( dev->hdc ) == AD_CLOCKWISE) reverse_points( points, 13 ); + if (dc->ArcDirection == AD_CLOCKWISE) reverse_points( points, 13 ); if (!(type = add_points( physdev->path, points, 13, PT_BEZIERTO ))) return FALSE; type[0] = PT_MOVETO; close_figure( physdev->path );

1 0

Huw Davies : gdi32: Access a few more properties directly from the DC in the path driver.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: e767a607909377752af355117b9403c25fa4a471 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e767a607909377752af355117… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:30 2016 +0100 gdi32: Access a few more properties directly from the DC in the path driver. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/path.c | 24 +++++++++++++++--------- 1 file changed, 15 insertions(+), 9 deletions(-) diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index 015a059..c204fde 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -702,7 +702,7 @@ HRGN WINAPI PathToRegion(HDC hdc) dc->path = NULL; if (path) { - ret = path_to_region( path, GetPolyFillMode( hdc )); + ret = path_to_region( path, dc->polyFillMode ); free_gdi_path( path ); } } @@ -873,10 +873,12 @@ BOOL PATH_RestorePath( DC *dst, DC *src ) static BOOL pathdrv_MoveTo( PHYSDEV dev, INT x, INT y ) { struct path_physdev *physdev = get_path_physdev( dev ); + DC *dc = get_physdev_dc( dev ); + physdev->path->newStroke = TRUE; physdev->path->pos.x = x; physdev->path->pos.y = y; - LPtoDP( physdev->dev.hdc, &physdev->path->pos, 1 ); + lp_to_dp( dc, &physdev->path->pos, 1 ); return TRUE; } @@ -1137,7 +1139,7 @@ static BOOL PATH_Arc( PHYSDEV dev, INT x1, INT y1, INT x2, INT y2, } /* In GM_COMPATIBLE, don't include bottom and right edges */ - if (GetGraphicsMode(dev->hdc) == GM_COMPATIBLE) + if (dc->GraphicsMode == GM_COMPATIBLE) { corners[1].x--; corners[1].y--; @@ -1225,8 +1227,9 @@ static BOOL pathdrv_AngleArc( PHYSDEV dev, INT x, INT y, DWORD radius, FLOAT eSt static BOOL pathdrv_Arc( PHYSDEV dev, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend ) { + DC *dc = get_physdev_dc( dev ); return PATH_Arc( dev, left, top, right, bottom, xstart, ystart, xend, yend, - GetArcDirection( dev->hdc ), 0 ); + dc->ArcDirection, 0 ); } @@ -1236,8 +1239,9 @@ static BOOL pathdrv_Arc( PHYSDEV dev, INT left, INT top, INT right, INT bottom, static BOOL pathdrv_ArcTo( PHYSDEV dev, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend ) { + DC *dc = get_physdev_dc( dev ); return PATH_Arc( dev, left, top, right, bottom, xstart, ystart, xend, yend, - GetArcDirection( dev->hdc ), -1 ); + dc->ArcDirection, -1 ); } @@ -1247,8 +1251,9 @@ static BOOL pathdrv_ArcTo( PHYSDEV dev, INT left, INT top, INT right, INT bottom static BOOL pathdrv_Chord( PHYSDEV dev, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend ) { + DC *dc = get_physdev_dc( dev ); return PATH_Arc( dev, left, top, right, bottom, xstart, ystart, xend, yend, - GetArcDirection( dev->hdc ), 1 ); + dc->ArcDirection, 1 ); } @@ -1258,8 +1263,9 @@ static BOOL pathdrv_Chord( PHYSDEV dev, INT left, INT top, INT right, INT bottom static BOOL pathdrv_Pie( PHYSDEV dev, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend ) { + DC *dc = get_physdev_dc( dev ); return PATH_Arc( dev, left, top, right, bottom, xstart, ystart, xend, yend, - GetArcDirection( dev->hdc ), 2 ); + dc->ArcDirection, 2 ); } @@ -1311,7 +1317,7 @@ static BOOL pathdrv_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, case PT_MOVETO: path->newStroke = TRUE; path->pos = pts[i]; - LPtoDP( dev->hdc, &path->pos, 1 ); + lp_to_dp( dc, &path->pos, 1 ); lastmove = path->count; break; case PT_LINETO: @@ -2022,7 +2028,7 @@ BOOL nulldrv_BeginPath( PHYSDEV dev ) physdev = get_path_physdev( find_dc_driver( dc, &path_driver )); physdev->path = path; path->pos = dc->cur_pos; - LPtoDP( dev->hdc, &path->pos, 1 ); + lp_to_dp( dc, &path->pos, 1 ); if (dc->path) free_gdi_path( dc->path ); dc->path = NULL; return TRUE;

1 0

Huw Davies : gdi32: Remove an unnecessary call to get_dc_ptr().
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: 637c99f675083984af401217b820017a38cafdc3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=637c99f675083984af401217b… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:31 2016 +0100 gdi32: Remove an unnecessary call to get_dc_ptr(). Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/path.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index c204fde..f6b6552 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -798,12 +798,9 @@ static BOOL pathdrv_CreateDC( PHYSDEV *dev, LPCWSTR driver, LPCWSTR device, LPCWSTR output, const DEVMODEW *devmode ) { struct path_physdev *physdev = HeapAlloc( GetProcessHeap(), 0, sizeof(*physdev) ); - DC *dc; if (!physdev) return FALSE; - dc = get_dc_ptr( (*dev)->hdc ); push_dc_driver( dev, &physdev->dev, &path_driver ); - release_dc_ptr( dc ); return TRUE; }

1 0

Huw Davies : gdi32: Add an internal version of DPtoLP that takes a DC pointer.
by Alexandre Julliard 29 Jul '16

29 Jul '16

Module: wine Branch: master Commit: e1130cbb99fb21f9884cd969b0121dd3ccb827b3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e1130cbb99fb21f9884cd969b… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Jul 29 15:09:32 2016 +0100 gdi32: Add an internal version of DPtoLP that takes a DC pointer. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/clipping.c | 2 +- dlls/gdi32/dc.c | 2 +- dlls/gdi32/font.c | 22 +++++++++++----------- dlls/gdi32/gdi_private.h | 1 + dlls/gdi32/mapping.c | 26 ++++++++++++++++++++------ dlls/gdi32/path.c | 2 +- 6 files changed, 35 insertions(+), 20 deletions(-) diff --git a/dlls/gdi32/clipping.c b/dlls/gdi32/clipping.c index 4e965b9..d5553ee 100644 --- a/dlls/gdi32/clipping.c +++ b/dlls/gdi32/clipping.c @@ -438,7 +438,7 @@ INT WINAPI GetClipBox( HDC hdc, LPRECT rect ) rect->left = rect->right - 1; rect->right = tmp - 1; } - DPtoLP( hdc, (LPPOINT)rect, 2 ); + dp_to_lp( dc, (LPPOINT)rect, 2 ); release_dc_ptr( dc ); TRACE("%p => %d %s\n", hdc, ret, wine_dbgstr_rect( rect )); return ret; diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index f8bf3f8..59a3d9b 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -1484,7 +1484,7 @@ UINT WINAPI GetBoundsRect(HDC hdc, LPRECT rect, UINT flags) rect->bottom = min( rect->bottom, dc->vis_rect.bottom - dc->vis_rect.top ); ret = DCB_SET; } - DPtoLP( hdc, (POINT *)rect, 2 ); + dp_to_lp( dc, (POINT *)rect, 2 ); } else ret = 0; diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 02efc8a..afeebb8 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -1901,7 +1901,7 @@ static RECT get_total_extents( HDC hdc, INT x, INT y, UINT flags, UINT aa_flags, } /* helper for nulldrv_ExtTextOut */ -static void draw_glyph( HDC hdc, INT origin_x, INT origin_y, const GLYPHMETRICS *metrics, +static void draw_glyph( DC *dc, INT origin_x, INT origin_y, const GLYPHMETRICS *metrics, const struct gdi_image_bits *image, const RECT *clip ) { static const BYTE masks[8] = {0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01}; @@ -1941,8 +1941,8 @@ static void draw_glyph( HDC hdc, INT origin_x, INT origin_y, const GLYPHMETRICS } } assert( count <= max_count ); - DPtoLP( hdc, pts, count ); - for (i = 0; i < count; i += 2) Polyline( hdc, pts + i, 2 ); + dp_to_lp( dc, pts, count ); + for (i = 0; i < count; i += 2) Polyline( dc->hSelf, pts + i, 2 ); HeapFree( GetProcessHeap(), 0, pts ); } @@ -1966,7 +1966,7 @@ BOOL nulldrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, const RECT *rect if (brush) { orig = SelectObject( dev->hdc, brush ); - DPtoLP( dev->hdc, (POINT *)&rc, 2 ); + dp_to_lp( dc, (POINT *)&rc, 2 ); PatBlt( dev->hdc, rc.left, rc.top, rc.right - rc.left, rc.bottom - rc.top, PATCOPY ); SelectObject( dev->hdc, orig ); DeleteObject( brush ); @@ -2063,7 +2063,7 @@ BOOL nulldrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, const RECT *rect err = get_glyph_bitmap( dev->hdc, str[i], flags, GGO_BITMAP, &metrics, &image ); if (err) continue; - if (image.ptr) draw_glyph( dev->hdc, x, y, &metrics, &image, (flags & ETO_CLIPPED) ? rect : NULL ); + if (image.ptr) draw_glyph( dc, x, y, &metrics, &image, (flags & ETO_CLIPPED) ? rect : NULL ); if (image.free) image.free( &image ); if (dx) @@ -2424,7 +2424,7 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, { pt.x = x + width.x; pt.y = y + width.y; - DPtoLP(hdc, &pt, 1); + dp_to_lp(dc, &pt, 1); MoveToEx(hdc, pt.x, pt.y, NULL); } break; @@ -2441,7 +2441,7 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, { pt.x = x; pt.y = y; - DPtoLP(hdc, &pt, 1); + dp_to_lp(dc, &pt, 1); MoveToEx(hdc, pt.x, pt.y, NULL); } break; @@ -2494,8 +2494,6 @@ done: if(reordered_str != str) HeapFree(GetProcessHeap(), 0, reordered_str); - release_dc_ptr( dc ); - if (ret && (lf.lfUnderline || lf.lfStrikeOut)) { int underlinePos, strikeoutPos; @@ -2541,7 +2539,7 @@ done: pts[3].y = pts[0].y + underlineWidth * cosEsc; pts[4].x = pts[0].x; pts[4].y = pts[0].y; - DPtoLP(hdc, pts, 5); + dp_to_lp(dc, pts, 5); Polygon(hdc, pts, 5); } @@ -2557,7 +2555,7 @@ done: pts[3].y = pts[0].y + strikeoutWidth * cosEsc; pts[4].x = pts[0].x; pts[4].y = pts[0].y; - DPtoLP(hdc, pts, 5); + dp_to_lp(dc, pts, 5); Polygon(hdc, pts, 5); } @@ -2566,6 +2564,8 @@ done: DeleteObject(hbrush); } + release_dc_ptr( dc ); + return ret; } diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index 3df3a2c..e4633d1 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -304,6 +304,7 @@ extern void GDI_hdc_using_object(HGDIOBJ obj, HDC hdc) DECLSPEC_HIDDEN; extern void GDI_hdc_not_using_object(HGDIOBJ obj, HDC hdc) DECLSPEC_HIDDEN; /* mapping.c */ +extern BOOL dp_to_lp( DC *dc, POINT *points, INT count ) DECLSPEC_HIDDEN; extern void lp_to_dp( DC *dc, POINT *points, INT count ) DECLSPEC_HIDDEN; /* metafile.c */ diff --git a/dlls/gdi32/mapping.c b/dlls/gdi32/mapping.c index 496643e..28b12bc 100644 --- a/dlls/gdi32/mapping.c +++ b/dlls/gdi32/mapping.c @@ -306,13 +306,12 @@ BOOL nulldrv_SetWorldTransform( PHYSDEV dev, const XFORM *xform ) } /*********************************************************************** - * DPtoLP (GDI32.@) + * dp_to_lp + * + * Internal version of DPtoLP that takes a DC *. */ -BOOL WINAPI DPtoLP( HDC hdc, LPPOINT points, INT count ) +BOOL dp_to_lp( DC *dc, POINT *points, INT count ) { - DC * dc = get_dc_ptr( hdc ); - if (!dc) return FALSE; - if (dc->vport2WorldValid) { while (count--) @@ -328,10 +327,25 @@ BOOL WINAPI DPtoLP( HDC hdc, LPPOINT points, INT count ) points++; } } - release_dc_ptr( dc ); return (count < 0); } +/*********************************************************************** + * DPtoLP (GDI32.@) + */ +BOOL WINAPI DPtoLP( HDC hdc, POINT *points, INT count ) +{ + DC * dc = get_dc_ptr( hdc ); + BOOL ret; + + if (!dc) return FALSE; + + ret = dp_to_lp( dc, points, count ); + + release_dc_ptr( dc ); + return ret; +} + /*********************************************************************** * lp_to_dp diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index f6b6552..4ae9ba2 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -670,7 +670,7 @@ INT WINAPI GetPath(HDC hdc, LPPOINT pPoints, LPBYTE pTypes, INT nSize) memcpy(pTypes, dc->path->flags, sizeof(BYTE)*dc->path->count); /* Convert the points to logical coordinates */ - if(!DPtoLP(hdc, pPoints, dc->path->count)) + if(!dp_to_lp(dc, pPoints, dc->path->count)) { /* FIXME: Is this the correct value? */ SetLastError(ERROR_CAN_NOT_COMPLETE);

1 0