Use sf::Shader::Bind() correctly after latest update of the SFML's API.

[SylvainBoilard]

Hi,

In latests revisions of SFML, to bind a shader, one now needs to call the static method sf::Shader::bind() giving it a pointer to the desired shader as a parameter, and unbinding is done by passing NULL to this method. It's also not necessary to check for the shader to be non-NULL before calling the method anymore, except if you want to avoid unnecessary GL calls at all when shader is NULL.

Attached patch allows to use the latest version of SFML and removes the non-NULL tests before calling sf::Shader::bind().

Bye,

Sylvain

[SylvainBoilard]

Hi,

So I see a bit of activity happened two days ago but this pull request hasn’t been merged yet. Incidentally the game still won’t build against the official release of SFML 2.0 .

Is there something I should modify in this patch to get it merged ?

[bigfatbird]

Well, the game in my setup only compiles with the fix commited by @DeathlyDeep.

[rekado]

I confirm that the patch by @DeathlyDeep does indeed fix the compilation issues on Fedora 19 with the released version of SFML 2.0.

[Mailaender]

👍 I can confirm that this fixes #13 on @openSUSE 13.1

[carstene1ns]

We are using a git build of SFML 2.1 on Arch Linux, a user of the m.a.r.s. AUR package created a similar patch some time ago to allow building. It seems like the patch here is better, though.

[eXpl0it3r]

Thanks for the fix! 🙂