search

Overv / OOGL

Object-oriented C++ wrapper for OpenGL.
Boost Software License 1.0
414 stars 45 forks source link

Develop #12

Closed necrophcodr closed 12 years ago

necrophcodr commented 12 years ago

Updated the stuff I wrote about on Facepunch, and also added in project files for all major platforms with the use of premake4.

Jookia commented 12 years ago

This fixes #12.

Overv commented 12 years ago

I will accept this request if you resubmit without the image and mesh defines.

necrophcodr commented 12 years ago

What about a --without-imageloader and changing the #ifdef to #ifndef so they're built in per standard?

Overv commented 12 years ago

Could you explain why the defines are necessary at all?

necrophcodr commented 12 years ago

I just thought that being able to decide if you want the mesh- and/or imageloader built in would be a nice thing. I guess I'm just a fan of modularity.

Overv commented 12 years ago

I know, but I think the defines would clutter for example the Texture and VertexBuffer constructors.

necrophcodr commented 12 years ago

Alright then, any suggestions for a better alternative? I've removed the defines, and it's ready to be pulled in.

Overv commented 12 years ago

Could you open an issue? Then we'll further discuss the options.