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

Jacek Caban : gdi32: Use NtGdiExtTextOutW for ExtTextOutW implementation.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: cc7e7002ef8f92f47fe047d598dce057fcc2885b URL: https://source.winehq.org/git/wine.git/?a=commit;h=cc7e7002ef8f92f47fe047d5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 23 10:49:42 2021 +0200 gdi32: Use NtGdiExtTextOutW for ExtTextOutW implementation. 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/enhmfdrv/dc.c | 15 +-------------- dlls/gdi32/enhmfdrv/graphics.c | 41 +++++++++++++++++++++++++---------------- dlls/gdi32/font.c | 6 +++--- dlls/gdi32/gdi_private.h | 4 ++++ dlls/gdi32/gdidc.c | 15 +++++++++++++++ dlls/gdi32/mfdrv/init.c | 2 +- dlls/gdi32/mfdrv/metafiledrv.h | 2 -- dlls/gdi32/mfdrv/text.c | 20 ++++++++++---------- include/ntgdi.h | 2 ++ 9 files changed, 61 insertions(+), 46 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=cc7e7002ef8f92f47fe0…

1 0

Zebediah Figura : wined3d: Separate a helper to update Vulkan CBV bindings.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 6416e39822fc748220f0cd889587b40dfa5e952d URL: https://source.winehq.org/git/wine.git/?a=commit;h=6416e39822fc748220f0cd88… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Jul 22 11:50:40 2021 -0500 wined3d: Separate a helper to update Vulkan CBV bindings. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context_vk.c | 40 ++++++++++++++++++++++++---------------- 1 file changed, 24 insertions(+), 16 deletions(-) diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c index d1a113d1f81..327656593b6 100644 --- a/dlls/wined3d/context_vk.c +++ b/dlls/wined3d/context_vk.c @@ -2650,6 +2650,28 @@ static bool wined3d_shader_resource_bindings_add_null_srv_binding(struct wined3d } } +static bool wined3d_shader_descriptor_writes_vk_add_cbv_write(struct wined3d_shader_descriptor_writes_vk *writes, + struct wined3d_context_vk *context_vk, VkDescriptorSet vk_descriptor_set, const struct wined3d_state *state, + const struct wined3d_shader_resource_binding *binding) +{ + struct wined3d_device_vk *device_vk = wined3d_device_vk(context_vk->c.device); + const VkDescriptorBufferInfo *buffer_info; + struct wined3d_buffer_vk *buffer_vk; + struct wined3d_buffer *buffer; + + if (!(buffer = state->cb[binding->shader_type][binding->resource_idx].buffer)) + return wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, binding->binding_idx, + VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, &device_vk->null_resources_vk.buffer_info, NULL, NULL); + + buffer_vk = wined3d_buffer_vk(buffer); + buffer_info = wined3d_buffer_vk_get_buffer_info(buffer_vk); + if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, + binding->binding_idx, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, buffer_info, NULL, NULL)) + return false; + wined3d_context_vk_reference_bo(context_vk, &buffer_vk->bo); + return true; +} + static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *context_vk, VkCommandBuffer vk_command_buffer, const struct wined3d_state *state, enum wined3d_pipeline pipeline) { @@ -2661,10 +2683,8 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con struct wined3d_unordered_access_view_vk *uav_vk; struct wined3d_shader_resource_view_vk *srv_vk; struct wined3d_unordered_access_view *uav; - const VkDescriptorBufferInfo *buffer_info; struct wined3d_shader_resource_view *srv; const VkDescriptorImageInfo *image_info; - struct wined3d_buffer_vk *buffer_vk; VkDescriptorSetLayout vk_set_layout; VkPipelineLayout vk_pipeline_layout; struct wined3d_resource *resource; @@ -2672,7 +2692,6 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con VkDescriptorSet vk_descriptor_set; struct wined3d_view_vk *view_vk; struct wined3d_sampler *sampler; - struct wined3d_buffer *buffer; VkBufferView *buffer_view; VkDescriptorType type; VkResult vr; @@ -2713,20 +2732,9 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con switch (binding->shader_descriptor_type) { case WINED3D_SHADER_DESCRIPTOR_TYPE_CBV: - if (!(buffer = state->cb[binding->shader_type][binding->resource_idx].buffer)) - { - if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, - binding->binding_idx, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, - &device_vk->null_resources_vk.buffer_info, NULL, NULL)) - return false; - break; - } - buffer_vk = wined3d_buffer_vk(buffer); - buffer_info = wined3d_buffer_vk_get_buffer_info(buffer_vk); - if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, - binding->binding_idx, VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, buffer_info, NULL, NULL)) + if (!wined3d_shader_descriptor_writes_vk_add_cbv_write(writes, + context_vk, vk_descriptor_set, state, binding)) return false; - wined3d_context_vk_reference_bo(context_vk, &buffer_vk->bo); break; case WINED3D_SHADER_DESCRIPTOR_TYPE_SRV:

1 0

