[v9 5/6] d3dx9: Implement array selectors in effect.
Matteo Bruni
matteo.mystral at gmail.com
Wed Apr 6 08:44:32 CDT 2016
2016-04-01 13:21 GMT+02:00 Paul Gofman <gofmanp at 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.
More information about the wine-devel
mailing list