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

Added constexpr modifiers for C++14 mode and some tests #13

Closed
wants to merge 1 commit into
base: develop
from

Conversation

Projects
None yet
2 participants
@apolukhin
Copy link
Member

apolukhin commented Jan 25, 2016

No description provided.

@mclow

This comment has been minimized.

Copy link
Collaborator

mclow commented Jan 26, 2016

I like this change. But the tests are incomplete. There's no constexpr tests for mismatch, for example. (or power)

@apolukhin

This comment has been minimized.

Copy link
Member Author

apolukhin commented Jan 26, 2016

OK. I'll add more tests soon. Anyway, this PR must be merged after constexpr for for begin()/end() PR

@mclow

This comment has been minimized.

Copy link
Collaborator

mclow commented Jan 26, 2016

Ok; I'll wait for your update.

@mclow

This comment has been minimized.

Copy link
Collaborator

mclow commented Mar 8, 2016

Looks like boostorg/range#44 hasn't been merged yet.

@apolukhin

This comment has been minimized.

Copy link
Member Author

apolukhin commented Apr 18, 2017

This pull request is superseded by #35

@apolukhin apolukhin closed this Apr 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment