Re: [v2 1/2] d3dx9: Add 'exp' preshader opcode.

[Matteo Bruni]

2016-04-25 10:52 GMT+02:00 Paul Gofman <gofmanp(a)gmail.com>:
> +    op_step = 2 + (args_count + 1) * 3;

You could instead calculate that in the caller and pass it in place of
args_count. Or, probably better, you could pass the entire op_tests[]
table entry to the function, you're currently passing all the fields
one by one anyway.
> +    for (i = 0; i < 4; ++i)
> +        todo_wine_if(todo && todo[i])
> +        ok(compare_float(v[i], ((float *)expected_result)[i], ulps),
> +                "Operation %s, component %u, expected %#x (%g), got %#x (%g).\n", op_mnem,
> +                i, expected_result[i], ((float *)expected_result)[i], ((unsigned int *)v)[i], v[i]);

todo currently can never be NULL.
> +    for (i = 0; i < ARRAY_SIZE(op_tests); ++i)
> +        test_preshader_op(device, test_effect_preshader_effect_blob, sizeof test_effect_preshader_effect_blob,
> +                blob_position[op_tests[i].args_count].pos, blob_position[op_tests[i].args_count].result_index,
> +                op_tests[i].mnem, op_tests[i].opcode, op_tests[i].args_count,
> +                op_tests[i].expected_result, &op_tests[i].fvect1, &op_tests[i].fvect2,
> +                op_tests[i].ulps, op_tests[i].todo);

Parenthesis around sizeof operand, please.