-
Notifications
You must be signed in to change notification settings - Fork 78
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
CXX_STANDARD 17 #1283
Comments
It depends on the platform. For example embedded / mobile systems are not necessarily using C++17. |
Don't get me wrong, `personally' is not one of my wet dreams to have CXX_STANDARD 17 but errors like:
using the defaults on 14 are a show stopper for newcomers. On another note, keep up the good work CEGUI team! |
Well tbh I'm kinda questioning recent modern C++ features added to the codebase too. Haven't had time to really pull the new version yet to review but I think that it should be optimized since there're multiple related issue tickets have been shown. |
Fixed unintended C++17 usage, now must build under C++14 |
Please default to CXX_STANDARD 17 on cegui/cmake/CEGUIMacros.cmake instead of 14.
The text was updated successfully, but these errors were encountered: