<div dir="ltr">With this patch gamestudio venice demo worked fine except the water which is, as expected, not rendered correctly.<div>For the other games i don't know. Could be other different problems as the games go further. </div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-03-07 0:58 GMT+01:00 Zebediah Figura <span dir="ltr"><<a href="mailto:z.figura12@gmail.com" target="_blank">z.figura12@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 06/03/18 17:53, Zebediah Figura wrote:<br>
> On 06/03/18 15:53, Matteo Bruni wrote:<br>
>> 2018-03-04 10:31 GMT-06:00 Zebediah Figura <<a href="mailto:z.figura12@gmail.com">z.figura12@gmail.com</a>>:<br>
>><br>
>>> diff --git a/dlls/d3dx9_36/texture.c b/dlls/d3dx9_36/texture.c<br>
>>> index b81f3a1..8d8a70e 100644<br>
>>> --- a/dlls/d3dx9_36/texture.c<br>
>>> +++ b/dlls/d3dx9_36/texture.c<br>
>>> @@ -1919,3 +1919,12 @@ HRESULT WINAPI D3DXSaveTextureToFileInMemory(<wbr>ID3DXBuffer **dst_buffer, D3DXIMAGE<br>
>>><br>
>>>      return hr;<br>
>>>  }<br>
>>> +<br>
>>> +HRESULT WINAPI D3DXComputeNormalMap(<wbr>IDirect3DTexture9 *texture, IDirect3DTexture9 *src_texture,<br>
>>> +        const PALETTEENTRY *src_palette, DWORD flags, DWORD channel, FLOAT amplitude)<br>
>>> +{<br>
>>> +    FIXME("texture %p, src_texture %p, src_palette %p, flags %#x, channel %u, amplitude %f stub.\n",<br>
>>> +            texture, src_texture, src_palette, flags, channel, amplitude);<br>
>>> +<br>
>>> +    return D3D_OK;<br>
>>> +}<br>
>><br>
>> I guess this is for bug 38352? Do you know if it actually helps any of<br>
>> those games? Is the D3D_OK return necessary?<br>
>><br>
>> I'm a bit uncomfortable with the patch at the moment. Ideally I'd like<br>
>> an actual implementation with tests (or even just some tests),<br>
>> although that is maybe asking too much?<br>
>><br>
>><br>
><br>
> Looking at the associated bug report, it seems that the stub probably<br>
> isn't enough, so yes, this probably shouldn't go upstream in this form.<br>
><br>
> I only submitted it because it looked like an easy target for<br>
> upstreaming, but perhaps removal from staging would be better, as it<br>
> fairly clearly isn't adding anything.<br>
><br>
><br>
<br>
</div></div>Apologies, missed Christian's reply. I suppose the decision is up to<br>
Matteo, then, although I'm inclined to agree.<br>
<br>
<br>
</blockquote></div><br></div>