Zebediah Figura : wined3d: Separate a helper to update Vulkan SRV bindings.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 5c8046d7b60b9f2c8184bda6fb36e6d784a872ab URL: https://source.winehq.org/git/wine.git/?a=commit;h=5c8046d7b60b9f2c8184bda6… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Jul 22 11:50:41 2021 -0500 wined3d: Separate a helper to update Vulkan SRV bindings. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context_vk.c | 80 +++++++++++++++++++++++++++-------------------- 1 file changed, 46 insertions(+), 34 deletions(-) diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c index 327656593b6..9b114f762c3 100644 --- a/dlls/wined3d/context_vk.c +++ b/dlls/wined3d/context_vk.c @@ -2672,6 +2672,50 @@ static bool wined3d_shader_descriptor_writes_vk_add_cbv_write(struct wined3d_sha return true; } +static bool wined3d_shader_descriptor_writes_vk_add_srv_write(struct wined3d_shader_descriptor_writes_vk *writes, + struct wined3d_context_vk *context_vk, VkDescriptorSet vk_descriptor_set, const struct wined3d_state *state, + const struct wined3d_shader_resource_binding *binding) +{ + struct wined3d_shader_resource_view_vk *srv_vk; + struct wined3d_shader_resource_view *srv; + const VkDescriptorImageInfo *image_info; + struct wined3d_resource *resource; + struct wined3d_view_vk *view_vk; + VkBufferView *buffer_view; + VkDescriptorType type; + + if (!(srv = state->shader_resource_view[binding->shader_type][binding->resource_idx])) + return wined3d_shader_resource_bindings_add_null_srv_binding(writes, vk_descriptor_set, + binding->binding_idx, binding->resource_type, binding->resource_data_type, context_vk); + + resource = srv->resource; + srv_vk = wined3d_shader_resource_view_vk(srv); + view_vk = &srv_vk->view_vk; + if (resource->type == WINED3D_RTYPE_BUFFER) + { + image_info = NULL; + buffer_view = &view_vk->u.vk_buffer_view; + type = VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER; + } + else + { + struct wined3d_texture_vk *texture_vk = wined3d_texture_vk(texture_from_resource(resource)); + + if (view_vk->u.vk_image_info.imageView) + image_info = &view_vk->u.vk_image_info; + else + image_info = wined3d_texture_vk_get_default_image_info(texture_vk, context_vk); + buffer_view = NULL; + type = VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE; + } + + if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, + binding->binding_idx, type, NULL, image_info, buffer_view)) + return false; + wined3d_context_vk_reference_shader_resource_view(context_vk, srv_vk); + return true; +} + static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *context_vk, VkCommandBuffer vk_command_buffer, const struct wined3d_state *state, enum wined3d_pipeline pipeline) { @@ -2681,9 +2725,7 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con const struct wined3d_shader_resource_binding *binding; struct wined3d_shader_resource_bindings *bindings; struct wined3d_unordered_access_view_vk *uav_vk; - struct wined3d_shader_resource_view_vk *srv_vk; struct wined3d_unordered_access_view *uav; - struct wined3d_shader_resource_view *srv; const VkDescriptorImageInfo *image_info; VkDescriptorSetLayout vk_set_layout; VkPipelineLayout vk_pipeline_layout; @@ -2738,39 +2780,9 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con break; case WINED3D_SHADER_DESCRIPTOR_TYPE_SRV: - if (!(srv = state->shader_resource_view[binding->shader_type][binding->resource_idx])) - { - if (!wined3d_shader_resource_bindings_add_null_srv_binding(writes, vk_descriptor_set, - binding->binding_idx, binding->resource_type, binding->resource_data_type, context_vk)) - return false; - break; - } - resource = srv->resource; - - srv_vk = wined3d_shader_resource_view_vk(srv); - view_vk = &srv_vk->view_vk; - if (resource->type == WINED3D_RTYPE_BUFFER) - { - image_info = NULL; - buffer_view = &view_vk->u.vk_buffer_view; - type = VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER; - } - else - { - struct wined3d_texture_vk *texture_vk = wined3d_texture_vk(texture_from_resource(resource)); - - if (view_vk->u.vk_image_info.imageView) - image_info = &view_vk->u.vk_image_info; - else - image_info = wined3d_texture_vk_get_default_image_info(texture_vk, context_vk); - buffer_view = NULL; - type = VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE; - } - - if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, - binding->binding_idx, type, NULL, image_info, buffer_view)) + if (!wined3d_shader_descriptor_writes_vk_add_srv_write(writes, + context_vk, vk_descriptor_set, state, binding)) return false; - wined3d_context_vk_reference_shader_resource_view(context_vk, srv_vk); break; case WINED3D_SHADER_DESCRIPTOR_TYPE_UAV:

1 0

