<div dir="ltr">Not really, I thought that since InitFromImage doesn't exist for version 1, I wasn't sure if it's a good idea to use it within CreateTexture for version 1.<div>Is it fine, though?. I'll use InitFromImage if it's a better approach. Besides, since they behave the same so far, there's no reason to not use it anyways.<div><br><div>I agree with thunking version 2's CreateTexture to version 3, I'll resend the patch for the same, after I get to know your opinion for the above question.</div><div><br><div>Cheers,</div><div>Aaryaman</div></div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 29, 2016 at 1:06 PM, 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"><span class=""><br>
> Am 28.04.2016 um 20:17 schrieb Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>>:<br>
><br>
> @@ -198,13 +198,24 @@ static HRESULT WINAPI d3drm1_CreateTexture(IDirect3DRM *iface,<br>
>     struct d3drm_texture *object;<br>
>     HRESULT hr;<br>
><br>
> -    FIXME("iface %p, image %p, texture %p partial stub.\n", iface, image, texture);<br>
> +    TRACE("iface %p, image %p, texture %p.\n", iface, image, texture);<br>
> +<br>
> +    if (!d3drm_validate_image(image))<br>
> +        return D3DRMERR_BADVALUE;<br>
><br>
>     if (FAILED(hr = d3drm_texture_create(&object)))<br>
> +    {<br>
> +        d3drm_texture_destroy(object);<br>
>         return hr;<br>
> +    }<br>
> +    object->d3drm = iface;<br>
> +    object->initialized = TRUE;<br>
> +    object->image = image;<br>
><br>
>     *texture = &object->IDirect3DRMTexture_iface;<br>
><br>
> +    IDirect3DRM_AddRef(iface);<br>
> +<br>
>     return D3DRM_OK;<br>
> }<br>
><br>
</span>Is there anything wrong with calling Texture3::InitFromImage here, or even calling IDirect3DRM3::CreateTexture? in D3DRM1::CreateTexture and D3DRM2::CreateTexture?<br>
</blockquote></div><br></div>