[PATCH] Enable GLSL by default on OpenGL 3+

[fabio]

Newer graphics cards should be able to properly manage GLSL, which can give better graphics not available without GLSL (windy tree, water texture, ...). This is not enabled by default anywhere, however. In r13678 GLSL was enabled, but later reverted in r13700.

The attached patch enable it by default on graphics card supporting OpenGL 3+, which should be able to properly manage GLSL (GLSL itself should be able to run with OpenGL 2+, but this patch won't force it since it may have bugs). This may eventually be tweaked if testing reveal problems (e.g. only enable on OpenGL 4+, force disable on Intel, ...).

Note for testers: remove any reference to preferglsl and gentangents in you local configuration dir before testing the patch, since local preferences have precedence.

When reviewed and applied I'll submit a patch enabling by default also advanced graphics options (e.g. materialmgr.quality = 10.0).

[fabio]

enable GLSL with OpenGL3+

[Stan]

Refs: #1877, #2596

[wraitii]

Not to complain, really, I believe this is a good idea (and we have been way too cautious about this), but why don't you remove the "preferGLSL/GenTangents" parameters and simply use a "OpenGL Renderer" option? It would at last remove that particular ugliness.

BTW I changed the default settings the other day to quality 2.0, which already looks a lot better than 0.0

[Stan]

Maybe because of #2506 ? GenTangents and Enable GLSL are supposed to be separate. Now I would rather have a very low, low, medium, high, very high, ultra settings dropbox.

[wraitii]

No, GenTangents is basically useless on its own and should be merged.

[Stan]

I guess that's a good thing to do first then.

[fabio]

I added a patch for #2506. I will update this once #2506 is settled and merged.

[fabio]

enable GLSL with OpenGL3+ - rebased after #2506

[fabio]

In 17338:

Enable GLSL by default on OpenGL 3+.

This doesn't override any user setting, it just changes the default when no preferglsl setting is specified.

Fixes #3641.
Refs #145.

[fabio]

Review by wraitii on IRC.