-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Error compilling result.hpp on apple-clang. #6
Comments
This change fixes for me:
should read:
|
Thanks for opening this issue. Funny, the StackOverflow question you linked is actually my question that I asked, researched, and answered regarding this exact problem; I just forgot to follow-up on this. 😅 I'll make this change shortly. |
Also my apologies for missing this when it was opened; I apparently need to update my notification preferences. |
@tcanabrava Are you interested in making the PR for this? |
I also don't mind making this change since it's basically a 1-line fix. The only thing that stopped me was CI failures since it's been a long time since I've updated everything. I prefer having the safety net of CI to tell me when changes aren't accepted for older compilers, since some of them are "quirky", in what they deem to be valid C++. |
Linux Compier: GCC 11
Mac Compiler: latest apple-clang
The resulting code works on linux / gcc, but on mac I get:
The text was updated successfully, but these errors were encountered: