<div dir="ltr"><div><div>Hi,<br></div><div><br>I've already submitted a patch with the exact same changes. See <a href="https://source.winehq.org/patches/data/121990">https://source.winehq.org/patches/data/121990</a><br><br></div>Cheers,<br></div>Aaryaman<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 2, 2016 at 1:42 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">Hi,<br>
<br>
I'd thunk CreateTexture1 -> CreateTexture3, CreateTexture2->CreateTexture3 and implement CreateTexture3 via CreateObject + InitFromImage - unless there's some behavior difference in those methods that prevents you from doing this.<br>
<br>
Cheers,<br>
Stefan<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Am 29.04.2016 um 11:25 schrieb Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com">jem456.vasishta@gmail.com</a>>:<br>
<br>
> 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.<br>
> 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.<br>
><br>
> 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.<br>
><br>
> Cheers,<br>
> Aaryaman<br>
><br>
> On Fri, Apr 29, 2016 at 1:06 PM, Stefan Dösinger <<a href="mailto:stefandoesinger@gmail.com">stefandoesinger@gmail.com</a>> wrote:<br>
><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>
> Is there anything wrong with calling Texture3::InitFromImage here, or even calling IDirect3DRM3::CreateTexture? in D3DRM1::CreateTexture and D3DRM2::CreateTexture?<br>
><br>
<br>
</div></div></blockquote></div><br></div>