wine-commits September 2021

wine-commits@winehq.org

2 participants
1030 discussions

Jacek Caban : gdi32: Directly use ntgdi functions in painting.c.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: 9f88685f5460da2bf6e39f24191a6b65be7baec8 URL: https://source.winehq.org/git/wine.git/?a=commit;h=9f88685f5460da2bf6e39f24… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 14 21:52:55 2021 +0200 gdi32: Directly use ntgdi functions in painting.c. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/painting.c | 26 +++++++++++++++++--------- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/dlls/gdi32/painting.c b/dlls/gdi32/painting.c index a7f06ea7df3..dba91daf788 100644 --- a/dlls/gdi32/painting.c +++ b/dlls/gdi32/painting.c @@ -40,13 +40,17 @@ WINE_DEFAULT_DEBUG_CHANNEL(gdi); BOOL CDECL nulldrv_AngleArc( PHYSDEV dev, INT x, INT y, DWORD radius, FLOAT start, FLOAT sweep ) { + DC *dc = get_physdev_dc( dev ); INT x1 = GDI_ROUND( x + cos( start * M_PI / 180 ) * radius ); INT y1 = GDI_ROUND( y - sin( start * M_PI / 180 ) * radius ); INT x2 = GDI_ROUND( x + cos( (start + sweep) * M_PI / 180) * radius ); INT y2 = GDI_ROUND( y - sin( (start + sweep) * M_PI / 180) * radius ); - INT arcdir = SetArcDirection( dev->hdc, sweep >= 0 ? AD_COUNTERCLOCKWISE : AD_CLOCKWISE ); - BOOL ret = ArcTo( dev->hdc, x - radius, y - radius, x + radius, y + radius, x1, y1, x2, y2 ); - SetArcDirection( dev->hdc, arcdir ); + INT arcdir = dc->attr->arc_direction; + BOOL ret; + dc->attr->arc_direction = sweep >= 0 ? AD_COUNTERCLOCKWISE : AD_CLOCKWISE; + ret = NtGdiArcInternal( NtGdiArcTo, dev->hdc, x - radius, y - radius, x + radius, y + radius, + x1, y1, x2, y2 ); + dc->attr->arc_direction = arcdir; return ret; } @@ -64,9 +68,10 @@ BOOL CDECL nulldrv_ArcTo( PHYSDEV dev, INT left, INT top, INT right, INT bottom, if (!height || !width) return FALSE; /* draw a line from the current position to the starting point of the arc, then draw the arc */ angle = atan2( (ystart - ycenter) / height, (xstart - xcenter) / width ); - LineTo( dev->hdc, GDI_ROUND( xcenter + cos(angle) * xradius ), - GDI_ROUND( ycenter + sin(angle) * yradius )); - return Arc( dev->hdc, left, top, right, bottom, xstart, ystart, xend, yend ); + NtGdiLineTo( dev->hdc, GDI_ROUND( xcenter + cos(angle) * xradius ), + GDI_ROUND( ycenter + sin(angle) * yradius )); + return NtGdiArcInternal( NtGdiArc, dev->hdc, left, top, right, bottom, + xstart, ystart, xend, yend ); } BOOL CDECL nulldrv_FillRgn( PHYSDEV dev, HRGN rgn, HBRUSH brush ) @@ -99,9 +104,12 @@ BOOL CDECL nulldrv_FrameRgn( PHYSDEV dev, HRGN rgn, HBRUSH brush, INT width, INT BOOL CDECL nulldrv_InvertRgn( PHYSDEV dev, HRGN rgn ) { - INT prev_rop = SetROP2( dev->hdc, R2_NOT ); - BOOL ret = NtGdiFillRgn( dev->hdc, rgn, get_stock_object(BLACK_BRUSH) ); - SetROP2( dev->hdc, prev_rop ); + DC *dc = get_physdev_dc( dev ); + INT prev_rop = dc->attr->rop_mode; + BOOL ret; + dc->attr->rop_mode = R2_NOT; + ret = NtGdiFillRgn( dev->hdc, rgn, get_stock_object(BLACK_BRUSH) ); + dc->attr->rop_mode = prev_rop; return ret; }

1 0

Jacek Caban : win32u: Directly use NtGdiGetDeviceCaps in ntgdi functions.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: d3d6b832a3ac1347aa444b87547b81b041631fbf URL: https://source.winehq.org/git/wine.git/?a=commit;h=d3d6b832a3ac1347aa444b87… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 14 21:52:46 2021 +0200 win32u: Directly use NtGdiGetDeviceCaps in ntgdi functions. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/bitmap.c | 2 +- dlls/gdi32/clipping.c | 4 ++-- dlls/gdi32/dc.c | 8 ++++---- dlls/gdi32/driver.c | 30 +++++++++++++++--------------- dlls/gdi32/mapping.c | 12 ++++++------ dlls/gdi32/palette.c | 7 +++---- 6 files changed, 31 insertions(+), 32 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=d3d6b832a3ac1347aa44…