Zebediah Figura : wined3d: Separate a helper to update Vulkan sampler bindings.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 5fe58921bfbee6f1ea12db800ff0534f86129bd3 URL: https://source.winehq.org/git/wine.git/?a=commit;h=5fe58921bfbee6f1ea12db80… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Thu Jul 22 11:50:44 2021 -0500 wined3d: Separate a helper to update Vulkan sampler bindings. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/context_vk.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c index 23c249c682c..0d38b9eb5ea 100644 --- a/dlls/wined3d/context_vk.c +++ b/dlls/wined3d/context_vk.c @@ -2784,6 +2784,21 @@ static bool wined3d_shader_descriptor_writes_vk_add_uav_counter_write( VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER, NULL, NULL, &uav_vk->vk_counter_view); } +static bool wined3d_shader_descriptor_writes_vk_add_sampler_write(struct wined3d_shader_descriptor_writes_vk *writes, + struct wined3d_context_vk *context_vk, VkDescriptorSet vk_descriptor_set, const struct wined3d_state *state, + const struct wined3d_shader_resource_binding *binding) +{ + struct wined3d_sampler *sampler; + + if (!(sampler = state->sampler[binding->shader_type][binding->resource_idx])) + sampler = context_vk->c.device->null_sampler; + if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, binding->binding_idx, + VK_DESCRIPTOR_TYPE_SAMPLER, NULL, &wined3d_sampler_vk(sampler)->vk_image_info, NULL)) + return false; + wined3d_context_vk_reference_sampler(context_vk, wined3d_sampler_vk(sampler)); + return true; +} + static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *context_vk, VkCommandBuffer vk_command_buffer, const struct wined3d_state *state, enum wined3d_pipeline pipeline) { @@ -2796,7 +2811,6 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con VkPipelineLayout vk_pipeline_layout; VkPipelineBindPoint vk_bind_point; VkDescriptorSet vk_descriptor_set; - struct wined3d_sampler *sampler; VkResult vr; size_t i; @@ -2859,12 +2873,9 @@ static bool wined3d_context_vk_update_descriptors(struct wined3d_context_vk *con break; case WINED3D_SHADER_DESCRIPTOR_TYPE_SAMPLER: - if (!(sampler = state->sampler[binding->shader_type][binding->resource_idx])) - sampler = context_vk->c.device->null_sampler; - if (!wined3d_shader_descriptor_writes_vk_add_write(writes, vk_descriptor_set, binding->binding_idx, - VK_DESCRIPTOR_TYPE_SAMPLER, NULL, &wined3d_sampler_vk(sampler)->vk_image_info, NULL)) + if (!wined3d_shader_descriptor_writes_vk_add_sampler_write(writes, + context_vk, vk_descriptor_set, state, binding)) return false; - wined3d_context_vk_reference_sampler(context_vk, wined3d_sampler_vk(sampler)); break; default:

1 0

Jacek Caban : gdi32: Use ntgdi interface for path functions.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 972cfaca1fd58bf399e033ee0b576e789404797a URL: https://source.winehq.org/git/wine.git/?a=commit;h=972cfaca1fd58bf399e033ee… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 23 10:50:37 2021 +0200 gdi32: Use ntgdi interface for path 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/enhmfdrv/dc.c | 27 ++++++++++++--------------- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 3 --- dlls/gdi32/enhmfdrv/init.c | 6 +++--- dlls/gdi32/gdi_private.h | 3 +++ dlls/gdi32/gdidc.c | 36 ++++++++++++++++++++++++++++++++++++ dlls/gdi32/path.c | 29 +++++++++-------------------- dlls/gdi32/tests/metafile.c | 22 ++++++++++++++-------- 7 files changed, 77 insertions(+), 49 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=972cfaca1fd58bf399e0…

1 0

Jacek Caban : gdi32: Use NtGdiCloseFigure for CloseFigure implementation.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 0f3173e7d97bfcd4a189cee1916f1bc1e0fb47b7 URL: https://source.winehq.org/git/wine.git/?a=commit;h=0f3173e7d97bfcd4a189cee1… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 23 10:50:13 2021 +0200 gdi32: Use NtGdiCloseFigure for CloseFigure implementation. 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/enhmfdrv/dc.c | 19 +++---------------- dlls/gdi32/enhmfdrv/enhmetafiledrv.h | 1 - dlls/gdi32/enhmfdrv/init.c | 4 ++-- dlls/gdi32/gdi_private.h | 1 + dlls/gdi32/gdidc.c | 12 ++++++++++++ dlls/gdi32/path.c | 6 ++---- 6 files changed, 20 insertions(+), 23 deletions(-) diff --git a/dlls/gdi32/enhmfdrv/dc.c b/dlls/gdi32/enhmfdrv/dc.c index 7c7c060a53b..855f01970d4 100644 --- a/dlls/gdi32/enhmfdrv/dc.c +++ b/dlls/gdi32/enhmfdrv/dc.c @@ -513,15 +513,14 @@ BOOL CDECL EMFDRV_BeginPath( PHYSDEV dev ) return TRUE; } -BOOL CDECL EMFDRV_CloseFigure( PHYSDEV dev ) +BOOL EMFDC_CloseFigure( DC_ATTR *dc_attr ) { EMRCLOSEFIGURE emr; emr.emr.iType = EMR_CLOSEFIGURE; emr.emr.nSize = sizeof(emr); - EMFDRV_WriteRecord( dev, &emr.emr ); - return FALSE; /* always fails without a path */ + return EMFDRV_WriteRecord( dc_attr->emf, &emr.emr ); } BOOL CDECL EMFDRV_EndPath( PHYSDEV dev ) @@ -611,18 +610,6 @@ static BOOL CDECL emfpathdrv_BeginPath( PHYSDEV dev ) return (emfdev->funcs->pBeginPath( emfdev ) && next->funcs->pBeginPath( next )); } -/*********************************************************************** - * emfpathdrv_CloseFigure - */ -static BOOL CDECL emfpathdrv_CloseFigure( PHYSDEV dev ) -{ - PHYSDEV emfdev = get_emfdev( dev ); - PHYSDEV next = GET_NEXT_PHYSDEV( dev, pCloseFigure ); - - emfdev->funcs->pCloseFigure( emfdev ); - return next->funcs->pCloseFigure( next ); -} - /*********************************************************************** * emfpathdrv_CreateDC */ @@ -670,7 +657,7 @@ static const struct gdi_dc_funcs emfpath_driver = emfpathdrv_BeginPath, /* pBeginPath */ NULL, /* pBlendImage */ NULL, /* pChord */ - emfpathdrv_CloseFigure, /* pCloseFigure */ + NULL, /* pCloseFigure */ NULL, /* pCreateCompatibleDC */ emfpathdrv_CreateDC, /* pCreateDC */ emfpathdrv_DeleteDC, /* pDeleteDC */ diff --git a/dlls/gdi32/enhmfdrv/enhmetafiledrv.h b/dlls/gdi32/enhmfdrv/enhmetafiledrv.h index 128b157c45a..554f55973f6 100644 --- a/dlls/gdi32/enhmfdrv/enhmetafiledrv.h +++ b/dlls/gdi32/enhmfdrv/enhmetafiledrv.h @@ -71,7 +71,6 @@ extern BOOL CDECL EMFDRV_BitBlt( PHYSDEV devDst, INT xDst, INT yDst, INT wid PHYSDEV devSrc, INT xSrc, INT ySrc, DWORD rop ) DECLSPEC_HIDDEN; extern BOOL CDECL EMFDRV_Chord( PHYSDEV dev, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend ) DECLSPEC_HIDDEN; -extern BOOL CDECL EMFDRV_CloseFigure( PHYSDEV dev ) DECLSPEC_HIDDEN; extern BOOL CDECL EMFDRV_DeleteObject( PHYSDEV dev, HGDIOBJ obj ) DECLSPEC_HIDDEN; extern BOOL CDECL EMFDRV_Ellipse( PHYSDEV dev, INT left, INT top, INT right, INT bottom ) DECLSPEC_HIDDEN; extern BOOL CDECL EMFDRV_EndPath( PHYSDEV dev ) DECLSPEC_HIDDEN; diff --git a/dlls/gdi32/enhmfdrv/init.c b/dlls/gdi32/enhmfdrv/init.c index 0aac934a87b..4e7105b1b80 100644 --- a/dlls/gdi32/enhmfdrv/init.c +++ b/dlls/gdi32/enhmfdrv/init.c @@ -44,8 +44,8 @@ static const struct gdi_dc_funcs emfdrv_driver = EMFDRV_ArcTo, /* pArcTo */ EMFDRV_BeginPath, /* pBeginPath */ NULL, /* pBlendImage */ - EMFDRV_Chord, /* pChord */ - EMFDRV_CloseFigure, /* pCloseFigure */ + NULL, /* pChord */ + NULL, /* pCloseFigure */ NULL, /* pCreateCompatibleDC */ NULL, /* pCreateDC */ EMFDRV_DeleteDC, /* pDeleteDC */ diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index ea670732f96..829dacd073c 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -67,6 +67,7 @@ extern BOOL EMFDC_AngleArc( DC_ATTR *dc_attr, INT x, INT y, DWORD radius, FLOAT extern BOOL EMFDC_ArcChordPie( DC_ATTR *dc_attr, INT left, INT top, INT right, INT bottom, INT xstart, INT ystart, INT xend, INT yend, DWORD type ) DECLSPEC_HIDDEN; +extern BOOL EMFDC_CloseFigure( DC_ATTR *dc_attr ) DECLSPEC_HIDDEN; extern BOOL EMFDC_Ellipse( DC_ATTR *dc_attr, INT left, INT top, INT right, INT bottom ) DECLSPEC_HIDDEN; extern BOOL EMFDC_ExtTextOut( DC_ATTR *dc_attr, INT x, INT y, UINT flags, const RECT *rect, diff --git a/dlls/gdi32/gdidc.c b/dlls/gdi32/gdidc.c index 9200803b6f0..1d2ad9766c4 100644 --- a/dlls/gdi32/gdidc.c +++ b/dlls/gdi32/gdidc.c @@ -375,3 +375,15 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, const RECT *rect, return FALSE; return NtGdiExtTextOutW( hdc, x, y, flags, rect, str, count, dx, 0 ); } + +/*********************************************************************** + * CloseFigure (GDI32.@) + */ +BOOL WINAPI CloseFigure( HDC hdc ) +{ + DC_ATTR *dc_attr; + + if (!(dc_attr = get_dc_attr( hdc ))) return FALSE; + if (dc_attr->emf && !EMFDC_CloseFigure( dc_attr )) return FALSE; + return NtGdiCloseFigure( hdc ); +} diff --git a/dlls/gdi32/path.c b/dlls/gdi32/path.c index 3646ba91079..fc9a4d4721f 100644 --- a/dlls/gdi32/path.c +++ b/dlls/gdi32/path.c @@ -617,11 +617,9 @@ BOOL WINAPI AbortPath( HDC hdc ) /*********************************************************************** - * CloseFigure (GDI32.@) - * - * FIXME: Check that SetLastError is being called correctly + * NtGdiCloseFigure (win32u.@) */ -BOOL WINAPI CloseFigure(HDC hdc) +BOOL WINAPI NtGdiCloseFigure( HDC hdc ) { BOOL ret = FALSE; DC *dc = get_dc_ptr( hdc );

