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

Change hash-map to use std::hash<T> as the hash functor template #5

Open
Dar13 opened this issue Mar 17, 2016 · 0 comments
Open

Change hash-map to use std::hash<T> as the hash functor template #5

Dar13 opened this issue Mar 17, 2016 · 0 comments

Comments

@Dar13
Copy link
Contributor

Dar13 commented Mar 17, 2016

This change is needed because right now only integral types can be directly used as a key in Tervel's hash map implementation due to restrictions put on the signature of the hash() method in the current functor definition.

This change would require keys' internal type be size_t which should be trivial to change. This approach also allows users of the hash map more flexibility with regard to the types of keys used in addition to the hashing algorithm they can use.

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

No branches or pull requests

1 participant