[3/8] WineD3D: Get rid of a version check

Henri Verbeet hverbeet at gmail.com
Thu Aug 6 02:34:47 CDT 2009


2009/8/5 Stefan Dösinger <stefan at codeweavers.com>:
> +void fixup_caps(WINED3DCAPS *pWineCaps)
> +{
> +    /* D3D8 doesn't support SM 2.0 or higher, so clamp to 1.x */
> +    if(pWineCaps->PixelShaderVersion > D3DPS_VERSION(1,4)){
> +        pWineCaps->PixelShaderVersion = D3DPS_VERSION(1,4);
> +    }
> +    if(pWineCaps->VertexShaderVersion > D3DVS_VERSION(1,1)){
> +        pWineCaps->VertexShaderVersion = D3DVS_VERSION(1,1);
> +    }
> +    pWineCaps->MaxVertexShaderConst = min(D3D8_MAX_VERTEX_SHADER_CONSTANTF, pWineCaps->MaxVertexShaderConst);
> +
> +    pWineCaps->StencilCaps &= ~WINED3DSTENCILCAPS_TWOSIDED;
> +}

Please use proper white space and naming conventions. I could live
with the missing space between the control statement and the brace, if
you really prefer that, but at least put *some* sort of whitespace
between the closing brace and the accolade. And "pWineCaps" is just
horrible.



More information about the wine-devel mailing list