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

Fixes for compatibility with Clang #81

Closed
wants to merge 1 commit into from

Conversation

jcelerier
Copy link
Contributor

cheers !

@leadedge
Copy link
Owner

Thanks very much for your attention. It will take me a little while to get up to this. I can merge the changes here, but I will also include them by hand in working files before the next commit because there are other changes pending. No problems though.

@leadedge
Copy link
Owner

OK -I got a few moments free time.

What about -
#include in SpoutCopy.h
std::floor instead of "using namespace std;"
(I removed "using namespace std;" some time back due to reported problems)

@jcelerier
Copy link
Contributor Author

okay, i'll do the change. thanks for the quick review !

@leadedge
Copy link
Owner

All sorted

@leadedge leadedge closed this Feb 24, 2022
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