1 0

Jacek Caban : win32u: Use NtGdiSetBrushOrg in NtGdiMaskBlt.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: ad4f9396e00f473f695e237ded01f461268d201f URL: https://source.winehq.org/git/wine.git/?a=commit;h=ad4f9396e00f473f695e237d… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 14 21:52:23 2021 +0200 win32u: Use NtGdiSetBrushOrg in NtGdiMaskBlt. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/bitblt.c | 2 +- dlls/gdi32/dc.c | 16 ++++++++++++++++ 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index 0de7932aa95..83ef949ec3f 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -810,7 +810,7 @@ BOOL WINAPI NtGdiMaskBlt( HDC hdcDest, INT nXDest, INT nYDest, INT nWidth, INT n /* combine both using the mask as a pattern brush */ NtGdiSelectBrush(hDC2, hbrMask); - SetBrushOrgEx(hDC2, -xMask, -yMask, NULL); + NtGdiSetBrushOrg( hDC2, -xMask, -yMask, NULL ); /* (D & P) | (S & ~P) */ NtGdiBitBlt(hDC2, 0, 0, nWidth, nHeight, hDC1, 0, 0, 0xac0744, 0, 0 ); NtGdiSelectBrush(hDC2, hbrTmp); diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index 3c84d2024fe..21ca88bab30 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -884,6 +884,22 @@ BOOL WINAPI NtGdiGetAndSetDCDword( HDC hdc, UINT method, DWORD value, DWORD *pre } +/*********************************************************************** + * NtGdiSetBrushOrg (win32u.@) + */ +BOOL WINAPI NtGdiSetBrushOrg( HDC hdc, INT x, INT y, POINT *oldorg ) +{ + DC *dc; + + if (!(dc = get_dc_ptr( hdc ))) return FALSE; + if (oldorg) *oldorg = dc->attr->brush_org; + dc->attr->brush_org.x = x; + dc->attr->brush_org.y = y; + release_dc_ptr( dc ); + return TRUE; +} + + /*********************************************************************** * NtGdiGetTransform (win32u.@) *

1 0

