Re: [PATCH 0/6] MR6360: d3dx9: Beginning of pixel format handling changes.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/surface.c:
> #include <stdint.h>
>  #include "d3dx9_test_images.h"
>
> +static inline BOOL compare_float(float expected, float got, float allowed_error)
> +{
> - return fabs(expected - got) <= allowed_error;
>
> +}
> -

It would be preferable to check for relative error, like e.g. in d3dx9_36/tests/{effect.c,math.c} or d3d9/tests/visual.c, if that works out alright.