[PATCH v3 1/3] d2d1: Add stubs for ID2D1EffectContext.

Ziqing Hui zhui at codeweavers.com
Fri Apr 29 05:10:20 CDT 2022


Signed-off-by: Ziqing Hui <zhui at codeweavers.com>
---

v3: * Move CreateEffect() implementation to effect_context in this patch.
    * Add effect_context to d2d_effect in this patch.
    * Remove the factory reference in struct d2d_effect. Use effect_context->device_context->factory instead.
    * Use d2d_device_context in d2d_effect_context.

 dlls/d2d1/d2d1_private.h |  31 ++++-
 dlls/d2d1/device.c       |  18 +--
 dlls/d2d1/effect.c       | 290 ++++++++++++++++++++++++++++++++++++++-
 3 files changed, 320 insertions(+), 19 deletions(-)

diff --git a/dlls/d2d1/d2d1_private.h b/dlls/d2d1/d2d1_private.h
index aa8e8569455..cb9ee6d5b80 100644
--- a/dlls/d2d1/d2d1_private.h
+++ b/dlls/d2d1/d2d1_private.h
@@ -34,6 +34,7 @@
 #include "initguid.h"
 #endif
 #include "dwrite_2.h"
+#include "d2d1effectauthor.h"
 
 enum d2d_brush_type
 {
@@ -567,6 +568,17 @@ struct d2d_device
 
 void d2d_device_init(struct d2d_device *device, ID2D1Factory1 *factory, IDXGIDevice *dxgi_device) DECLSPEC_HIDDEN;
 
+struct d2d_effect_context
+{
+    ID2D1EffectContext ID2D1EffectContext_iface;
+    LONG refcount;
+
+    struct d2d_device_context *device_context;
+};
+
+void d2d_effect_context_init(struct d2d_effect_context *effect_context,
+        struct d2d_device_context *device_context) DECLSPEC_HIDDEN;
+
 struct d2d_effect_info
 {
     const CLSID *clsid;
@@ -583,13 +595,14 @@ struct d2d_effect
 
     const struct d2d_effect_info *info;
 
-    ID2D1Factory *factory;
+    struct d2d_effect_context *effect_context;
     ID2D1Image **inputs;
     size_t inputs_size;
     size_t input_count;
 };
 
-HRESULT d2d_effect_init(struct d2d_effect *effect, ID2D1Factory *factory, const CLSID *effect_id) DECLSPEC_HIDDEN;
+HRESULT d2d_effect_init(struct d2d_effect *effect,
+        struct d2d_effect_context *effect_context, const CLSID *effect_id) DECLSPEC_HIDDEN;
 
 static inline BOOL d2d_array_reserve(void **elements, size_t *capacity, size_t count, size_t size)
 {
@@ -694,6 +707,13 @@ static inline const char *debug_d2d_point_2f(const D2D1_POINT_2F *point)
     return wine_dbg_sprintf("{%.8e, %.8e}", point->x, point->y);
 }
 
+static inline const char *debug_d2d_point_2l(const D2D1_POINT_2L *point)
+{
+    if (!point)
+        return "(null)";
+    return wine_dbg_sprintf("{%ld, %ld}", point->x, point->y);
+}
+
 static inline const char *debug_d2d_rect_f(const D2D1_RECT_F *rect)
 {
     if (!rect)
@@ -701,6 +721,13 @@ static inline const char *debug_d2d_rect_f(const D2D1_RECT_F *rect)
     return wine_dbg_sprintf("(%.8e, %.8e)-(%.8e, %.8e)", rect->left, rect->top, rect->right, rect->bottom);
 }
 
+static inline const char *debug_d2d_rect_l(const D2D1_RECT_L *rect)
+{
+    if (!rect)
+        return "(null)";
+    return wine_dbg_sprintf("(%ld, %ld)-(%ld, %ld)", rect->left, rect->top, rect->right, rect->bottom);
+}
+
 static inline const char *debug_d2d_rounded_rect(const D2D1_ROUNDED_RECT *rounded_rect)
 {
     if (!rounded_rect)
diff --git a/dlls/d2d1/device.c b/dlls/d2d1/device.c
index 13c99458cfa..80badcddaac 100644
--- a/dlls/d2d1/device.c
+++ b/dlls/d2d1/device.c
@@ -1890,25 +1890,19 @@ static HRESULT STDMETHODCALLTYPE d2d_device_context_CreateEffect(ID2D1DeviceCont
         REFCLSID effect_id, ID2D1Effect **effect)
 {
     struct d2d_device_context *context = impl_from_ID2D1DeviceContext(iface);
-    struct d2d_effect *object;
+    struct d2d_effect_context *effect_context;
     HRESULT hr;
 
     FIXME("iface %p, effect_id %s, effect %p stub!\n", iface, debugstr_guid(effect_id), effect);
 
-    if (!(object = heap_alloc_zero(sizeof(*object))))
+    if (!(effect_context = heap_alloc_zero(sizeof(*effect_context))))
         return E_OUTOFMEMORY;
+    d2d_effect_context_init(effect_context, context);
 
-    if (FAILED(hr = d2d_effect_init(object, context->factory, effect_id)))
-    {
-        WARN("Failed to initialise effect, hr %#lx.\n", hr);
-        heap_free(object);
-        return hr;
-    }
+    hr = ID2D1EffectContext_CreateEffect(&effect_context->ID2D1EffectContext_iface, effect_id, effect);
 
-    TRACE("Created effect %p.\n", object);
-    *effect = &object->ID2D1Effect_iface;
-
-    return S_OK;
+    ID2D1EffectContext_Release(&effect_context->ID2D1EffectContext_iface);
+    return hr;
 }
 
 static HRESULT STDMETHODCALLTYPE d2d_device_context_ID2D1DeviceContext_CreateGradientStopCollection(
diff --git a/dlls/d2d1/effect.c b/dlls/d2d1/effect.c
index 350ac3c7bbf..65183b4f0f1 100644
--- a/dlls/d2d1/effect.c
+++ b/dlls/d2d1/effect.c
@@ -17,7 +17,6 @@
  */
 
 #include "d2d1_private.h"
-#include "d2d1effectauthor.h"
 
 WINE_DEFAULT_DEBUG_CHANNEL(d2d);
 
@@ -31,6 +30,286 @@ static const struct d2d_effect_info builtin_effects[] =
     {&CLSID_D2D1Grayscale,              1, 1, 1},
 };
 
+static inline struct d2d_effect_context *impl_from_ID2D1EffectContext(ID2D1EffectContext *iface)
+{
+    return CONTAINING_RECORD(iface, struct d2d_effect_context, ID2D1EffectContext_iface);
+}
+
+static void d2d_effect_context_cleanup(struct d2d_effect_context *effect_context)
+{
+    ID2D1DeviceContext_Release(&effect_context->device_context->ID2D1DeviceContext_iface);
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_QueryInterface(ID2D1EffectContext *iface, REFIID iid, void **out)
+{
+    TRACE("iface %p, iid %s, out %p.\n", iface, debugstr_guid(iid), out);
+
+    if (IsEqualGUID(iid, &IID_ID2D1EffectContext)
+            || IsEqualGUID(iid, &IID_IUnknown))
+    {
+        ID2D1EffectContext_AddRef(iface);
+        *out = iface;
+        return S_OK;
+    }
+
+    WARN("%s not implemented, returning E_NOINTERFACE.\n", debugstr_guid(iid));
+
+    *out = NULL;
+    return E_NOINTERFACE;
+}
+
+static ULONG STDMETHODCALLTYPE d2d_effect_context_AddRef(ID2D1EffectContext *iface)
+{
+    struct d2d_effect_context *effect_context = impl_from_ID2D1EffectContext(iface);
+    ULONG refcount = InterlockedIncrement(&effect_context->refcount);
+
+    TRACE("%p increasing refcount to %lu.\n", iface, refcount);
+
+    return refcount;
+}
+
+static ULONG STDMETHODCALLTYPE d2d_effect_context_Release(ID2D1EffectContext *iface)
+{
+    struct d2d_effect_context *effect_context = impl_from_ID2D1EffectContext(iface);
+    ULONG refcount = InterlockedDecrement(&effect_context->refcount);
+
+    TRACE("%p decreasing refcount to %lu.\n", iface, refcount);
+
+    if (!refcount)
+    {
+        d2d_effect_context_cleanup(effect_context);
+        heap_free(effect_context);
+    }
+
+    return refcount;
+}
+
+static void STDMETHODCALLTYPE d2d_effect_context_GetDpi(ID2D1EffectContext *iface, float *dpi_x, float *dpi_y)
+{
+    FIXME("iface %p, dpi_x %p, dpi_y %p stub!\n", iface, dpi_x, dpi_y);
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateEffect(ID2D1EffectContext *iface,
+        REFCLSID clsid, ID2D1Effect **effect)
+{
+    struct d2d_effect_context *effect_context = impl_from_ID2D1EffectContext(iface);
+    struct d2d_effect *object;
+    HRESULT hr;
+
+    TRACE("iface %p, clsid %s, effect %p.\n", iface, debugstr_guid(clsid), effect);
+
+    if (!(object = heap_alloc_zero(sizeof(*object))))
+        return E_OUTOFMEMORY;
+
+    if (FAILED(hr = d2d_effect_init(object, effect_context, clsid)))
+    {
+        WARN("Failed to initialise effect, hr %#lx.\n", hr);
+        heap_free(object);
+        return hr;
+    }
+
+    TRACE("Created effect %p.\n", object);
+    *effect = &object->ID2D1Effect_iface;
+
+    return S_OK;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_GetMaximumSupportedFeatureLevel(ID2D1EffectContext *iface,
+        const D3D_FEATURE_LEVEL *levels, UINT32 level_count, D3D_FEATURE_LEVEL *max_level)
+{
+    FIXME("iface %p, levels %p, level_count %u, max_level %p stub!\n", iface, levels, level_count, max_level);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateTransformNodeFromEffect(ID2D1EffectContext *iface,
+        ID2D1Effect *effect, ID2D1TransformNode **node)
+{
+    FIXME("iface %p, effect %p, node %p stub!\n", iface, effect, node);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateBlendTransform(ID2D1EffectContext *iface,
+        UINT32 num_inputs, const D2D1_BLEND_DESCRIPTION *description, ID2D1BlendTransform **transform)
+{
+    FIXME("iface %p, num_inputs %u, description %p, transform %p stub!\n", iface, num_inputs, description, transform);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateBorderTransform(ID2D1EffectContext *iface,
+        D2D1_EXTEND_MODE mode_x, D2D1_EXTEND_MODE mode_y, ID2D1BorderTransform **transform)
+{
+    FIXME("iface %p, mode_x %#x, mode_y %#x, transform %p stub!\n", iface, mode_x, mode_y, transform);
+
+    return E_NOTIMPL;
+}
+
+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);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateBoundsAdjustmentTransform(ID2D1EffectContext *iface,
+        const D2D1_RECT_L *output_rect, ID2D1BoundsAdjustmentTransform **transform)
+{
+    FIXME("iface %p, output_rect %s, transform %p stub!\n", iface, debug_d2d_rect_l(output_rect), transform);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_LoadPixelShader(ID2D1EffectContext *iface,
+        REFGUID shader_id, const BYTE *buffer, UINT32 buffer_size)
+{
+    FIXME("iface %p, shader_id %s, buffer %p, buffer_size %u stub!\n",
+            iface, debugstr_guid(shader_id), buffer, buffer_size);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_LoadVertexShader(ID2D1EffectContext *iface,
+        REFGUID shader_id, const BYTE *buffer, UINT32 buffer_size)
+{
+    FIXME("iface %p, shader_id %s, buffer %p, buffer_size %u stub!\n",
+            iface, debugstr_guid(shader_id), buffer, buffer_size);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_LoadComputeShader(ID2D1EffectContext *iface,
+        REFGUID shader_id, const BYTE *buffer, UINT32 buffer_size)
+{
+    FIXME("iface %p, shader_id %s, buffer %p, buffer_size %u stub!\n",
+            iface, debugstr_guid(shader_id), buffer, buffer_size);
+
+    return E_NOTIMPL;
+}
+
+static BOOL STDMETHODCALLTYPE d2d_effect_context_IsShaderLoaded(ID2D1EffectContext *iface, REFGUID shader_id)
+{
+    FIXME("iface %p, shader_id %s stub!\n", iface, debugstr_guid(shader_id));
+
+    return FALSE;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateResourceTexture(ID2D1EffectContext *iface,
+        const GUID *id,  const D2D1_RESOURCE_TEXTURE_PROPERTIES *texture_properties,
+        const BYTE *data, const UINT32 *strides, UINT32 data_size, ID2D1ResourceTexture **texture)
+{
+    FIXME("iface %p, id %s, texture_properties %p, data %p, strides %s, data_size %u, texture %p stub!\n",
+            iface, debugstr_guid(id), texture_properties, data,
+            strides ? wine_dbg_sprintf("%u", *strides) : "(null)", data_size, texture);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_FindResourceTexture(ID2D1EffectContext *iface,
+        const GUID *id, ID2D1ResourceTexture **texture)
+{
+    FIXME("iface %p, id %s, texture %p stub!\n", iface, debugstr_guid(id), texture);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateVertexBuffer(ID2D1EffectContext *iface,
+        const D2D1_VERTEX_BUFFER_PROPERTIES *buffer_properties, const GUID *id,
+        const D2D1_CUSTOM_VERTEX_BUFFER_PROPERTIES *custom_buffer_properties, ID2D1VertexBuffer **buffer)
+{
+    FIXME("iface %p, buffer_properties %p, id %s, custom_buffer_properties %p, buffer %p stub!\n",
+            iface, buffer_properties, debugstr_guid(id), custom_buffer_properties, buffer);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_FindVertexBuffer(ID2D1EffectContext *iface,
+        const GUID *id, ID2D1VertexBuffer **buffer)
+{
+    FIXME("iface %p, id %s, buffer %p stub!\n", iface, debugstr_guid(id), buffer);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateColorContext(ID2D1EffectContext *iface,
+        D2D1_COLOR_SPACE space, const BYTE *profile, UINT32 profile_size, ID2D1ColorContext **color_context)
+{
+    FIXME("iface %p, space %#x, profile %p, profile_size %u, color_context %p stub!\n",
+            iface, space, profile, profile_size, color_context);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateColorContextFromFilename(ID2D1EffectContext *iface,
+        const WCHAR *filename, ID2D1ColorContext **color_context)
+{
+    FIXME("iface %p, filename %s, color_context %p stub!\n", iface, debugstr_w(filename), color_context);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CreateColorContextFromWicColorContext(ID2D1EffectContext *iface,
+        IWICColorContext *wic_color_context, ID2D1ColorContext **color_context)
+{
+    FIXME("iface %p, wic_color_context %p, color_context %p stub!\n", iface, wic_color_context, color_context);
+
+    return E_NOTIMPL;
+}
+
+static HRESULT STDMETHODCALLTYPE d2d_effect_context_CheckFeatureSupport(ID2D1EffectContext *iface,
+        D2D1_FEATURE feature, void *data, UINT32 data_size)
+{
+    FIXME("iface %p, feature %#x, data %p, data_size %u stub!\n", iface, feature, data, data_size);
+
+    return E_NOTIMPL;
+}
+
+static BOOL STDMETHODCALLTYPE d2d_effect_context_IsBufferPrecisionSupported(ID2D1EffectContext *iface,
+        D2D1_BUFFER_PRECISION precision)
+{
+    FIXME("iface %p, precision %#x stub!\n", iface, precision);
+
+    return FALSE;
+}
+
+static const ID2D1EffectContextVtbl d2d_effect_context_vtbl =
+{
+    d2d_effect_context_QueryInterface,
+    d2d_effect_context_AddRef,
+    d2d_effect_context_Release,
+    d2d_effect_context_GetDpi,
+    d2d_effect_context_CreateEffect,
+    d2d_effect_context_GetMaximumSupportedFeatureLevel,
+    d2d_effect_context_CreateTransformNodeFromEffect,
+    d2d_effect_context_CreateBlendTransform,
+    d2d_effect_context_CreateBorderTransform,
+    d2d_effect_context_CreateOffsetTransform,
+    d2d_effect_context_CreateBoundsAdjustmentTransform,
+    d2d_effect_context_LoadPixelShader,
+    d2d_effect_context_LoadVertexShader,
+    d2d_effect_context_LoadComputeShader,
+    d2d_effect_context_IsShaderLoaded,
+    d2d_effect_context_CreateResourceTexture,
+    d2d_effect_context_FindResourceTexture,
+    d2d_effect_context_CreateVertexBuffer,
+    d2d_effect_context_FindVertexBuffer,
+    d2d_effect_context_CreateColorContext,
+    d2d_effect_context_CreateColorContextFromFilename,
+    d2d_effect_context_CreateColorContextFromWicColorContext,
+    d2d_effect_context_CheckFeatureSupport,
+    d2d_effect_context_IsBufferPrecisionSupported,
+};
+
+void d2d_effect_context_init(struct d2d_effect_context *effect_context, struct d2d_device_context *device_context)
+{
+    effect_context->ID2D1EffectContext_iface.lpVtbl = &d2d_effect_context_vtbl;
+    effect_context->refcount = 1;
+    effect_context->device_context = device_context;
+    ID2D1DeviceContext_AddRef(&device_context->ID2D1DeviceContext_iface);
+}
+
 static inline struct d2d_effect *impl_from_ID2D1Effect(ID2D1Effect *iface)
 {
     return CONTAINING_RECORD(iface, struct d2d_effect, ID2D1Effect_iface);
@@ -46,7 +325,7 @@ static void d2d_effect_cleanup(struct d2d_effect *effect)
             ID2D1Image_Release(effect->inputs[i]);
     }
     heap_free(effect->inputs);
-    ID2D1Factory_Release(effect->factory);
+    ID2D1EffectContext_Release(&effect->effect_context->ID2D1EffectContext_iface);
 }
 
 static HRESULT STDMETHODCALLTYPE d2d_effect_QueryInterface(ID2D1Effect *iface, REFIID iid, void **out)
@@ -362,7 +641,7 @@ static void STDMETHODCALLTYPE d2d_effect_image_GetFactory(ID2D1Image *iface, ID2
 
     TRACE("iface %p, factory %p.\n", iface, factory);
 
-    ID2D1Factory_AddRef(*factory = effect->factory);
+    ID2D1Factory_AddRef(*factory = effect->effect_context->device_context->factory);
 }
 
 static const ID2D1ImageVtbl d2d_effect_image_vtbl =
@@ -373,7 +652,7 @@ static const ID2D1ImageVtbl d2d_effect_image_vtbl =
     d2d_effect_image_GetFactory,
 };
 
-HRESULT d2d_effect_init(struct d2d_effect *effect, ID2D1Factory *factory, const CLSID *effect_id)
+HRESULT d2d_effect_init(struct d2d_effect *effect, struct d2d_effect_context *effect_context, const CLSID *effect_id)
 {
     unsigned int i;
 
@@ -387,7 +666,8 @@ HRESULT d2d_effect_init(struct d2d_effect *effect, ID2D1Factory *factory, const
         {
             effect->info = &builtin_effects[i];
             d2d_effect_SetInputCount(&effect->ID2D1Effect_iface, effect->info->default_input_count);
-            ID2D1Factory_AddRef(effect->factory = factory);
+            effect->effect_context = effect_context;
+            ID2D1EffectContext_AddRef(&effect_context->ID2D1EffectContext_iface);
             return S_OK;
         }
     }
-- 
2.25.1




More information about the wine-devel mailing list