Re: [PATCH v4 0/2] MR979: d3d9: Implement d3dx_effect_SetRawValue

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/effect.c:
> -
>  0,   0,   1,   0,   0,   0,
> -
>  6,   0,   0,   0,  98, 108,
> -
> 97, 110, 107,   0,   0,   0
>
> +};
> -
>
> +struct test_set_raw_data_value
> +{
> - D3DXPARAMETER_TYPE param_type;
> - unsigned int expected_data_len;
> - unsigned int param_data_len;
> - unsigned int offset;
> - HRESULT expected_result;
> - const void *expected_data;
> - const char *param_name;
> - const void *param_data;
>
> +};

Can you reorder the fields such that the values to set are all at the start and the expected values all at the end? E.g.:

```suggestion:-10+0
struct test_set_raw_data_value
{
    const char *param_name;
    D3DXPARAMETER_TYPE param_type;
    const void *data;
    unsigned int data_size;
    unsigned int offset;
    HRESULT expected_hr;
    const void *expected_data;
    unsigned int expected_data_size;
};
```