<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 17, 2016 at 1:04 AM, Stefan Dösinger <span dir="ltr"><<a href="mailto:stefandoesinger@gmail.com" target="_blank">stefandoesinger@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA256<br>
<span class=""><br>
Am 2016-05-13 um 22:21 schrieb Aaryaman Vasishta:<br>
>  static HRESULT WINAPI d3drm_texture1_InitFromFile(IDirect3DRMTexture *iface, const char *filename)<br>
>  {<br>
> -    FIXME("iface %p, filename %s stub!\n", iface, debugstr_a(filename));<br>
> +    struct d3drm_texture *texture = impl_from_IDirect3DRMTexture(iface);<br>
> +    D3DRMIMAGE *image;<br>
> +    HRESULT hr;<br>
><br>
> -    return E_NOTIMPL;<br>
> +    TRACE("iface %p, filename %s.\n", iface, debugstr_a(filename));<br>
> +<br>
> +    if (texture->image)<br>
> +    {<br>
> +        /* d3drm intentionally leaks a reference IDirect3DRM if texture is already initialized. */<br>
> +        IDirect3DRM_AddRef(texture->d3drm);<br>
> +        return D3DRMERR_BADOBJECT;<br>
> +    }<br>
> +    if (FAILED(hr = d3drm_texture_load(texture, filename, FALSE, &image)))<br>
> +        return hr;<br>
> +<br>
> +    hr = IDirect3DRMTexture3_InitFromImage(&texture->IDirect3DRMTexture3_iface, image);<br>
</span>I think you forgot to remove the extra check here :-) . (<a href="https://www.winehq.org/pipermail/wine-devel/2016-May/113212.html" rel="noreferrer" target="_blank">https://www.winehq.org/pipermail/wine-devel/2016-May/113212.html</a>)<br></blockquote><div>Right, my bad. I was a bit sloppy in checking version 1 after making changes to version 3. I'll be extra careful next time :) <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> -    IDirect3DRM_AddRef(texture->d3drm);<br>
> +    if (texture->d3drm)<br>
> +        IDirect3DRM_AddRef(texture->d3drm);<br>
</span>This if check looks like it is an accidental leftover from before you reordered patch 1 too.<br></blockquote><div>Thanks for catching this! I'll remove it in the resend.<br><br></div><div>Cheers,<br></div><div>Aaryaman <br></div></div></div></div>