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 for issue 17: rename concept to concepts to fix clash with C++20 keyword #18

Merged
merged 1 commit into from
Oct 6, 2024

Conversation

willowell
Copy link
Contributor

@willowell willowell commented May 9, 2020

#17

@elmajime
Copy link

elmajime commented Feb 7, 2022

Hi, is there any plans to merge this branch ?

@willowell
Copy link
Contributor Author

willowell commented Feb 7, 2022

@elmajime Hi there,

Please note that oktal gave this an open source license, so you (or I!) can fork this repository and apply the fix.

My fix is obviously trivial enough that you can do it yourself :-)

This also does not preclude giving back via a PR across forks.

@LuisFilipeDias
Copy link

@oktal this is becoming more of an issue. https://releases.llvm.org/13.0.0/tools/clang/docs/ReleaseNotes.html#c-20-feature-support is also problematic, as it provides: C++20 Feature Support.

Can we go ahead and merge this into mainline? Thanks in advance.

@oktal oktal merged commit d3598ec into oktal:master Oct 6, 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.

4 participants