Re: [PATCH v3 0/6] MR11674: d3dx10: Some fixes and tests for sprites.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx10_43/sprite.c:
> return E_FAIL;
>
> +    if (!d3dx_array_reserve((void **)&sprite->buffer.sprites, &sprite->buffer.capacity,
> +            sprite->buffer.count + count, sizeof(*sprite->buffer.sprites)))
> +    {
> +        return E_OUTOFMEMORY;
> +    }
> +
> +    for (unsigned int i = 0; i < count; ++i)
> +    {
> +        sprite->buffer.sprites[sprite->buffer.count++] = sprites[i];
> +        if (sprite->flags & D3DX10_SPRITE_ADDREF_TEXTURES)
> +            ID3D10ShaderResourceView_AddRef(sprites[i].pTexture);
> +    }
> +
>      return E_NOTIMPL;

Does it make sense to `return S_OK;` now, or is it too early?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/11674#note_149359