Re: [v9 5/6] d3dx9: Implement array selectors in effect.

[Matteo Bruni]

2016-04-01 13:21 GMT+02:00 Paul Gofman <gofmanp(a)gmail.com>:
> case ST_ARRAY_SELECTOR:
> -            FIXME("Array selector.\n");
> -            break;
> +        {
> +            struct d3dx_parameter array_idx_par;

I'd call that array_idx_param, for consistency. Also you can probably
declare it static const and initialize it just once.
> +    if (FAILED(hr = d3dx9_get_param_value_ptr(effect, pass, state, ¶m_value, ¶m)))
> +        /* Native d3dx returns OK from BeginPass or Commit involving out of bound array
> +            access, and does not touch affected state. */

Nitpicks: please spell D3D_OK in full, boundS, drop the comma. Usually
we put a '*' at the start of following comment lines for multiline
comments, aligned with the '*' from the first line.