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

Fix the compilation warnings #99

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

Conversation

gostefan
Copy link
Contributor

@gostefan gostefan commented Aug 1, 2024

No description provided.

@gostefan
Copy link
Contributor Author

gostefan commented Sep 4, 2024

Commit 05a58f4 omits to add the utils.h to the public interface list.
I hope I can fix this tonight.

@gostefan
Copy link
Contributor Author

gostefan commented Sep 4, 2024

Created a new header warnings.h as to not expose utils.h in the public interface.
Therefore needed to force-push.

@gostefan
Copy link
Contributor Author

gostefan commented Sep 5, 2024

Fixed another problem in suppressing MSVC warnings. --> Needed to force-push again.

@gostefan
Copy link
Contributor Author

I'm sorry. There was another problem with the new includes. I still had the utils.h referenced instead of the warnings.h in many places. This should be fixed now.

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.

1 participant