Jacek Caban : gdi32: Direcly use ntgdi functions in dib driver.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: c165a83478a41fab731473828dcc8a2c7db6f7e2 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c165a83478a41fab73147382… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 14 21:52:08 2021 +0200 gdi32: Direcly use ntgdi functions in dib driver. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/dibdrv/dc.c | 4 +--- dlls/gdi32/dibdrv/graphics.c | 10 ++++++---- dlls/gdi32/dibdrv/objects.c | 19 ++++++++++++------- 3 files changed, 19 insertions(+), 14 deletions(-) diff --git a/dlls/gdi32/dibdrv/dc.c b/dlls/gdi32/dibdrv/dc.c index c4474cbb63e..8b2eb6a7480 100644 --- a/dlls/gdi32/dibdrv/dc.c +++ b/dlls/gdi32/dibdrv/dc.c @@ -30,8 +30,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(dib); -extern BOOL WINAPI GdiSetPixelFormat( HDC hdc, INT fmt, const PIXELFORMATDESCRIPTOR *pfd ); - static const struct osmesa_funcs *osmesa_funcs; static const DWORD bit_fields_888[3] = {0xff0000, 0x00ff00, 0x0000ff}; @@ -564,7 +562,7 @@ static BOOL WINAPI dibdrv_wglMakeCurrent( HDC hdc, struct wgl_context *context ) static BOOL WINAPI dibdrv_wglSetPixelFormat( HDC hdc, int fmt, const PIXELFORMATDESCRIPTOR *descr ) { if (fmt <= 0 || fmt > ARRAY_SIZE( pixel_formats )) return FALSE; - return GdiSetPixelFormat( hdc, fmt, descr ); + return NtGdiSetPixelFormat( hdc, fmt ); } /*********************************************************************** diff --git a/dlls/gdi32/dibdrv/graphics.c b/dlls/gdi32/dibdrv/graphics.c index 9bd6dc557a5..9ebcaf7e4a3 100644 --- a/dlls/gdi32/dibdrv/graphics.c +++ b/dlls/gdi32/dibdrv/graphics.c @@ -558,7 +558,7 @@ static struct cached_font *add_cached_font( DC *dc, HFONT hfont, UINT aa_flags ) struct cached_font font, *ptr, *last_unused = NULL; UINT i = 0, j, k; - GetObjectW( hfont, sizeof(font.lf), &font.lf ); + NtGdiExtGetObjectW( hfont, sizeof(font.lf), &font.lf ); font.xform = dc->xformWorld2Vport; font.xform.eDx = font.xform.eDy = 0; /* unused, would break hashing */ if (dc->attr->graphics_mode == GM_COMPATIBLE) @@ -761,7 +761,8 @@ static struct cached_glyph *cache_glyph_bitmap( DC *dc, struct cached_font *font for (i = 0; i < ARRAY_SIZE( indices ); i++) { index = indices[i]; - ret = GetGlyphOutlineW( dc->hSelf, index, ggo_flags, &metrics, 0, NULL, &identity ); + ret = NtGdiGetGlyphOutline( dc->hSelf, index, ggo_flags, &metrics, 0, NULL, + &identity, FALSE ); if (ret != GDI_ERROR) break; } if (ret == GDI_ERROR) return NULL; @@ -776,7 +777,8 @@ static struct cached_glyph *cache_glyph_bitmap( DC *dc, struct cached_font *font if (bit_count == 8) pad = padding[ metrics.gmBlackBoxX % 4 ]; - ret = GetGlyphOutlineW( dc->hSelf, index, ggo_flags, &metrics, size, glyph->bits, &identity ); + ret = NtGdiGetGlyphOutline( dc->hSelf, index, ggo_flags, &metrics, size, glyph->bits, + &identity, FALSE ); if (ret == GDI_ERROR) { HeapFree( GetProcessHeap(), 0, glyph ); @@ -1421,7 +1423,7 @@ BOOL CDECL dibdrv_Rectangle( PHYSDEV dev, INT left, INT top, INT right, INT bott { if (pdev->brush.style != BS_NULL) { - HRGN interior = CreateRectRgnIndirect( &rect ); + HRGN interior = NtGdiCreateRectRgn( rect.left, rect.top, rect.right, rect.bottom ); NtGdiCombineRgn( interior, interior, outline, RGN_DIFF ); brush_region( pdev, interior ); diff --git a/dlls/gdi32/dibdrv/objects.c b/dlls/gdi32/dibdrv/objects.c index e7eff6fdd03..a61ae3e226f 100644 --- a/dlls/gdi32/dibdrv/objects.c +++ b/dlls/gdi32/dibdrv/objects.c @@ -1272,6 +1272,11 @@ static void add_cap( dibdrv_physdev *pdev, HRGN region, HRGN round_cap, const PO #define round( f ) (((f) > 0) ? (f) + 0.5 : (f) - 0.5) +static HRGN create_polygon_region( const POINT *points, INT count, INT mode ) +{ + return create_polypolygon_region( points, &count, 1, mode, NULL ); +} + /******************************************************************************* * create_miter_region * @@ -1326,7 +1331,7 @@ static HRGN create_miter_region( dibdrv_physdev *pdev, const POINT *pt, pts[3] = face_2->end; pts[4] = face_1->end; - return CreatePolygonRgn( pts, 5, ALTERNATE ); + return create_polygon_region( pts, 5, ALTERNATE ); } static void add_join( dibdrv_physdev *pdev, HRGN region, HRGN round_cap, const POINT *pt, @@ -1360,7 +1365,7 @@ static void add_join( dibdrv_physdev *pdev, HRGN region, HRGN round_cap, const P pts[1] = face_2->end; pts[2] = face_1->end; pts[3] = face_2->start; - join = CreatePolygonRgn( pts, 4, ALTERNATE ); + join = create_polygon_region( pts, 4, ALTERNATE ); break; } @@ -1497,7 +1502,7 @@ static BOOL wide_line_segment( dibdrv_physdev *pdev, HRGN total, set_rect( &clip_rect, seg_pts[2].x, seg_pts[3].y, seg_pts[0].x, seg_pts[1].y ); if (clip_rect_to_dib( &pdev->dib, &clip_rect )) { - segment = CreatePolygonRgn( seg_pts, 4, ALTERNATE ); + segment = create_polygon_region( seg_pts, 4, ALTERNATE ); NtGdiCombineRgn( total, total, segment, RGN_OR ); NtGdiDeleteObjectApp( segment ); } @@ -2134,7 +2139,7 @@ HBRUSH CDECL dibdrv_SelectBrush( PHYSDEV dev, HBRUSH hbrush, const struct brush_ TRACE("(%p, %p)\n", dev, hbrush); - GetObjectW( hbrush, sizeof(logbrush), &logbrush ); + NtGdiExtGetObjectW( hbrush, sizeof(logbrush), &logbrush ); if (hbrush == get_stock_object( DC_BRUSH )) logbrush.lbColor = dc->attr->brush_color; @@ -2157,16 +2162,16 @@ HPEN CDECL dibdrv_SelectPen( PHYSDEV dev, HPEN hpen, const struct brush_pattern TRACE("(%p, %p)\n", dev, hpen); - if (!GetObjectW( hpen, sizeof(logpen), &logpen )) + if (!NtGdiExtGetObjectW( hpen, sizeof(logpen), &logpen )) { /* must be an extended pen */ - INT size = GetObjectW( hpen, 0, NULL ); + INT size = NtGdiExtGetObjectW( hpen, 0, NULL ); if (!size) return 0; elp = HeapAlloc( GetProcessHeap(), 0, size ); - GetObjectW( hpen, size, elp ); + NtGdiExtGetObjectW( hpen, size, elp ); logpen.lopnStyle = elp->elpPenStyle; logpen.lopnWidth.x = elp->elpWidth; /* cosmetic ext pens are always 1-pixel wide */

1 0

Jacek Caban : gdi32: Introduce get_gdi_object_type.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: a222f974c3144d98d690c1235d5a53c3c39d7334 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a222f974c3144d98d690c123… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Sep 14 21:51:51 2021 +0200 gdi32: Introduce get_gdi_object_type. And use it instead of GetObjectType in ntgdi functions. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdi32/bitblt.c | 2 +- dlls/gdi32/bitmap.c | 4 ++-- dlls/gdi32/dc.c | 6 +++--- dlls/gdi32/dib.c | 4 ++-- dlls/gdi32/gdiobj.c | 5 +++++ dlls/gdi32/ntgdi_private.h | 1 + dlls/gdi32/palette.c | 2 +- dlls/gdi32/path.c | 14 ++++++++------ 8 files changed, 23 insertions(+), 15 deletions(-) diff --git a/dlls/gdi32/bitblt.c b/dlls/gdi32/bitblt.c index cff84bc1dce..0de7932aa95 100644 --- a/dlls/gdi32/bitblt.c +++ b/dlls/gdi32/bitblt.c @@ -866,7 +866,7 @@ BOOL WINAPI NtGdiTransparentBlt( HDC hdcDest, int xDest, int yDest, int widthDes if(oldStretchMode == BLACKONWHITE || oldStretchMode == WHITEONBLACK) SetStretchBltMode(hdcSrc, COLORONCOLOR); hdcWork = NtGdiCreateCompatibleDC( hdcDest ); - if ((GetObjectType( hdcDest ) != OBJ_MEMDC || + if ((get_gdi_object_type( hdcDest ) != NTGDI_OBJ_MEMDC || NtGdiExtGetObjectW( NtGdiGetDCObject( hdcDest, NTGDI_OBJ_SURF ), sizeof(dib), &dib ) == sizeof(BITMAP)) && NtGdiGetDeviceCaps( hdcDest, BITSPIXEL ) == 32) diff --git a/dlls/gdi32/bitmap.c b/dlls/gdi32/bitmap.c index ad04603bf69..4a8b899dd52 100644 --- a/dlls/gdi32/bitmap.c +++ b/dlls/gdi32/bitmap.c @@ -58,7 +58,7 @@ HBITMAP WINAPI NtGdiCreateCompatibleBitmap( HDC hdc, INT width, INT height ) if (!width || !height) return 0; - if (GetObjectType( hdc ) != OBJ_MEMDC) + if (get_gdi_object_type( hdc ) != NTGDI_OBJ_MEMDC) return NtGdiCreateBitmap( width, height, NtGdiGetDeviceCaps( hdc, PLANES ), NtGdiGetDeviceCaps( hdc, BITSPIXEL ), NULL ); @@ -373,7 +373,7 @@ HGDIOBJ WINAPI NtGdiSelectBitmap( HDC hdc, HGDIOBJ handle ) if (!(dc = get_dc_ptr( hdc ))) return 0; - if (GetObjectType( hdc ) != OBJ_MEMDC) + if (get_gdi_object_type( hdc ) != NTGDI_OBJ_MEMDC) { ret = 0; goto done; diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index f7b56b06f6f..3c84d2024fe 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -375,7 +375,7 @@ void DC_UpdateXforms( DC *dc ) /* Reselect the font and pen back into the dc so that the size gets updated. */ if (linear_xform_cmp( &oldworld2vport, &dc->xformWorld2Vport ) && - GetObjectType( dc->hSelf ) != OBJ_METADC) + get_gdi_object_type( dc->hSelf ) != NTGDI_OBJ_METADC) { NtGdiSelectFont(dc->hSelf, dc->hFont); NtGdiSelectPen(dc->hSelf, dc->hPen); @@ -1012,7 +1012,7 @@ BOOL WINAPI NtGdiGetDeviceGammaRamp( HDC hdc, void *ptr ) TRACE("%p, %p\n", hdc, ptr); if( dc ) { - if (GetObjectType( hdc ) != OBJ_MEMDC) + if (get_gdi_object_type( hdc ) != NTGDI_OBJ_MEMDC) { PHYSDEV physdev = GET_DC_PHYSDEV( dc, pGetDeviceGammaRamp ); ret = physdev->funcs->pGetDeviceGammaRamp( physdev, ptr ); @@ -1112,7 +1112,7 @@ BOOL WINAPI NtGdiSetDeviceGammaRamp( HDC hdc, void *ptr ) TRACE( "%p, %p\n", hdc, ptr ); if( dc ) { - if (GetObjectType( hdc ) != OBJ_MEMDC) + if (get_gdi_object_type( hdc ) != NTGDI_OBJ_MEMDC) { PHYSDEV physdev = GET_DC_PHYSDEV( dc, pSetDeviceGammaRamp ); diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c index f539df6d41e..9f10f479c2f 100644 --- a/dlls/gdi32/dib.c +++ b/dlls/gdi32/dib.c @@ -1689,8 +1689,8 @@ NTSTATUS WINAPI NtGdiDdDDIDestroyDCFromMemory( const D3DKMT_DESTROYDCFROMMEMORY TRACE("dc %p, bitmap %p.\n", desc->hDc, desc->hBitmap); - if (GetObjectType( desc->hDc ) != OBJ_MEMDC || - GetObjectType( desc->hBitmap ) != OBJ_BITMAP) return STATUS_INVALID_PARAMETER; + if (get_gdi_object_type( desc->hDc ) != NTGDI_OBJ_MEMDC || + get_gdi_object_type( desc->hBitmap ) != NTGDI_OBJ_BITMAP) return STATUS_INVALID_PARAMETER; NtGdiDeleteObjectApp( desc->hBitmap ); NtGdiDeleteObjectApp( desc->hDc ); diff --git a/dlls/gdi32/gdiobj.c b/dlls/gdi32/gdiobj.c index 5787231d087..917d1c91f01 100644 --- a/dlls/gdi32/gdiobj.c +++ b/dlls/gdi32/gdiobj.c @@ -816,6 +816,11 @@ void *free_gdi_handle( HGDIOBJ handle ) return object; } +DWORD get_gdi_object_type( HGDIOBJ obj ) +{ + GDI_HANDLE_ENTRY *entry = handle_entry( obj ); + return entry ? entry->ExtType << NTGDI_HANDLE_TYPE_SHIFT : 0; +} /*********************************************************************** * get_any_obj_ptr diff --git a/dlls/gdi32/ntgdi_private.h b/dlls/gdi32/ntgdi_private.h index 7f21608b244..8e122dec6a0 100644 --- a/dlls/gdi32/ntgdi_private.h +++ b/dlls/gdi32/ntgdi_private.h @@ -383,6 +383,7 @@ extern BOOL GDI_dec_ref_count( HGDIOBJ handle ) DECLSPEC_HIDDEN; extern DWORD get_dpi(void) DECLSPEC_HIDDEN; extern DWORD get_system_dpi(void) DECLSPEC_HIDDEN; extern HGDIOBJ get_stock_object( INT obj ) DECLSPEC_HIDDEN; +extern DWORD get_gdi_object_type( HGDIOBJ obj ) DECLSPEC_HIDDEN; /* mapping.c */ extern BOOL dp_to_lp( DC *dc, POINT *points, INT count ) DECLSPEC_HIDDEN; diff --git a/dlls/gdi32/palette.c b/dlls/gdi32/palette.c index e48d4501c6d..c8cd98f6778 100644 --- a/dlls/gdi32/palette.c +++ b/dlls/gdi32/palette.c @@ -518,7 +518,7 @@ HPALETTE WINAPI GDISelectPalette( HDC hdc, HPALETTE hpal, WORD wBkg) TRACE("%p %p\n", hdc, hpal ); - if (GetObjectType(hpal) != OBJ_PAL) + if (get_gdi_object_type(hpal) != NTGDI_OBJ_PAL) { WARN("invalid selected palette %p\n",hpal); return 0; diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index 94c69302d06..cc6c6c714c2 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -1627,14 +1627,16 @@ static struct gdi_path *PATH_WidenPath(DC *dc) elp = HeapAlloc( GetProcessHeap(), 0, size ); GetObjectW( dc->hPen, size, elp ); - obj_type = GetObjectType(dc->hPen); - if(obj_type == OBJ_PEN) { + obj_type = get_gdi_object_type(dc->hPen); + switch (obj_type) + { + case NTGDI_OBJ_PEN: penStyle = ((LOGPEN*)elp)->lopnStyle; - } - else if(obj_type == OBJ_EXTPEN) { + break; + case NTGDI_OBJ_EXTPEN: penStyle = elp->elpPenStyle; - } - else { + break; + default: SetLastError(ERROR_CAN_NOT_COMPLETE); HeapFree( GetProcessHeap(), 0, elp ); return NULL;

1 0

Nikolay Sivov : msi: Fixup Installer\\Patches\\<...> key path (Coverity).
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: 856e8ea298eebf0eaedd5f352476e6e986b1bd67 URL: https://source.winehq.org/git/wine.git/?a=commit;h=856e8ea298eebf0eaedd5f35… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 14 22:44:50 2021 +0300 msi: Fixup Installer\\Patches\\<...> key path (Coverity). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Hans Leidekker <hans(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msi/registry.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/msi/registry.c b/dlls/msi/registry.c index 623bc1866d3..5dc3c0a81c3 100644 --- a/dlls/msi/registry.c +++ b/dlls/msi/registry.c @@ -773,8 +773,8 @@ UINT MSIREG_OpenPatchesKey(LPCWSTR szPatch, HKEY *key, BOOL create) if (!squash_guid( szPatch, squashed_pc )) return ERROR_FUNCTION_FAILED; TRACE("%s squashed %s\n", debugstr_w(szPatch), debugstr_w(squashed_pc)); - swprintf( keypath, ARRAY_SIZE(keypath), L"Software\\Microsoft\\Windows\\CurrentVersion\\Installer\\Patches", - squashed_pc ); + lstrcpyW( keypath, L"Software\\Microsoft\\Windows\\CurrentVersion\\Installer\\Patches\\" ); + lstrcatW( keypath, squashed_pc ); if (create) return RegCreateKeyExW(HKEY_LOCAL_MACHINE, keypath, 0, NULL, 0, access, NULL, key, NULL); return RegOpenKeyExW(HKEY_LOCAL_MACHINE, keypath, 0, access, key);

1 0

Nikolay Sivov : d3dx10_36/tests: Enable tests.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: fbb8ab801ef13f422cc73a169478d4eb4623fe56 URL: https://source.winehq.org/git/wine.git/?a=commit;h=fbb8ab801ef13f422cc73a16… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Sep 14 15:39:00 2021 +0300 d3dx10_36/tests: Enable tests. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- configure | 1 + configure.ac | 1 + dlls/d3dx10_36/Makefile.in | 1 + dlls/d3dx10_36/tests/Makefile.in | 6 ++++++ 4 files changed, 9 insertions(+) diff --git a/configure b/configure index 4306ea70100..00a5be6a504 100755 --- a/configure +++ b/configure @@ -20341,6 +20341,7 @@ wine_fn_config_makefile dlls/d3dx10_33 enable_d3dx10_33 wine_fn_config_makefile dlls/d3dx10_34 enable_d3dx10_34 wine_fn_config_makefile dlls/d3dx10_35 enable_d3dx10_35 wine_fn_config_makefile dlls/d3dx10_36 enable_d3dx10_36 +wine_fn_config_makefile dlls/d3dx10_36/tests enable_tests wine_fn_config_makefile dlls/d3dx10_37 enable_d3dx10_37 wine_fn_config_makefile dlls/d3dx10_37/tests enable_tests wine_fn_config_makefile dlls/d3dx10_38 enable_d3dx10_38 diff --git a/configure.ac b/configure.ac index 6ca4eb5c923..dea5fe61bf3 100644 --- a/configure.ac +++ b/configure.ac @@ -3065,6 +3065,7 @@ WINE_CONFIG_MAKEFILE(dlls/d3dx10_33) WINE_CONFIG_MAKEFILE(dlls/d3dx10_34) WINE_CONFIG_MAKEFILE(dlls/d3dx10_35) WINE_CONFIG_MAKEFILE(dlls/d3dx10_36) +WINE_CONFIG_MAKEFILE(dlls/d3dx10_36/tests) WINE_CONFIG_MAKEFILE(dlls/d3dx10_37) WINE_CONFIG_MAKEFILE(dlls/d3dx10_37/tests) WINE_CONFIG_MAKEFILE(dlls/d3dx10_38) diff --git a/dlls/d3dx10_36/Makefile.in b/dlls/d3dx10_36/Makefile.in index efd498f7e2c..a15593f62f1 100644 --- a/dlls/d3dx10_36/Makefile.in +++ b/dlls/d3dx10_36/Makefile.in @@ -1,4 +1,5 @@ MODULE = d3dx10_36.dll +IMPORTLIB = d3dx10_36 EXTRADLLFLAGS = -mno-cygwin -Wb,--prefer-native diff --git a/dlls/d3dx10_36/tests/Makefile.in b/dlls/d3dx10_36/tests/Makefile.in new file mode 100644 index 00000000000..22e9a91f581 --- /dev/null +++ b/dlls/d3dx10_36/tests/Makefile.in @@ -0,0 +1,6 @@ +TESTDLL = d3dx10_36.dll +IMPORTS = d3dx10_36 ole32 gdi32 +PARENTSRC = ../../d3dx10_43/tests + +C_SRCS = \ + d3dx10.c

1 0

Nikolay Sivov : d3d10/effect: Remove no longer used fourcc symbols.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: edf965389b8e2672c48a8f2b96f05d4950bff622 URL: https://source.winehq.org/git/wine.git/?a=commit;h=edf965389b8e2672c48a8f2b… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Sep 12 16:37:04 2021 +0300 d3d10/effect: Remove no longer used fourcc symbols. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/effect.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/dlls/d3d10/effect.c b/dlls/d3d10/effect.c index a9e97725659..e77d0691fcf 100644 --- a/dlls/d3d10/effect.c +++ b/dlls/d3d10/effect.c @@ -29,9 +29,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(d3d10); ((DWORD)(ch2) << 16) | ((DWORD)(ch3) << 24 )) #define TAG_DXBC MAKE_TAG('D', 'X', 'B', 'C') #define TAG_FX10 MAKE_TAG('F', 'X', '1', '0') -#define TAG_ISGN MAKE_TAG('I', 'S', 'G', 'N') -#define TAG_OSGN MAKE_TAG('O', 'S', 'G', 'N') -#define TAG_SHDR MAKE_TAG('S', 'H', 'D', 'R') #define D3D10_FX10_TYPE_COLUMN_SHIFT 11 #define D3D10_FX10_TYPE_COLUMN_MASK (0x7 << D3D10_FX10_TYPE_COLUMN_SHIFT)

1 0

Nikolay Sivov : d3d10/tests: Add a test for default numeric variable values.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: 4a3e038022d73b5d7ca3f6b4d0881f2f3497c39c URL: https://source.winehq.org/git/wine.git/?a=commit;h=4a3e038022d73b5d7ca3f6b4… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Sep 12 16:37:03 2021 +0300 d3d10/tests: Add a test for default numeric variable values. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/tests/effect.c | 112 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 112 insertions(+) diff --git a/dlls/d3d10/tests/effect.c b/dlls/d3d10/tests/effect.c index 88b6b2e170d..9223fdf51ef 100644 --- a/dlls/d3d10/tests/effect.c +++ b/dlls/d3d10/tests/effect.c @@ -6638,6 +6638,117 @@ todo_wine ok(!refcount, "Device has %u references left.\n", refcount); } +#if 0 +float4 f4 = {1,2,3,4}; +float2 f2 = {1,2}; +float1 f1 = {1}; +int i = 10; +int i2[2] = {9,12}; +float f = 0.2f; +#endif +static DWORD fx_test_default_variable_value[] = +{ + 0x43425844, 0xe57b41f2, 0x98392802, 0xed3d94b4, 0xc4074c4f, 0x00000001, 0x00000248, 0x00000001, + 0x00000024, 0x30315846, 0x0000021c, 0xfeff1001, 0x00000001, 0x00000006, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000110, 0x00000000, 0x00000000, 0x00000000, 0x00000000, + 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x6f6c4724, + 0x736c6162, 0x6f6c6600, 0x00347461, 0x0000000d, 0x00000001, 0x00000000, 0x00000010, 0x00000010, + 0x00000010, 0x0000210a, 0x00003466, 0x003f8000, 0x00400000, 0x00404000, 0x66408000, 0x74616f6c, + 0x00430032, 0x00010000, 0x00000000, 0x00080000, 0x00100000, 0x00080000, 0x110a0000, 0x32660000, + 0x80000000, 0x0000003f, 0x6f6c6640, 0x00317461, 0x00000071, 0x00000001, 0x00000000, 0x00000004, + 0x00000010, 0x00000004, 0x0000090a, 0x00003166, 0x693f8000, 0x9b00746e, 0x01000000, 0x00000000, + 0x04000000, 0x10000000, 0x04000000, 0x11000000, 0x69000009, 0x00000a00, 0x00009b00, 0x00000100, + 0x00000200, 0x00001400, 0x00001000, 0x00000800, 0x00091100, 0x00326900, 0x00000009, 0x0000000c, + 0x616f6c66, 0x00e80074, 0x00010000, 0x00000000, 0x00040000, 0x00100000, 0x00040000, 0x09090000, + 0x00660000, 0x3e4ccccd, 0x00000004, 0x00000040, 0x00000000, 0x00000006, 0xffffffff, 0x00000000, + 0x00000030, 0x00000014, 0x00000000, 0x00000000, 0x00000033, 0x00000000, 0x00000000, 0x00000066, + 0x0000004a, 0x00000000, 0x00000010, 0x00000069, 0x00000000, 0x00000000, 0x00000094, 0x00000078, + 0x00000000, 0x00000018, 0x00000097, 0x00000000, 0x00000000, 0x000000bb, 0x0000009f, 0x00000000, + 0x0000001c, 0x000000bd, 0x00000000, 0x00000000, 0x000000dd, 0x000000c1, 0x00000000, 0x00000020, + 0x000000e0, 0x00000000, 0x00000000, 0x0000010a, 0x000000ee, 0x00000000, 0x00000034, 0x0000010c, + 0x00000000, 0x00000000, +}; + +static void test_effect_default_variable_value(void) +{ + ID3D10EffectVectorVariable *vector; + ID3D10EffectScalarVariable *scalar; + float float_v[4], float_s; + ID3D10EffectVariable *v; + ID3D10Effect *effect; + ID3D10Device *device; + int int_v[2], int_s; + ULONG refcount; + HRESULT hr; + + if (!(device = create_device())) + { + skip("Failed to create device, skipping tests.\n"); + return; + } + + hr = create_effect(fx_test_default_variable_value, 0, device, NULL, &effect); + ok(SUCCEEDED(hr), "D3D10CreateEffectFromMemory failed (%x)\n", hr); + + memset(float_v, 0, sizeof(float_v)); + v = effect->lpVtbl->GetVariableByName(effect, "f4"); + vector = v->lpVtbl->AsVector(v); + hr = vector->lpVtbl->GetFloatVector(vector, float_v); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(float_v[0] == 1.0f && float_v[1] == 2.0f && float_v[2] == 3.0f && float_v[3] == 4.0f, + "Unexpected vector {%.8e,%.8e,%.8e,%.8e}\n", float_v[0], float_v[1], float_v[2], float_v[3]); + + memset(float_v, 0, sizeof(float_v)); + float_v[2] = float_v[3] = 5.0f; + v = effect->lpVtbl->GetVariableByName(effect, "f2"); + vector = v->lpVtbl->AsVector(v); + hr = vector->lpVtbl->GetFloatVector(vector, float_v); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(float_v[0] == 1.0f && float_v[1] == 2.0f && float_v[2] == 5.0f && float_v[3] == 5.0f, + "Unexpected vector {%.8e,%.8e,%.8e,%.8e}\n", float_v[0], float_v[1], float_v[2], float_v[3]); + + memset(float_v, 0, sizeof(float_v)); + float_v[1] = float_v[2] = float_v[3] = 5.0f; + v = effect->lpVtbl->GetVariableByName(effect, "f1"); + vector = v->lpVtbl->AsVector(v); + hr = vector->lpVtbl->GetFloatVector(vector, float_v); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(float_v[0] == 1.0f && float_v[1] == 5.0f && float_v[2] == 5.0f && float_v[3] == 5.0f, + "Unexpected vector {%.8e,%.8e,%.8e,%.8e}\n", float_v[0], float_v[1], float_v[2], float_v[3]); + + int_s = 0; + v = effect->lpVtbl->GetVariableByName(effect, "i"); + scalar = v->lpVtbl->AsScalar(v); + hr = scalar->lpVtbl->GetInt(scalar, &int_s); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(int_s == 10, "Unexpected value %d.\n", int_s); + + memset(int_v, 0, sizeof(int_v)); + v = effect->lpVtbl->GetVariableByName(effect, "i2"); + scalar = v->lpVtbl->AsScalar(v); + hr = scalar->lpVtbl->GetIntArray(scalar, int_v, 0, 2); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(int_v[0] == 9 && int_v[1] == 12, "Unexpected vector {%d,%d}\n", int_v[0], int_v[1]); + + float_s = 0.0f; + v = effect->lpVtbl->GetVariableByName(effect, "f"); + scalar = v->lpVtbl->AsScalar(v); + hr = scalar->lpVtbl->GetFloat(scalar, &float_s); + ok(hr == S_OK, "Unexpected hr %#x.\n", hr); +todo_wine + ok(float_s == 0.2f, "Unexpected value %.8e.\n", float_s); + + effect->lpVtbl->Release(effect); + + refcount = ID3D10Device_Release(device); + ok(!refcount, "Device has %u references left.\n", refcount); +} + START_TEST(effect) { test_effect_constant_buffer_type(); @@ -6658,4 +6769,5 @@ START_TEST(effect) test_effect_shader_description(); test_effect_shader_object(); test_effect_pool(); + test_effect_default_variable_value(); }

1 0

Nikolay Sivov : d3d10/tests: Add some tests for effect pools.
by Alexandre Julliard 15 Sep '21

15 Sep '21

Module: wine Branch: master Commit: 20952bcd3d28f36a255c79bbd0f9b20c73458924 URL: https://source.winehq.org/git/wine.git/?a=commit;h=20952bcd3d28f36a255c79bb… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Sep 15 09:43:25 2021 +0200 d3d10/tests: Add some tests for effect pools. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/d3d10/tests/effect.c | 371 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 371 insertions(+) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=20952bcd3d28f36a255c…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2021