[PATCH 2/2] d3dx9/line: Implemented tests for ID3DXLine's Begin and End methods.

Matteo Bruni matteo.mystral at gmail.com
Mon Jun 20 08:20:03 CDT 2011


2011/6/18 Charles Welton <charleswfb at gmail.com>:
> ---
>  dlls/d3dx9_36/tests/line.c |   23 +++++++++++++++++++++++
>  1 files changed, 23 insertions(+), 0 deletions(-)
>
> +    srand(time(NULL));
> +    random = ((FLOAT) rand())/RAND_MAX; /* Random value between 0.0 and 1.0. We're using this later. */

I didn't check why your patch failed with the testbot. Nevertheless,
you should avoid to use random values: the tests should be repeatable.
Just pick some arbitrary values and use them. Also, for better
clarity, you may want to use a different variable for the modified
identity matrix (because, at that point, it isn't identity anymore).
Maybe you could check the world matrix between ID3DXLine_Begin and
ID3DXLine_End too.



More information about the wine-devel mailing list