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

Standard lib includes maphash #156

Closed
masknu opened this issue May 7, 2020 · 2 comments
Closed

Standard lib includes maphash #156

masknu opened this issue May 7, 2020 · 2 comments
Labels
kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. status/accepted We accept to work on it.

Comments

@masknu
Copy link

masknu commented May 7, 2020

It seems like golang standard library includes memhash in version 1.14.1.
The import path is "hash/maphash".
Didn't test it yet, perhaps it could replace memhash in the code.

@shekarm shekarm added status/needs-attention This issue needs more eyes on it, more investigation might be required before accepting/rejecting it kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. labels May 11, 2020
@parasssh parasssh added status/accepted We accept to work on it. and removed status/needs-attention This issue needs more eyes on it, more investigation might be required before accepting/rejecting it labels May 11, 2020
@parasssh
Copy link

We will look at it.

@minhaj-shakeel
Copy link
Contributor

Github issues have been deprecated.
This issue has been moved to discuss. You can follow the conversation there and also subscribe to updates by changing your notification preferences.

drawing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/maintenance Maintenance tasks, such as refactoring, with no impact in features. status/accepted We accept to work on it.
Projects
None yet
Development

No branches or pull requests

4 participants