search

erich666 / GraphicsGems

Code for the "Graphics Gems" book series
Other
1.39k stars 265 forks source link

Add Graphics Gems I to CMakeLists.txt and fix all errors and warnings. #12

Closed gonsolo closed 8 years ago

erich666 commented 8 years ago

Should I wait a few days until the code settles down?

gonsolo commented 8 years ago

Should I wait a few days until the code settles down?

I just used one file but it is kind of addictive when you know there is something unfinished. Honestly, I don't know, I might finish it or not...

g

erich666 commented 8 years ago

Go for it - I know what you mean. No rush from my end, but I'd rather any fixes accumulate into one pull, so I'll grab this pull request in a few days (or more, if you wish).

gonsolo commented 8 years ago

Go for it - I know what you mean. No rush from my end, but I'd rather any fixes accumulate into one pull, so I'll grab this pull request in a few days (or more, if you wish).

Ok. Done. I added all files for Gems 1 to 5 but didn't bother about warnings. Everything compiles with VS2015. I think I'm done now.

g

P.S.: If somebody is interested in doing a Graphics Gems Revisisted Kindle Edition with everything updated to modern C++ I'd be happy to help out. But this would be a lot of work. :)

gonsolo commented 8 years ago

All errors and warnings fixed on Linux/OS X/Windows 10.

This should be it. :)

erich666 commented 8 years ago

Thanks! Merged.