1 0

Jacek Caban : gdi32: Store text alignment in DC_ATTR.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: aca66fb50399b862ca91a6a2e6a56a81f8bcdfdb URL: https://source.winehq.org/git/wine.git/?a=commit;h=aca66fb50399b862ca91a6a2… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 23 10:49:19 2021 +0200 gdi32: Store text alignment in DC_ATTR. 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/dc.c | 26 +++++--------------------- dlls/gdi32/enhmfdrv/graphics.c | 2 +- dlls/gdi32/font.c | 2 +- dlls/gdi32/gdidc.c | 9 +++++++++ dlls/gdi32/ntgdi_private.h | 1 - include/ntgdi.h | 1 + 6 files changed, 17 insertions(+), 24 deletions(-) diff --git a/dlls/gdi32/dc.c b/dlls/gdi32/dc.c index 2573390c6f2..deca9510f20 100644 --- a/dlls/gdi32/dc.c +++ b/dlls/gdi32/dc.c @@ -94,7 +94,7 @@ static void set_initial_dc_state( DC *dc ) dc->brush_org.x = 0; dc->brush_org.y = 0; dc->mapperFlags = 0; - dc->textAlign = TA_LEFT | TA_TOP | TA_NOUPDATECP; + dc->attr->text_align = TA_LEFT | TA_TOP | TA_NOUPDATECP; dc->charExtra = 0; dc->breakExtra = 0; dc->breakRem = 0; @@ -409,7 +409,7 @@ INT CDECL nulldrv_SaveDC( PHYSDEV dev ) newdc->dcPenColor = dc->dcPenColor; newdc->brush_org = dc->brush_org; newdc->mapperFlags = dc->mapperFlags; - newdc->textAlign = dc->textAlign; + newdc->attr->text_align = dc->attr->text_align; newdc->charExtra = dc->charExtra; newdc->breakExtra = dc->breakExtra; newdc->breakRem = dc->breakRem; @@ -486,7 +486,7 @@ BOOL CDECL nulldrv_RestoreDC( PHYSDEV dev, INT level ) dc->dcPenColor = dcs->dcPenColor; dc->brush_org = dcs->brush_org; dc->mapperFlags = dcs->mapperFlags; - dc->textAlign = dcs->textAlign; + dc->attr->text_align = dcs->attr->text_align; dc->charExtra = dcs->charExtra; dc->breakExtra = dcs->breakExtra; dc->breakRem = dcs->breakRem; @@ -988,22 +988,6 @@ COLORREF WINAPI SetTextColor( HDC hdc, COLORREF color ) } -/*********************************************************************** - * GetTextAlign (GDI32.@) - */ -UINT WINAPI GetTextAlign( HDC hdc ) -{ - UINT ret = 0; - DC * dc = get_dc_ptr( hdc ); - if (dc) - { - ret = dc->textAlign; - release_dc_ptr( dc ); - } - return ret; -} - - /*********************************************************************** * SetTextAlign (GDI32.@) */ @@ -1020,8 +1004,8 @@ UINT WINAPI SetTextAlign( HDC hdc, UINT align ) align = physdev->funcs->pSetTextAlign( physdev, align ); if (align != GDI_ERROR) { - ret = dc->textAlign; - dc->textAlign = align; + ret = dc->attr->text_align; + dc->attr->text_align = align; } release_dc_ptr( dc ); } diff --git a/dlls/gdi32/enhmfdrv/graphics.c b/dlls/gdi32/enhmfdrv/graphics.c index 7c800148cf8..67def307f1e 100644 --- a/dlls/gdi32/enhmfdrv/graphics.c +++ b/dlls/gdi32/enhmfdrv/graphics.c @@ -936,7 +936,7 @@ BOOL CDECL EMFDRV_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, const RECT BOOL ret; int textHeight = 0; int textWidth = 0; - const UINT textAlign = dc->textAlign; + const UINT textAlign = dc->attr->text_align; const INT graphicsMode = dc->attr->graphics_mode; FLOAT exScale, eyScale; diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index 41d6f9dda69..d89f9182b9b 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -5949,7 +5949,7 @@ BOOL WINAPI ExtTextOutW( HDC hdc, INT x, INT y, UINT flags, if (!dc) return FALSE; if (count > INT_MAX) return FALSE; - align = dc->textAlign; + align = dc->attr->text_align; breakRem = dc->breakRem; layout = dc->layout; diff --git a/dlls/gdi32/gdidc.c b/dlls/gdi32/gdidc.c index c6a8cbd567f..e2161d062b8 100644 --- a/dlls/gdi32/gdidc.c +++ b/dlls/gdi32/gdidc.c @@ -37,6 +37,15 @@ static DC_ATTR *get_dc_attr( HDC hdc ) return dc_attr; } +/*********************************************************************** + * GetTextAlign (GDI32.@) + */ +UINT WINAPI GetTextAlign( HDC hdc ) +{ + DC_ATTR *dc_attr = get_dc_attr( hdc ); + return dc_attr ? dc_attr->text_align : 0; +} + /*********************************************************************** * GetCurrentPositionEx (GDI32.@) */ diff --git a/dlls/gdi32/ntgdi_private.h b/dlls/gdi32/ntgdi_private.h index 4527651bb33..dc36789fdec 100644 --- a/dlls/gdi32/ntgdi_private.h +++ b/dlls/gdi32/ntgdi_private.h @@ -130,7 +130,6 @@ typedef struct tagDC POINT brush_org; DWORD mapperFlags; /* Font mapper flags */ - WORD textAlign; /* Text alignment from SetTextAlign() */ INT charExtra; /* Spacing from SetTextCharacterExtra() */ INT breakExtra; /* breakTotalExtra / breakCount */ INT breakRem; /* breakTotalExtra % breakCount */ diff --git a/include/ntgdi.h b/include/ntgdi.h index dce35a8b0b8..33c5b53f50f 100644 --- a/include/ntgdi.h +++ b/include/ntgdi.h @@ -100,6 +100,7 @@ typedef struct DC_ATTR { POINT cur_pos; INT graphics_mode; + WORD text_align; void *emf; } DC_ATTR;

