wined3d: Enable clip planes with GLSL shader backend
Stefan Dösinger
stefan at codeweavers.com
Thu Nov 1 06:54:31 CDT 2007
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
More information about the wine-devel
mailing list