Re: [PATCH 0/2] MR1295: Enable compilation with long types in d3dx9_36/tests.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/effect.c:
>  ID3DXEffect *effect;
>  HRESULT hr;
>  D3DXHANDLE param;
> - unsigned int i, passes_count, value;
> - unsigned int i, passes_count;
> - DWORD value;

It's probably better to introduce a union like:
<pre>
union
{
    DWORD d;
    float f;
} value;
</pre>
and then use the appropriate member in the following code as necessary. It might also make sense to split out a separate DWORD variable for the cases where the variable is only used as DWORD (e.g. the GetSamplerState() part).