[PATCH] d3dx9: implement effect state management.

[Henri Verbeet]

>> +static void test_effect_states(IDirect3DDevice9 *device)
>> +{
>> +    D3DMATRIX test_mat =
>> +    {
>> +        {
>> +        -1.0f, 0.0f, 0.0f, 0.0f,
>> +        0.0f, 0.0f, 0.0f, 0.0f,
>> +        0.0f, 0.0f, 0.0f, 0.0f,
>> +        0.0f, 0.0f, 0.0f, 0.0f
>> +        }
>> +    };
>> +    D3DMATRIX mat;
>> +    HRESULT hr;
>> +    ID3DXEffect *effect;
>> +    UINT cPasses;
>
> No camelcase or similar, just lowercase with underscores.
>
Somewhat related, "This" really isn't a great variable name.
Unfortunately it's fairly common in Wine source, I suppose it's
something people picked up from C++ and then propagated.