[PATCH v2] d2d1: Implement ID2D1DeviceContext::CreateImageBrush().

Nikolay Sivov nsivov at codeweavers.com
Fri May 6 14:06:40 CDT 2022



On 5/6/22 19:34, Dmitry Timoshkov wrote:
> +static void STDMETHODCALLTYPE d2d_image_brush_SetImage(ID2D1ImageBrush *iface, ID2D1Image *image)
> +{
> +    struct d2d_brush *brush = impl_from_ID2D1ImageBrush(iface);
> +
> +    TRACE("iface %p, image %p.\n", iface, image);
> +
> +    if (brush->u.image.image)
> +        ID2D1Image_Release(brush->u.image.image);
> +    brush->u.image.image = image;
> +}
This one is probably missing an AddRef. Is it possible to have null 
images here? If not, it could be unconditional here, and on release.



More information about the wine-devel mailing list