d3dx9_36: Add tests for D3DXSHRotateZ
Rico Schüller
kgbricola at web.de
Mon Oct 22 02:06:45 CDT 2012
On 19.10.2012 13:15, Nozomi Kodama wrote:
> This patch adds tests for the patch sent by Rico for D3DXSHRotateZ.
>
> Moreover, I changed const in CONST in the declaration of the function in
> order to uniformize with the file.
> + const FLOAT angle[] = ...
I'd prefer "const" because it is used a little bit more often than
CONST, but it is only a matter of taste.
git grep "[ (,]const " | wc -l
~61k
git grep "[ (,]CONST " | wc -l
~1.2k
> + if ( i >= square || ( (order >= D3DXSH_MAXORDER) && ( i >= D3DXSH_MAXORDER * D3DXSH_MAXORDER ) ) )
> + if (l > 0)
I'd write it "if (i >= square || ((... for consistency.
> + ok(relative_error(expected, out_temp[i]) < admitted_error, "angle %f, order %u index %u, Expected %f, received %f\n", angle[j], order, i, expected, out_temp[i]);
Expected -> expected? (2 times)
> + expected = table[36 * (l + 3 * j ) + i];
The spacing looks a bit ugly.
Cheers
Rico
More information about the wine-devel
mailing list