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

Added example for the Marmalade platform (2) #375

Merged
merged 4 commits into from
Oct 15, 2015
Merged

Conversation

gzito
Copy link
Contributor

@gzito gzito commented Oct 14, 2015

Modified as requested

@gzito gzito changed the title Port of ImGui to Marmalade Added example for the Marmalade platform (2) Oct 14, 2015
ocornut added a commit that referenced this pull request Oct 15, 2015
Examples: Added example app for the Marmalade platform.
@ocornut ocornut merged commit 81927b0 into ocornut:master Oct 15, 2015
@ocornut
Copy link
Owner

ocornut commented Oct 15, 2015

Thanks!

@ocornut
Copy link
Owner

ocornut commented Oct 15, 2015

Shouldn't the IwGxInit() and IwGxTerminate() calls be in main.cpp and not the imgui_impl* file ?
The imgui_impl* are meant to be copied unmodified and included in user's program.

@ocornut
Copy link
Owner

ocornut commented Oct 15, 2015

Have also made several coding style change to match the rest of the code. Needless to say none of that is tested cough I should download Marmalade. Perhaps you could double check I haven't broken compilation with none of my OCD change?

@gzito
Copy link
Contributor Author

gzito commented Oct 15, 2015

Ok! I will sync my fork and test your changes later ... And I'll try conform to the new imgui_impl* coding style too.

Thanks.

ocornut added a commit that referenced this pull request Jul 30, 2016
…les (#323)

Missing support Vulkan (#549), Apple (#575, #247), SDL (#58, #356),
Allegro, Marmalade (#368, #375)
ocornut added a commit that referenced this pull request Sep 18, 2017
…les (#323)

Missing support Vulkan (#549), Apple (#575, #247), SDL (#58, #356),
Allegro, Marmalade (#368, #375)
ocornut added a commit that referenced this pull request Dec 20, 2021
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