[PATCH] wined3d: Dont just print a FIXME on error, handle it (Coverity)

Henri Verbeet hverbeet at gmail.com
Sat Jun 11 10:53:38 CDT 2011


On 11 June 2011 17:26, Marcus Meissner <marcus at jet.franken.de> wrote:
>     if (state <= HIGHEST_TRANSFORMSTATE)
> +    {
>         mat = &device->updateStateBlock->state.transforms[state];
> +    }
>     else
> +    {
>         FIXME("Unhandled transform state %#x.\n", state);
> +        return WINED3D_OK;
> +    }
>
This works, so I'm not too bothered either way, but I think the
following looks slightly nicer:

    if (state > HIGHEST_TRANSFORMSTATE)
    {
        WARN("Invalid transform state %#x.\n", state);
        return WINED3D_OK;
    }

    mat = &device->updateStateBlock->state.transforms[state];
    multiply_matrix(&temp, mat, matrix);



More information about the wine-devel mailing list