1 0

Jacek Caban : gdi32: Use NtGdiPolyDraw for PolyDraw implementation.
by Alexandre Julliard July 23, 2021

July 23, 2021

Module: wine Branch: master Commit: 11f09c2d74d0eaeeaba5f93544166f87881b7f40 URL: https://source.winehq.org/git/wine.git/?a=commit;h=11f09c2d74d0eaeeaba5f935… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Fri Jul 23 10:48:48 2021 +0200 gdi32: Use NtGdiPolyDraw for PolyDraw implementation. 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/enhmfdrv/dc.c | 14 +------------- dlls/gdi32/enhmfdrv/graphics.c | 22 ++++++++++++++++------ dlls/gdi32/gdi_private.h | 2 ++ dlls/gdi32/gdidc.c | 14 ++++++++++++++ dlls/gdi32/painting.c | 13 +++++-------- dlls/gdi32/tests/metafile.c | 9 +++++++++ 6 files changed, 47 insertions(+), 27 deletions(-) diff --git a/dlls/gdi32/enhmfdrv/dc.c b/dlls/gdi32/enhmfdrv/dc.c index b2b6bb4866e..7af63eed23b 100644 --- a/dlls/gdi32/enhmfdrv/dc.c +++ b/dlls/gdi32/enhmfdrv/dc.c @@ -671,18 +671,6 @@ static BOOL CDECL emfpathdrv_ExtTextOut( PHYSDEV dev, INT x, INT y, UINT flags, next->funcs->pExtTextOut( next, x, y, flags, rect, str, count, dx )); } -/*********************************************************************** - * emfpathdrv_PolyDraw - */ -static BOOL CDECL emfpathdrv_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, DWORD count ) -{ - PHYSDEV emfdev = get_emfdev( dev ); - PHYSDEV next = GET_NEXT_PHYSDEV( dev, pPolyDraw ); - - return (emfdev->funcs->pPolyDraw( emfdev, pts, types, count ) && - next->funcs->pPolyDraw( next, pts, types, count )); -} - static const struct gdi_dc_funcs emfpath_driver = { @@ -757,7 +745,7 @@ static const struct gdi_dc_funcs emfpath_driver = NULL, /* pPie */ NULL, /* pPolyBezier */ NULL, /* pPolyBezierTo */ - emfpathdrv_PolyDraw, /* pPolyDraw */ + NULL, /* pPolyDraw */ NULL, /* pPolyPolygon */ NULL, /* pPolyPolyline */ NULL, /* pPolylineTo */ diff --git a/dlls/gdi32/enhmfdrv/graphics.c b/dlls/gdi32/enhmfdrv/graphics.c index 80cffdbdebf..7c800148cf8 100644 --- a/dlls/gdi32/enhmfdrv/graphics.c +++ b/dlls/gdi32/enhmfdrv/graphics.c @@ -741,11 +741,11 @@ BOOL CDECL EMFDRV_PolyPolygon( PHYSDEV dev, const POINT* pt, const INT* counts, } /********************************************************************** - * EMFDRV_PolyDraw + * EMFDC_PolyDraw */ -BOOL CDECL EMFDRV_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, DWORD count ) +BOOL EMFDC_PolyDraw( DC_ATTR *dc_attr, const POINT *pts, const BYTE *types, DWORD count ) { - EMFDRV_PDEVICE *physDev = get_emf_physdev( dev ); + EMFDRV_PDEVICE *emf = dc_attr->emf; EMRPOLYDRAW *emr; BOOL ret; BYTE *types_dest; @@ -765,18 +765,28 @@ BOOL CDECL EMFDRV_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, DW memcpy( types_dest, types, count ); if (count & 3) memset( types_dest + count, 0, 4 - (count & 3) ); - if (!physDev->path) + if (!emf->path) get_points_bounds( &emr->rclBounds, pts, count, 0 ); else emr->rclBounds = empty_bounds; - ret = EMFDRV_WriteRecord( dev, &emr->emr ); - if (ret && !physDev->path) EMFDRV_UpdateBBox( dev, &emr->rclBounds ); + ret = EMFDRV_WriteRecord( &emf->dev, &emr->emr ); + if (ret && !emf->path) EMFDRV_UpdateBBox( &emf->dev, &emr->rclBounds ); HeapFree( GetProcessHeap(), 0, emr ); return ret; } +/********************************************************************** + * EMFDRV_PolyDraw + */ +BOOL CDECL EMFDRV_PolyDraw( PHYSDEV dev, const POINT *pts, const BYTE *types, DWORD count ) +{ + /* FIXME: update bounding rect */ + return TRUE; +} + + /********************************************************************** * EMFDRV_ExtFloodFill */ diff --git a/dlls/gdi32/gdi_private.h b/dlls/gdi32/gdi_private.h index fd36712dfbc..f7b04be18dc 100644 --- a/dlls/gdi32/gdi_private.h +++ b/dlls/gdi32/gdi_private.h @@ -71,6 +71,8 @@ extern BOOL EMFDC_LineTo( DC_ATTR *dc_attr, INT x, INT y ) DECLSPEC_HIDDEN; extern BOOL EMFDC_MoveTo( DC_ATTR *dc_attr, INT x, INT y ) DECLSPEC_HIDDEN; extern BOOL EMFDC_PolyBezier( DC_ATTR *dc_attr, const POINT *points, DWORD count ) DECLSPEC_HIDDEN; extern BOOL EMFDC_PolyBezierTo( DC_ATTR *dc_attr, const POINT *points, DWORD count ) DECLSPEC_HIDDEN; +extern BOOL EMFDC_PolyDraw( DC_ATTR *dc_attr, const POINT *points, const BYTE *types, + DWORD count ) DECLSPEC_HIDDEN; extern BOOL EMFDC_PolyPolyline( DC_ATTR *dc_attr, const POINT *points, const DWORD *counts, DWORD polys ) DECLSPEC_HIDDEN; extern BOOL EMFDC_PolyPolygon( DC_ATTR *dc_attr, const POINT *points, const INT *counts, diff --git a/dlls/gdi32/gdidc.c b/dlls/gdi32/gdidc.c index c51892d3a9e..c6a8cbd567f 100644 --- a/dlls/gdi32/gdidc.c +++ b/dlls/gdi32/gdidc.c @@ -337,3 +337,17 @@ BOOL WINAPI PolylineTo( HDC hdc, const POINT *points, DWORD count ) if (dc_attr->emf && !EMFDC_PolylineTo( dc_attr, points, count )) return FALSE; return NtGdiPolyPolyDraw( hdc, points, &count, 1, NtGdiPolylineTo ); } + +/*********************************************************************** + * PolyDraw (GDI32.@) + */ +BOOL WINAPI PolyDraw( HDC hdc, const POINT *points, const BYTE *types, DWORD count ) +{ + DC_ATTR *dc_attr; + + TRACE( "%p, %p, %p, %u\n", hdc, points, types, count ); + + if (!(dc_attr = get_dc_attr( hdc ))) return FALSE; + if (dc_attr->emf && !EMFDC_PolyDraw( dc_attr, points, types, count )) return FALSE; + return NtGdiPolyDraw( hdc, points, types, count ); +} diff --git a/dlls/gdi32/painting.c b/dlls/gdi32/painting.c index 9c7081296df..f9fa18c4de0 100644 --- a/dlls/gdi32/painting.c +++ b/dlls/gdi32/painting.c @@ -699,24 +699,21 @@ BOOL WINAPI NtGdiAngleArc( HDC hdc, INT x, INT y, DWORD dwRadius, FLOAT eStartAn } /*********************************************************************** - * PolyDraw (GDI32.@) + * NtGdiPolyDraw (win32u.@) */ -BOOL WINAPI PolyDraw(HDC hdc, const POINT *lppt, const BYTE *lpbTypes, - DWORD cCount) +BOOL WINAPI NtGdiPolyDraw( HDC hdc, const POINT *points, const BYTE *types, DWORD count ) { DC *dc = get_dc_ptr( hdc ); PHYSDEV physdev; BOOL result; - TRACE( "%p, %p, %p, %u\n", hdc, lppt, lpbTypes, cCount ); - if(!dc) return FALSE; update_dc( dc ); physdev = GET_DC_PHYSDEV( dc, pPolyDraw ); - result = physdev->funcs->pPolyDraw( physdev, lppt, lpbTypes, cCount ); - if (result && cCount) - dc->attr->cur_pos = lppt[cCount - 1]; + result = physdev->funcs->pPolyDraw( physdev, points, types, count ); + if (result && count) + dc->attr->cur_pos = points[count - 1]; release_dc_ptr( dc ); return result; diff --git a/dlls/gdi32/tests/metafile.c b/dlls/gdi32/tests/metafile.c index 4f72079a615..d9d43b80fc4 100644 --- a/dlls/gdi32/tests/metafile.c +++ b/dlls/gdi32/tests/metafile.c @@ -3324,6 +3324,9 @@ static void test_mf_Graphics(void) INT type; BOOL ret; + static const POINT points[] = { {1, 1}, {2, 2} }; + static const BYTE types[] = { PT_MOVETO, PT_LINETO }; + hdcMetafile = CreateMetaFileA(NULL); ok(hdcMetafile != 0, "CreateMetaFileA(NULL) error %d\n", GetLastError()); @@ -3344,6 +3347,12 @@ static void test_mf_Graphics(void) ret = ArcTo(hdcMetafile, 1, 2, 30, 40, 11, 12, 23, 24 ); ok( !ret, "ArcTo succeeded\n" ); + SetLastError(0xdeadbeef); + ret = PolyDraw(hdcMetafile, points, types, ARRAY_SIZE(points)); + ok(!ret, "PolyDraw succeeded\n"); + todo_wine + ok(GetLastError() == 0xdeadbeef, "GetLastError() = %u\n", GetLastError()); + hMetafile = CloseMetaFile(hdcMetafile); ok(hMetafile != 0, "CloseMetaFile error %d\n", GetLastError()); type = GetObjectType(hdcMetafile);

