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

Refactor KTfwd::traits #46

Closed
molpopgen opened this issue Feb 26, 2017 · 1 comment
Closed

Refactor KTfwd::traits #46

molpopgen opened this issue Feb 26, 2017 · 1 comment
Assignees

Comments

@molpopgen
Copy link
Owner

molpopgen commented Feb 26, 2017

Issues #45 brings up some aesthetic issues with how type traits are implemented. They should be cleaned up and cues taken from the C++17 additions to the standard library's type traits. This basically involves some renaming and adding template aliases for convenience.

In addressing the issue described above, I discovered that a lot of the type traits fail to do their job. Many will fail to compile when provided "bad" or incorrect types rather than have SFINAE result in a static type that can be evaluated. This issue is turning into a "refactor the type traits part of fwdpp".

@molpopgen molpopgen self-assigned this Feb 26, 2017
molpopgen added a commit that referenced this issue Feb 27, 2017
molpopgen added a commit that referenced this issue Feb 27, 2017
…s model type. Will eval to void if argument types are incorrect. #46
molpopgen added a commit that referenced this issue Feb 27, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Feb 28, 2017
molpopgen added a commit that referenced this issue Mar 1, 2017
@molpopgen
Copy link
Owner Author

Done in 0.5.5

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

No branches or pull requests

1 participant