d3dx9_36: Implementation of D3DXSHRotateZ
Matteo Bruni
matteo.mystral at gmail.com
Wed Jul 18 07:17:05 CDT 2012
2012/7/18 Nozomi Kodama <nozomi.kodama at yahoo.com>:
>
Minor nitpicks, but nevertheless:
> -@ stub D3DXSHRotateZ(ptr long long ptr)
> +@ stdcall D3DXSHRotateZ(ptr long long ptr)
...
> +FLOAT* WINAPI D3DXSHRotateZ(FLOAT *out, UINT order, FLOAT angle, CONST FLOAT *in)
I guess using "float" in the spec file for the third parameter would
be more appropriate (yes, it was already wrong but that's not an
excuse ;)
+ if ( i >= order * order || ( (order >
D3DXSH_MAXORDER) && ( i >= D3DXSH_MAXORDER * D3DXSH_MAXORDER ) ) )
+ expected = ( i + 1.0f ) * ( i + 1.0f );
Bad indentation.
In general the inner for loop in the tests feels awkward, try to
rewrite it to avoid the duplicated stuff.
+ }
+ }
+ }
You're adding a tab character here, please use only whitespaces for indentation.
That said, thanks for your work on d3dx9!
More information about the wine-devel
mailing list