1 0

Daniel Lehman : include: Add _tcsxfrm_l declaration.
by Alexandre Julliard July 22, 2021

July 22, 2021

Module: wine Branch: master Commit: c518a5362b925379b1a79e8323d60e19863effc1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=c518a5362b925379b1a79e83… Author: Daniel Lehman <dlehman(a)esri.com> Date: Thu Jul 22 10:25:50 2021 -0700 include: Add _tcsxfrm_l declaration. Signed-off-by: Daniel Lehman <dlehman(a)esri.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- include/msvcrt/corecrt_wstring.h | 1 + include/msvcrt/string.h | 1 + include/tchar.h | 1 + 3 files changed, 3 insertions(+) diff --git a/include/msvcrt/corecrt_wstring.h b/include/msvcrt/corecrt_wstring.h index 53537185f43..00e7863b5c3 100644 --- a/include/msvcrt/corecrt_wstring.h +++ b/include/msvcrt/corecrt_wstring.h @@ -47,6 +47,7 @@ _ACRTIMP wchar_t* __cdecl _wcsrev(wchar_t*); _ACRTIMP wchar_t* __cdecl _wcsset(wchar_t*,wchar_t); _ACRTIMP wchar_t* __cdecl _wcsupr(wchar_t*); _ACRTIMP errno_t __cdecl _wcsupr_s(wchar_t*, size_t); +_ACRTIMP size_t __cdecl _wcsxfrm_l(wchar_t*,const wchar_t*,size_t,_locale_t); _ACRTIMP wchar_t* __cdecl wcscat(wchar_t*,const wchar_t*); _ACRTIMP errno_t __cdecl wcscat_s(wchar_t*,size_t,const wchar_t*); diff --git a/include/msvcrt/string.h b/include/msvcrt/string.h index 2beb429f036..5e3d51076c6 100644 --- a/include/msvcrt/string.h +++ b/include/msvcrt/string.h @@ -40,6 +40,7 @@ _ACRTIMP char* __cdecl _strrev(char*); _ACRTIMP char* __cdecl _strset(char*,int); _ACRTIMP char* __cdecl _strupr(char*); _ACRTIMP errno_t __cdecl _strupr_s(char *, size_t); +_ACRTIMP size_t __cdecl _strxfrm_l(char*,const char*,size_t,_locale_t); _ACRTIMP errno_t __cdecl memmove_s(void*,size_t,const void*,size_t); _ACRTIMP char* __cdecl strcat(char*,const char*); diff --git a/include/tchar.h b/include/tchar.h index a8e64b4bf6a..1231edc565c 100644 --- a/include/tchar.h +++ b/include/tchar.h @@ -154,6 +154,7 @@ extern "C" { #define _tcsupr WINE_tchar_routine(_strupr, _mbsupr, _wcsupr) #define _tcsupr_s WINE_tchar_routine(_strupr_s, _mbsupr_s, _wcsupr_s) #define _tcsxfrm WINE_tchar_routine(strxfrm, strxfrm, wcsxfrm) +#define _tcsxfrm_l WINE_tchar_routine(_strxfrm_l, _strxfrm_l, _ wcsxfrm_l) #define _tctime WINE_tchar_routine(ctime, ctime, _wctime) #define _tenviron WINE_tchar_routine(_environ, _environ, _wenviron) #define _texecl WINE_tchar_routine(execl, _execl, _wexecl)

1 0

Nikolay Sivov : mfmediaengine: Handle B8G8R8X8 format for d3d11 texture output.
by Alexandre Julliard July 22, 2021

July 22, 2021

Module: wine Branch: master Commit: aeabe991ff8ae69ee9959b908851f8b1148f7cf4 URL: https://source.winehq.org/git/wine.git/?a=commit;h=aeabe991ff8ae69ee9959b90… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Jul 22 17:46:12 2021 +0300 mfmediaengine: Handle B8G8R8X8 format for d3d11 texture output. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mfmediaengine/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/dlls/mfmediaengine/main.c b/dlls/mfmediaengine/main.c index a555d0dad99..98504c5e269 100644 --- a/dlls/mfmediaengine/main.c +++ b/dlls/mfmediaengine/main.c @@ -2018,6 +2018,7 @@ static void media_engine_update_d3d11_frame_surface(ID3D11DeviceContext *context switch (surface_desc.Format) { case DXGI_FORMAT_B8G8R8A8_UNORM: + case DXGI_FORMAT_B8G8R8X8_UNORM: surface_desc.Width *= 4; break; default:

1 0