Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update nanogui and use our own mainloop #221

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

petterarvidsson
Copy link
Member

To make it easy to follow nanogui development, remove our changes and instead implement a simple main loop of our won based on: https://github.com/wjakob/nanogui/blob/master/src/example3.cpp

Also replace nanogui dependency with the one directly from https://github.com/wjakob/nanogui

@petterarvidsson
Copy link
Member Author

@nsg What do you think?

@petterarvidsson petterarvidsson force-pushed the feature-stop-using-nanogui-main-loop branch from 370a845 to 22687ed Compare November 11, 2016 11:06
@petterarvidsson
Copy link
Member Author

Apparently they moved from glew to glad for windows compatibility... ugh...

@petterarvidsson petterarvidsson force-pushed the feature-stop-using-nanogui-main-loop branch from 9041a8f to 715f31b Compare November 11, 2016 15:29
@nsg
Copy link
Member

nsg commented Nov 12, 2016

@petterarvidsson Looks good, a little work to keep all these callbacks up-2-date in the future but over all this is a good thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants