April 2024 - Wine-commits - list.winehq.org

Rémi Bernon : mfreadwrite/reader: Make the GetTransformForStream category parameter optional.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: b5f7ce42638156769d16a114aa173825cb068be2 URL: https://gitlab.winehq.org/wine/wine/-/commit/b5f7ce42638156769d16a114aa1738… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Mar 27 14:58:49 2024 +0100 mfreadwrite/reader: Make the GetTransformForStream category parameter optional. --- dlls/mfreadwrite/reader.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/dlls/mfreadwrite/reader.c b/dlls/mfreadwrite/reader.c index 2baa3810ee0..252e0c1f51c 100644 --- a/dlls/mfreadwrite/reader.c +++ b/dlls/mfreadwrite/reader.c @@ -2512,6 +2512,9 @@ static HRESULT WINAPI src_reader_GetTransformForStream(IMFSourceReaderEx *iface, TRACE("%p, %#lx, %#lx, %p, %p.\n", iface, stream_index, transform_index, category, transform); + if (!transform) + return E_POINTER; + EnterCriticalSection(&reader->cs); if (stream_index == MF_SOURCE_READER_FIRST_VIDEO_STREAM) @@ -2525,7 +2528,8 @@ static HRESULT WINAPI src_reader_GetTransformForStream(IMFSourceReaderEx *iface, hr = MF_E_INVALIDINDEX; else { - *category = entry->category; + if (category) + *category = entry->category; *transform = entry->transform; IMFTransform_AddRef(*transform); hr = S_OK;

1 0

Rémi Bernon : mfreadwrite/tests: Test the source reader stream change events.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: ed5031ebd8315cadf13b58d2250e0f882c8c8e88 URL: https://gitlab.winehq.org/wine/wine/-/commit/ed5031ebd8315cadf13b58d2250e0f… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Mon Mar 25 19:40:04 2024 +0100 mfreadwrite/tests: Test the source reader stream change events. --- dlls/mfreadwrite/tests/mfplat.c | 524 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 524 insertions(+)

1 0

Rémi Bernon : mfreadwrite/tests: Test the D3D awareness of source reader transforms.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: 5cd5e7b8c6c07179c2a698ef175c20cfd9ea4087 URL: https://gitlab.winehq.org/wine/wine/-/commit/5cd5e7b8c6c07179c2a698ef175c20… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Mar 27 15:00:51 2024 +0100 mfreadwrite/tests: Test the D3D awareness of source reader transforms. --- dlls/mfreadwrite/tests/Makefile.in | 2 +- dlls/mfreadwrite/tests/mfplat.c | 387 ++++++++++++++++++++++++++++++++++++- 2 files changed, 387 insertions(+), 2 deletions(-)

1 0

Rémi Bernon : mfreadwrite/reader: Avoid accessing an invalid stream index.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: e3510c6bdb85273fb186d97377b5cb46644c36c0 URL: https://gitlab.winehq.org/wine/wine/-/commit/e3510c6bdb85273fb186d97377b5cb… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Wed Mar 27 11:02:24 2024 +0100 mfreadwrite/reader: Avoid accessing an invalid stream index. Fixes d6c9ac94d2fc92d86639e74699ef470b1d8a9861. --- dlls/mfreadwrite/reader.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/dlls/mfreadwrite/reader.c b/dlls/mfreadwrite/reader.c index 252e0c1f51c..873eac959f1 100644 --- a/dlls/mfreadwrite/reader.c +++ b/dlls/mfreadwrite/reader.c @@ -2365,7 +2365,6 @@ static HRESULT WINAPI src_reader_GetServiceForStream(IMFSourceReaderEx *iface, D REFIID riid, void **object) { struct source_reader *reader = impl_from_IMFSourceReaderEx(iface); - struct media_stream *stream = &reader->streams[index]; IUnknown *obj = NULL; HRESULT hr = S_OK; @@ -2386,7 +2385,7 @@ static HRESULT WINAPI src_reader_GetServiceForStream(IMFSourceReaderEx *iface, D if (index >= reader->stream_count) hr = MF_E_INVALIDSTREAMNUMBER; - else if (!(obj = (IUnknown *)stream->transform_service)) + else if (!(obj = (IUnknown *)reader->streams[index].transform_service)) hr = E_NOINTERFACE; break; }

1 0

