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

sanity check template parameters at compile time #29

Merged
merged 1 commit into from
Jul 3, 2013

Conversation

mpetri
Copy link
Collaborator

@mpetri mpetri commented Jul 3, 2013

in c++11 we can now sanity check the template parameters of the
different data structures used in the library using static_assert.
this commit introduces basic checks to all applicable data structures in
the library.

p.s. this also fixes the formating of csa_sada

in c++11 we can now sanity check the template parameters of the
different data structures used in the library using static_assert.
this commit introduces basic checks to all applicable data structures in
the library.
@ghost ghost assigned simongog Jul 3, 2013
@simongog
Copy link
Owner

simongog commented Jul 3, 2013

Thanks Matthias. I will add a constexpr function for the "power of two" check.

simongog added a commit that referenced this pull request Jul 3, 2013
sanity check template parameters at compile time
@simongog simongog merged commit 510bceb into simongog:c++11 Jul 3, 2013
simongog pushed a commit that referenced this pull request Sep 10, 2013
Replaced double bracket notation from bash to single
bracket notation to make the script work with sh.

Added force flag to the copy command for the gtest
files.
mpetri pushed a commit to mpetri/sdsl-lite that referenced this pull request Sep 10, 2013
Replaced double bracket notation from bash to single
bracket notation to make the script work with sh.

Added force flag to the copy command for the gtest
files.
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