Re: [PATCH v2 0/6] MR1682: d3dx9: Partially implement texture filling using texture shaders.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/preshader.c:
>      struct d3dx_parameters_store *parameters)
>
> {
>      unsigned int *p;
> - unsigned int i, j, const_count;
> -
>
> unsigned int i, j, const_count, magic;
> double *dconst;
> HRESULT hr;
> unsigned int saved_word;
> unsigned int section_size;
> -
>
> magic = *ptr;
> -
>
> TRACE("Preshader version %#x.\n", *ptr & 0xffff);

It seems nice to also update the TRACE() to output the full 32-bit value.

Additional nitpick: you can probably drop the "for 2 registers" part of the patch subject.