Nikolay Sivov : d2d1/tests: Add a basic test for ID2D1DrawInfo instance.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: 6ecd59ca39b10eab63f50d32d8c4f80b2d5e2d86 URL: https://gitlab.winehq.org/wine/wine/-/commit/6ecd59ca39b10eab63f50d32d8c4f8… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Mon Apr 1 23:55:40 2024 +0200 d2d1/tests: Add a basic test for ID2D1DrawInfo instance. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/tests/d2d1.c | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index 301cbdca51f..862a08ae467 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -34,6 +34,28 @@ DEFINE_GUID(CLSID_TestEffect, 0xb9ee12e9,0x32d9,0xe659,0xac,0x61,0x2d,0x7c,0xea, DEFINE_GUID(GUID_TestVertexShader, 0x5bcdcfae,0x1e92,0x4dc1,0x94,0xfa,0x3b,0x01,0xca,0x54,0x59,0x20); DEFINE_GUID(GUID_TestPixelShader, 0x53015748,0xfc13,0x4168,0xbd,0x13,0x0f,0xcf,0x15,0x29,0x7f,0x01); +static ULONG get_refcount(void *iface) +{ + IUnknown *unknown = iface; + IUnknown_AddRef(unknown); + return IUnknown_Release(unknown); +} + +#define check_interface(a, b, c) check_interface_(__LINE__, a, b, c) +static void check_interface_(unsigned int line, void *iface_ptr, REFIID iid, BOOL supported) +{ + IUnknown *iface = iface_ptr; + HRESULT hr, expected_hr; + IUnknown *unk; + + expected_hr = supported ? S_OK : E_NOINTERFACE; + + hr = IUnknown_QueryInterface(iface, iid, (void **)&unk); + ok_(__FILE__, line)(hr == expected_hr, "Got hr %#lx, expected %#lx.\n", hr, expected_hr); + if (SUCCEEDED(hr)) + IUnknown_Release(unk); +} + static const WCHAR *effect_xml_a = L"<?xml version='1.0'?> \ <Effect> \ @@ -14365,6 +14387,15 @@ static HRESULT STDMETHODCALLTYPE effect_impl_draw_transform_MapInvalidRect( static HRESULT STDMETHODCALLTYPE effect_impl_draw_transform_SetDrawInfo(ID2D1DrawTransform *iface, ID2D1DrawInfo *info) { + ULONG refcount; + + check_interface(info, &IID_IUnknown, TRUE); + check_interface(info, &IID_ID2D1RenderInfo, TRUE); + check_interface(info, &IID_ID2D1EffectContext, FALSE); + + refcount = get_refcount(info); + ok(refcount == 1, "Unexpected refcount %lu.\n", refcount); + return ID2D1DrawInfo_SetPixelShader(info, &GUID_TestPixelShader, 0); }

1 0

