Re: wined3d: Implement check for shader creation if hardware supports requested version.

[Henri Verbeet]

2008/12/17 Pauli Nieminen <suokkos(a)gmail.com>:
> +static inline BOOL is_pshader_version_supported(IWineD3DDeviceImpl *deviceImpl, const DWORD *byte_code)
> +{
> +    struct shader_caps shader_caps;
> +    const DWORD shader_version = *byte_code; /* First instruction should be shader version */
> Please don't do that, use the reg_maps.shader_version instead.
> +    ERR("First instruction in shader isn't correct shader version for pixel shader. (0x%x)\n", shader_version);
> ERR is for internal errors, this should be a WARN. You probably don't
> need to print anything at all here though, doing it from the caller
> makes more sense (and you've got a message there already anyway).
> +            ERR("No hardware support for pixel shader version (0x%x).\n",*pFunction);
> Like above, this should be a WARN. That also means you can get rid of
> the code to only print the message once.