wined3d: Enable clip planes with GLSL shader backend

[Stefan Dösinger]

Am Donnerstag, 1. November 2007 02:01:14 schrieb Ivan Gyurdiev:
> > +    if (use_vs(stateblock->wineD3DDevice) &&
> > stateblock->wineD3DDevice->vs_selected_mode == SHADER_ARB)
>
> GLSL specific logic should go in glsl_shader.c.
>
> Please try to get away from writing code in terms of flags and if
> statements - use the OOP model.
> Backend-specific logic should be removed from pixelshader and
> vertexshader too...
We've had this discussion over and over, but I'm not too fond of pressing 
everything into an OOP model by force. I agree it is useful in general, and 
I'm currently working on making the surface type organisation more OOPish. 
However, in this special case the state management collides with shader 
differences. Adding a shader model callback for every of those cases doesn't 
make the code more readable IMHO.

I agree that the backend specific code in GenerateShader in vertexshader.c / 
pixelshader.c should be moved to arb / glsl backends. Then we'd get vertex 
and pixel shader specifics into arb / glsl, but we can't avoid those anyway.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20071101/74bb9e80/attachment.pgp