Update Picking to use builtin shader

mosra / magnum-examples

Examples for the Magnum C++11 graphics engine

https://magnum.graphics/

The Unlicense

282 stars 95 forks source link

Update Picking to use builtin shader #81

Closed pomeroyb closed 4 years ago

pomeroyb commented 4 years ago

The old shader did not work on intel machines (caused all objects to be shaded pure black)

mosra commented 4 years ago

Thank you! Coincidentally I just made the same change locally and here it seems an autoformatter made a mess out of the diff, so as a tradeoff I'll commit my version with your name on it ;)

mosra commented 4 years ago

In master as 04c83b860caf66cd4a354b1c519e1a500df4115b (yeah, the CIs take ages to run through).

And according to https://github.com/mosra/magnum-examples/graphs/contributors, you're now on the fourth place when it comes to the amount of deleted code :fire: