Skip to content

Enable include reordering #3886

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

Closed

Conversation

marcogmaia
Copy link
Contributor

The purpose of this is for us to be able to do the following:

#include <raymath.h>
#include <rlgl.h>
#include <raylib.h>

Right now the only possibility is to include raylib.h first.

@marcogmaia
Copy link
Contributor Author

@raysan5 I have one question:

Why don't we extract this repeating redefinition to a single source of truth? Something like internal/common_types.h?

@raysan5
Copy link
Owner

raysan5 commented Mar 28, 2024

@marcogmaia Sorry but this change adds extra complexity to the header that I prefer to avoid.

Why don't we extract this repeating redefinition to a single source of truth? Something like internal/common_types.h?

I prefer to keep the raylib.h header self-contained and not-dependant on other headers.

@raysan5 raysan5 closed this Mar 28, 2024
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