Nikolay Sivov : d2d1/tests: Add a test for transform graph input count.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: 55f80c4ea48574dc59dfa8c7a7a9ec312bb4d3a7 URL: https://gitlab.winehq.org/wine/wine/-/commit/55f80c4ea48574dc59dfa8c7a7a9ec… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Apr 2 09:16:15 2024 +0200 d2d1/tests: Add a test for transform graph input count. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/tests/d2d1.c | 29 ++++++++++++++++++++++++++++- 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index 862a08ae467..b6db1e81178 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -267,6 +267,9 @@ L"<?xml version='1.0'?> \ <Input name='Source2'/> \ <Input name='Source3'/> \ </Inputs> \ + <Property name='Graph' type='iunknown'> \ + <Property name='DisplayName' type='string' value='Graph'/> \ + </Property> \ </Effect> \ "; @@ -11284,10 +11287,12 @@ static void test_effect_register(BOOL d3d11) ID2D1DeviceContext *device_context; ID2D1EffectContext *effect_context; struct d2d1_test_context ctx; + ID2D1TransformGraph *graph; unsigned int i, integer; ID2D1Factory1 *factory; WCHAR display_name[64]; ID2D1Effect *effect; + UINT32 count; HRESULT hr; const struct xml_test @@ -11320,6 +11325,11 @@ static void test_effect_register(BOOL d3d11) {L"DeadBeef", effect_impl_set_integer, effect_impl_get_integer}, }; + static const D2D1_PROPERTY_BINDING bindings2[] = + { + {L"Graph", NULL, effect_impl_get_graph}, + }; + const struct binding_test { const D2D1_PROPERTY_BINDING *binding; @@ -11502,13 +11512,20 @@ static void test_effect_register(BOOL d3d11) /* Variable input count. */ hr = ID2D1Factory1_RegisterEffectFromString(factory, &CLSID_TestEffect, - effect_variable_input_count, NULL, 0, effect_impl_create); + effect_variable_input_count, bindings2, ARRAY_SIZE(bindings2), effect_impl_create); ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1DeviceContext_CreateEffect(device_context, &CLSID_TestEffect, &effect); ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); integer = ID2D1Effect_GetInputCount(effect); ok(integer == 3, "Unexpected input count %u.\n", integer); + hr = ID2D1Effect_GetValueByName(effect, L"Graph", D2D1_PROPERTY_TYPE_IUNKNOWN, (BYTE *)&graph, sizeof(graph)); + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + + count = ID2D1TransformGraph_GetInputCount(graph); + todo_wine + ok(count == 3, "Unexpected input count %u.\n", count); + integer = 0; hr = ID2D1Effect_GetValue(effect, D2D1_PROPERTY_MIN_INPUTS, D2D1_PROPERTY_TYPE_UINT32, (BYTE *)&integer, sizeof(integer)); @@ -11529,6 +11546,12 @@ static void test_effect_register(BOOL d3d11) ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); ok(integer == 3, "Unexpected data %u.\n", integer); + hr = ID2D1Effect_GetValueByName(effect, L"Graph", D2D1_PROPERTY_TYPE_IUNKNOWN, (BYTE *)&graph, sizeof(graph)); + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + count = ID2D1TransformGraph_GetInputCount(graph); + todo_wine + ok(count == 4, "Unexpected input count %u.\n", count); + ID2D1Effect_Release(effect); hr = ID2D1Factory1_UnregisterEffect(factory, &CLSID_TestEffect); @@ -12696,6 +12719,10 @@ static void test_transform_graph(BOOL d3d11) D2D1_PROPERTY_TYPE_IUNKNOWN, (BYTE *)&effect_context, sizeof(effect_context)); ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + count = ID2D1TransformGraph_GetInputCount(graph); + todo_wine + ok(count == 1, "Unexpected input count %u.\n", count); + /* Create transforms */ hr = ID2D1EffectContext_CreateOffsetTransform(effect_context, point, &offset_transform); todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);

1 0

Nikolay Sivov : d2d1/effect: Keep input count as a graph property when recreating it.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: a9926aebae5343052afaf5ca943bea601d10b19d URL: https://gitlab.winehq.org/wine/wine/-/commit/a9926aebae5343052afaf5ca943bea… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Apr 2 09:35:57 2024 +0200 d2d1/effect: Keep input count as a graph property when recreating it. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/d2d1_private.h | 2 ++ dlls/d2d1/effect.c | 51 +++++++++++++++++++++++++++--------------------- dlls/d2d1/tests/d2d1.c | 3 --- 3 files changed, 31 insertions(+), 25 deletions(-) diff --git a/dlls/d2d1/d2d1_private.h b/dlls/d2d1/d2d1_private.h index dced9847390..9c314e08694 100644 --- a/dlls/d2d1/d2d1_private.h +++ b/dlls/d2d1/d2d1_private.h @@ -712,6 +712,8 @@ struct d2d_transform_graph { ID2D1TransformGraph ID2D1TransformGraph_iface; LONG refcount; + + UINT32 input_count; }; struct d2d_effect diff --git a/dlls/d2d1/effect.c b/dlls/d2d1/effect.c index 50c4b01de12..3a3bc896fe9 100644 --- a/dlls/d2d1/effect.c +++ b/dlls/d2d1/effect.c @@ -66,9 +66,11 @@ static ULONG STDMETHODCALLTYPE d2d_transform_graph_Release(ID2D1TransformGraph * static UINT32 STDMETHODCALLTYPE d2d_transform_graph_GetInputCount(ID2D1TransformGraph *iface) { - FIXME("iface %p stub!\n", iface); + struct d2d_transform_graph *graph = impl_from_ID2D1TransformGraph(iface); + + TRACE("iface %p.\n", iface); - return 0; + return graph->input_count; } static HRESULT STDMETHODCALLTYPE d2d_transform_graph_SetSingleTransformNode(ID2D1TransformGraph *iface, @@ -144,10 +146,20 @@ static const ID2D1TransformGraphVtbl d2d_transform_graph_vtbl = d2d_transform_graph_SetPassthroughGraph, }; -static void d2d_transform_graph_init(struct d2d_transform_graph *graph) +static HRESULT d2d_transform_graph_create(UINT32 input_count, struct d2d_transform_graph **graph) { - graph->ID2D1TransformGraph_iface.lpVtbl = &d2d_transform_graph_vtbl; - graph->refcount = 1; + struct d2d_transform_graph *object; + + if (!(object = calloc(1, sizeof(*object)))) + return E_OUTOFMEMORY; + + object->ID2D1TransformGraph_iface.lpVtbl = &d2d_transform_graph_vtbl; + object->refcount = 1; + object->input_count = input_count; + + *graph = object; + + return S_OK; } static HRESULT STDMETHODCALLTYPE d2d_effect_impl_QueryInterface(ID2D1EffectImpl *iface, REFIID iid, void **out) @@ -1273,12 +1285,11 @@ static HRESULT d2d_effect_set_input_count(struct d2d_effect *effect, UINT32 coun ID2D1TransformGraph_Release(&effect->graph->ID2D1TransformGraph_iface); effect->graph = NULL; - if (!(effect->graph = calloc(1, sizeof(*effect->graph)))) - return E_OUTOFMEMORY; - d2d_transform_graph_init(effect->graph); - - if (FAILED(hr = ID2D1EffectImpl_SetGraph(effect->impl, &effect->graph->ID2D1TransformGraph_iface))) - WARN("Failed to set a new transform graph, hr %#lx.\n", hr); + if (SUCCEEDED(hr = d2d_transform_graph_create(count, &effect->graph))) + { + if (FAILED(hr = ID2D1EffectImpl_SetGraph(effect->impl, &effect->graph->ID2D1TransformGraph_iface))) + WARN("Failed to set a new transform graph, hr %#lx.\n", hr); + } } return hr; @@ -1626,7 +1637,6 @@ HRESULT d2d_effect_create(struct d2d_device_context *context, const CLSID *effec { struct d2d_effect_context *effect_context; const struct d2d_effect_registration *reg; - struct d2d_transform_graph *graph; struct d2d_effect *object; UINT32 input_count; WCHAR clsidW[39]; @@ -1642,16 +1652,8 @@ HRESULT d2d_effect_create(struct d2d_device_context *context, const CLSID *effec return E_OUTOFMEMORY; d2d_effect_context_init(effect_context, context); - if (!(graph = calloc(1, sizeof(*graph)))) - { - ID2D1EffectContext_Release(&effect_context->ID2D1EffectContext_iface); - return E_OUTOFMEMORY; - } - d2d_transform_graph_init(graph); - if (!(object = calloc(1, sizeof(*object)))) { - ID2D1TransformGraph_Release(&graph->ID2D1TransformGraph_iface); ID2D1EffectContext_Release(&effect_context->ID2D1EffectContext_iface); return E_OUTOFMEMORY; } @@ -1660,7 +1662,6 @@ HRESULT d2d_effect_create(struct d2d_device_context *context, const CLSID *effec object->ID2D1Image_iface.lpVtbl = &d2d_effect_image_vtbl; object->refcount = 1; object->effect_context = effect_context; - object->graph = graph; /* Create properties */ d2d_effect_duplicate_properties(&object->properties, &reg->properties); @@ -1675,6 +1676,12 @@ HRESULT d2d_effect_create(struct d2d_device_context *context, const CLSID *effec d2d_effect_get_value(object, D2D1_PROPERTY_INPUTS, D2D1_PROPERTY_TYPE_ARRAY, (BYTE *)&input_count, sizeof(input_count)); d2d_effect_set_input_count(object, input_count); + if (FAILED(hr = d2d_transform_graph_create(input_count, &object->graph))) + { + ID2D1EffectContext_Release(&effect_context->ID2D1EffectContext_iface); + return hr; + } + if (FAILED(hr = reg->factory((IUnknown **)&object->impl))) { WARN("Failed to create implementation object, hr %#lx.\n", hr); @@ -1683,7 +1690,7 @@ HRESULT d2d_effect_create(struct d2d_device_context *context, const CLSID *effec } if (FAILED(hr = ID2D1EffectImpl_Initialize(object->impl, &effect_context->ID2D1EffectContext_iface, - &graph->ID2D1TransformGraph_iface))) + &object->graph->ID2D1TransformGraph_iface))) { WARN("Failed to initialize effect, hr %#lx.\n", hr); ID2D1Effect_Release(&object->ID2D1Effect_iface); diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index b6db1e81178..05ed2d985e5 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -11523,7 +11523,6 @@ static void test_effect_register(BOOL d3d11) ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); count = ID2D1TransformGraph_GetInputCount(graph); - todo_wine ok(count == 3, "Unexpected input count %u.\n", count); integer = 0; @@ -11549,7 +11548,6 @@ static void test_effect_register(BOOL d3d11) hr = ID2D1Effect_GetValueByName(effect, L"Graph", D2D1_PROPERTY_TYPE_IUNKNOWN, (BYTE *)&graph, sizeof(graph)); ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); count = ID2D1TransformGraph_GetInputCount(graph); - todo_wine ok(count == 4, "Unexpected input count %u.\n", count); ID2D1Effect_Release(effect); @@ -12720,7 +12718,6 @@ static void test_transform_graph(BOOL d3d11) ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); count = ID2D1TransformGraph_GetInputCount(graph); - todo_wine ok(count == 1, "Unexpected input count %u.\n", count); /* Create transforms */

1 0

Nikolay Sivov : d2d1/effect: Implement offset transform object.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: 5908d7faad5a654922eca7d789a370c119f8d676 URL: https://gitlab.winehq.org/wine/wine/-/commit/5908d7faad5a654922eca7d789a370… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Apr 2 10:30:51 2024 +0200 d2d1/effect: Implement offset transform object. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/d2d1_private.h | 11 +++++ dlls/d2d1/effect.c | 105 ++++++++++++++++++++++++++++++++++++++++++++++- dlls/d2d1/tests/d2d1.c | 10 ++--- 3 files changed, 117 insertions(+), 9 deletions(-) diff --git a/dlls/d2d1/d2d1_private.h b/dlls/d2d1/d2d1_private.h index 9c314e08694..1ea0e1fdbd3 100644 --- a/dlls/d2d1/d2d1_private.h +++ b/dlls/d2d1/d2d1_private.h @@ -708,6 +708,17 @@ void d2d_factory_register_effect(struct d2d_factory *factory, HRESULT d2d_effect_property_get_uint32_value(const struct d2d_effect_properties *properties, const struct d2d_effect_property *prop, UINT32 *value); +struct d2d_transform +{ + ID2D1TransformNode ID2D1TransformNode_iface; + LONG refcount; + + union + { + D2D1_POINT_2L offset; + }; +}; + struct d2d_transform_graph { ID2D1TransformGraph ID2D1TransformGraph_iface; diff --git a/dlls/d2d1/effect.c b/dlls/d2d1/effect.c index 3a3bc896fe9..0c9c027aed0 100644 --- a/dlls/d2d1/effect.c +++ b/dlls/d2d1/effect.c @@ -20,6 +20,107 @@ WINE_DEFAULT_DEBUG_CHANNEL(d2d); +static inline struct d2d_transform *impl_from_ID2D1OffsetTransform(ID2D1OffsetTransform *iface) +{ + return CONTAINING_RECORD(iface, struct d2d_transform, ID2D1TransformNode_iface); +} + +static HRESULT STDMETHODCALLTYPE d2d_offset_transform_QueryInterface(ID2D1OffsetTransform *iface, + REFIID iid, void **out) +{ + TRACE("iface %p, iid %s, out %p.\n", iface, debugstr_guid(iid), out); + + if (IsEqualGUID(iid, &IID_ID2D1OffsetTransform) + || IsEqualGUID(iid, &IID_ID2D1TransformNode) + || IsEqualGUID(iid, &IID_IUnknown)) + { + *out = iface; + ID2D1OffsetTransform_AddRef(iface); + return S_OK; + } + + WARN("Unsupported interface %s.\n", debugstr_guid(iid)); + *out = NULL; + return E_NOINTERFACE; +} + +static ULONG STDMETHODCALLTYPE d2d_offset_transform_AddRef(ID2D1OffsetTransform *iface) +{ + struct d2d_transform *transform = impl_from_ID2D1OffsetTransform(iface); + ULONG refcount = InterlockedIncrement(&transform->refcount); + + TRACE("%p increasing refcount to %lu.\n", iface, refcount); + + return refcount; +} + +static ULONG STDMETHODCALLTYPE d2d_offset_transform_Release(ID2D1OffsetTransform *iface) +{ + struct d2d_transform *transform = impl_from_ID2D1OffsetTransform(iface); + ULONG refcount = InterlockedDecrement(&transform->refcount); + + TRACE("%p decreasing refcount to %lu.\n", iface, refcount); + + if (!refcount) + free(transform); + + return refcount; +} + +static UINT32 STDMETHODCALLTYPE d2d_offset_transform_GetInputCount(ID2D1OffsetTransform *iface) +{ + TRACE("iface %p.\n", iface); + + return 1; +} + +static void STDMETHODCALLTYPE d2d_offset_transform_SetOffset(ID2D1OffsetTransform *iface, + D2D1_POINT_2L offset) +{ + struct d2d_transform *transform = impl_from_ID2D1OffsetTransform(iface); + + TRACE("iface %p, offset %s.\n", iface, debug_d2d_point_2l(&offset)); + + transform->offset = offset; +} + +static D2D1_POINT_2L * STDMETHODCALLTYPE d2d_offset_transform_GetOffset(ID2D1OffsetTransform *iface, + D2D1_POINT_2L *offset) +{ + struct d2d_transform *transform = impl_from_ID2D1OffsetTransform(iface); + + TRACE("iface %p.\n", iface); + + *offset = transform->offset; + return offset; +} + +static const ID2D1OffsetTransformVtbl d2d_offset_transform_vtbl = +{ + d2d_offset_transform_QueryInterface, + d2d_offset_transform_AddRef, + d2d_offset_transform_Release, + d2d_offset_transform_GetInputCount, + d2d_offset_transform_SetOffset, + d2d_offset_transform_GetOffset, +}; + +static HRESULT d2d_offset_transform_create(D2D1_POINT_2L offset, ID2D1OffsetTransform **transform) +{ + struct d2d_transform *object; + + if (!(object = calloc(1, sizeof(*object)))) + return E_OUTOFMEMORY; + + object->ID2D1TransformNode_iface.lpVtbl = (ID2D1TransformNodeVtbl *)&d2d_offset_transform_vtbl; + object->refcount = 1; + object->offset = offset; + + *transform = (ID2D1OffsetTransform *)&object->ID2D1TransformNode_iface; + + return S_OK; +} + static inline struct d2d_transform_graph *impl_from_ID2D1TransformGraph(ID2D1TransformGraph *iface) { return CONTAINING_RECORD(iface, struct d2d_transform_graph, ID2D1TransformGraph_iface); @@ -802,9 +903,9 @@ static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateBorderTransform(ID2D1E static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateOffsetTransform(ID2D1EffectContext *iface, D2D1_POINT_2L offset, ID2D1OffsetTransform **transform) { - FIXME("iface %p, offset %s, transform %p stub!\n", iface, debug_d2d_point_2l(&offset), transform); + TRACE("iface %p, offset %s, transform %p.\n", iface, debug_d2d_point_2l(&offset), transform); - return E_NOTIMPL; + return d2d_offset_transform_create(offset, transform); } static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateBoundsAdjustmentTransform(ID2D1EffectContext *iface, diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index 05ed2d985e5..4fb9893fd14 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -12722,7 +12722,7 @@ static void test_transform_graph(BOOL d3d11) /* Create transforms */ hr = ID2D1EffectContext_CreateOffsetTransform(effect_context, point, &offset_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1EffectContext_CreateBlendTransform(effect_context, 2, &blend_desc, &blend_transform); todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); if (!offset_transform || !blend_transform) @@ -12836,9 +12836,7 @@ static void test_offset_transform(BOOL d3d11) offset.x = 1; offset.y = 2; hr = ID2D1EffectContext_CreateOffsetTransform(effect_context, offset, &transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); - if (hr != S_OK) - goto done; + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); offset = ID2D1OffsetTransform_GetOffset(transform); ok(offset.x == 1 && offset.y == 2, "Got unexpected offset {%ld, %ld}.\n", offset.x, offset.y); @@ -12853,9 +12851,7 @@ static void test_offset_transform(BOOL d3d11) offset = ID2D1OffsetTransform_GetOffset(transform); ok(offset.x == -10 && offset.y == 20, "Got unexpected offset {%ld, %ld}.\n", offset.x, offset.y); -done: - if (transform) - ID2D1OffsetTransform_Release(transform); + ID2D1OffsetTransform_Release(transform); ID2D1Effect_Release(effect); hr = ID2D1Factory1_UnregisterEffect(factory, &CLSID_TestEffect); ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr);

1 0

Nikolay Sivov : d2d1/effect: Implement blend transform object.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: 1707550daeebe4c2667b6040f4d45864f880354d URL: https://gitlab.winehq.org/wine/wine/-/commit/1707550daeebe4c2667b6040f4d458… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Apr 2 12:04:32 2024 +0200 d2d1/effect: Implement blend transform object. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/d2d1_private.h | 3 ++ dlls/d2d1/effect.c | 130 ++++++++++++++++++++++++++++++++++++++++++++++- dlls/d2d1/tests/d2d1.c | 41 +++++++++------ 3 files changed, 157 insertions(+), 17 deletions(-)

1 0

Nikolay Sivov : d2d1/effect: Implement node list for the transform graph.
by Alexandre Julliard 02 Apr '24

02 Apr '24

Module: wine Branch: master Commit: fb4548b432ea07b93e62ab04945bceb9ef890cf0 URL: https://gitlab.winehq.org/wine/wine/-/commit/fb4548b432ea07b93e62ab04945bce… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Apr 2 12:29:31 2024 +0200 d2d1/effect: Implement node list for the transform graph. Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d2d1/d2d1_private.h | 8 +++++ dlls/d2d1/effect.c | 84 ++++++++++++++++++++++++++++++++++++++++++++---- dlls/d2d1/tests/d2d1.c | 10 ------ 3 files changed, 85 insertions(+), 17 deletions(-) diff --git a/dlls/d2d1/d2d1_private.h b/dlls/d2d1/d2d1_private.h index 177b4149cbf..1f5093e29a4 100644 --- a/dlls/d2d1/d2d1_private.h +++ b/dlls/d2d1/d2d1_private.h @@ -722,12 +722,20 @@ struct d2d_transform UINT32 input_count; }; +struct d2d_transform_node +{ + struct list entry; + ID2D1TransformNode *object; +}; + struct d2d_transform_graph { ID2D1TransformGraph ID2D1TransformGraph_iface; LONG refcount; UINT32 input_count; + + struct list nodes; }; struct d2d_effect diff --git a/dlls/d2d1/effect.c b/dlls/d2d1/effect.c index 3fb48db8bb0..f441abde2b9 100644 --- a/dlls/d2d1/effect.c +++ b/dlls/d2d1/effect.c @@ -247,6 +247,54 @@ static HRESULT d2d_blend_transform_create(UINT32 input_count, const D2D1_BLEND_D return S_OK; } +static struct d2d_transform_node * d2d_transform_graph_get_node(const struct d2d_transform_graph *graph, + ID2D1TransformNode *object) +{ + struct d2d_transform_node *node; + + LIST_FOR_EACH_ENTRY(node, &graph->nodes, struct d2d_transform_node, entry) + { + if (node->object == object) + return node; + } + + return NULL; +} + +static HRESULT d2d_transform_graph_add_node(struct d2d_transform_graph *graph, + ID2D1TransformNode *object) +{ + struct d2d_transform_node *node; + + if (!(node = calloc(1, sizeof(*node)))) + return E_OUTOFMEMORY; + + node->object = object; + ID2D1TransformNode_AddRef(node->object); + list_add_tail(&graph->nodes, &node->entry); + + return S_OK; +} + +static void d2d_transform_graph_delete_node(struct d2d_transform_graph *graph, + struct d2d_transform_node *node) +{ + list_remove(&node->entry); + ID2D1TransformNode_Release(node->object); + + free(node); +} + +static void d2d_transform_graph_clear(struct d2d_transform_graph *graph) +{ + struct d2d_transform_node *node, *node_next; + + LIST_FOR_EACH_ENTRY_SAFE(node, node_next, &graph->nodes, struct d2d_transform_node, entry) + { + d2d_transform_graph_delete_node(graph, node); + } +} + static inline struct d2d_transform_graph *impl_from_ID2D1TransformGraph(ID2D1TransformGraph *iface) { return CONTAINING_RECORD(iface, struct d2d_transform_graph, ID2D1TransformGraph_iface); @@ -286,7 +334,10 @@ static ULONG STDMETHODCALLTYPE d2d_transform_graph_Release(ID2D1TransformGraph * TRACE("%p decreasing refcount to %lu.\n", iface, refcount); if (!refcount) + { + d2d_transform_graph_clear(graph); free(graph); + } return refcount; } @@ -308,18 +359,32 @@ static HRESULT STDMETHODCALLTYPE d2d_transform_graph_SetSingleTransformNode(ID2D return E_NOTIMPL; } -static HRESULT STDMETHODCALLTYPE d2d_transform_graph_AddNode(ID2D1TransformGraph *iface, ID2D1TransformNode *node) +static HRESULT STDMETHODCALLTYPE d2d_transform_graph_AddNode(ID2D1TransformGraph *iface, + ID2D1TransformNode *object) { - FIXME("iface %p, node %p stub!\n", iface, node); + struct d2d_transform_graph *graph = impl_from_ID2D1TransformGraph(iface); - return E_NOTIMPL; + TRACE("iface %p, object %p.\n", iface, object); + + if (d2d_transform_graph_get_node(graph, object)) + return E_INVALIDARG; + + return d2d_transform_graph_add_node(graph, object); } -static HRESULT STDMETHODCALLTYPE d2d_transform_graph_RemoveNode(ID2D1TransformGraph *iface, ID2D1TransformNode *node) +static HRESULT STDMETHODCALLTYPE d2d_transform_graph_RemoveNode(ID2D1TransformGraph *iface, + ID2D1TransformNode *object) { - FIXME("iface %p, node %p stub!\n", iface, node); + struct d2d_transform_graph *graph = impl_from_ID2D1TransformGraph(iface); + struct d2d_transform_node *node; - return E_NOTIMPL; + TRACE("iface %p, object %p.\n", iface, object); + + if (!(node = d2d_transform_graph_get_node(graph, object))) + return HRESULT_FROM_WIN32(ERROR_NOT_FOUND); + + d2d_transform_graph_delete_node(graph, node); + return S_OK; } static HRESULT STDMETHODCALLTYPE d2d_transform_graph_SetOutputNode(ID2D1TransformGraph *iface, ID2D1TransformNode *node) @@ -347,7 +412,11 @@ static HRESULT STDMETHODCALLTYPE d2d_transform_graph_ConnectToEffectInput(ID2D1T static void STDMETHODCALLTYPE d2d_transform_graph_Clear(ID2D1TransformGraph *iface) { - FIXME("iface %p stub!\n", iface); + struct d2d_transform_graph *graph = impl_from_ID2D1TransformGraph(iface); + + TRACE("iface %p.\n", iface); + + d2d_transform_graph_clear(graph); } static HRESULT STDMETHODCALLTYPE d2d_transform_graph_SetPassthroughGraph(ID2D1TransformGraph *iface, UINT32 index) @@ -383,6 +452,7 @@ static HRESULT d2d_transform_graph_create(UINT32 input_count, struct d2d_transfo object->ID2D1TransformGraph_iface.lpVtbl = &d2d_transform_graph_vtbl; object->refcount = 1; object->input_count = input_count; + list_init(&object->nodes); *graph = object; diff --git a/dlls/d2d1/tests/d2d1.c b/dlls/d2d1/tests/d2d1.c index b26e5d6fe3c..15d3b8b82ee 100644 --- a/dlls/d2d1/tests/d2d1.c +++ b/dlls/d2d1/tests/d2d1.c @@ -12728,24 +12728,18 @@ static void test_transform_graph(BOOL d3d11) /* Add nodes */ hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == E_INVALIDARG, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)blend_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); /* Remove nodes */ hr = ID2D1TransformGraph_RemoveNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_RemoveNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == HRESULT_FROM_WIN32(ERROR_NOT_FOUND), "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_RemoveNode(graph, (ID2D1TransformNode *)blend_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); /* Connect nodes which are both un-added */ @@ -12757,7 +12751,6 @@ static void test_transform_graph(BOOL d3d11) /* Connect added node to un-added node */ hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_ConnectNode(graph, (ID2D1TransformNode *)offset_transform, (ID2D1TransformNode *)blend_transform, 0); @@ -12767,7 +12760,6 @@ static void test_transform_graph(BOOL d3d11) /* Connect un-added node to added node */ ID2D1TransformGraph_Clear(graph); hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)blend_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_ConnectNode(graph, (ID2D1TransformNode *)offset_transform, (ID2D1TransformNode *)blend_transform, 0); @@ -12777,10 +12769,8 @@ static void test_transform_graph(BOOL d3d11) /* Connect nodes */ ID2D1TransformGraph_Clear(graph); hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)offset_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); hr = ID2D1TransformGraph_AddNode(graph, (ID2D1TransformNode *)blend_transform); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); count = ID2D1BlendTransform_GetInputCount(blend_transform); for (i = 0; i < count; ++i)

1 0