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

IterableOf, MapEntry and MapEnvelope should use HashCode instead of own hash code algorithm #1440

Open
fabriciofx opened this issue Sep 6, 2020 · 7 comments

Comments

@fabriciofx
Copy link
Contributor

fabriciofx commented Sep 6, 2020

IterableOf, MapEntry and MapEnvelope (or MapOf if #1384 has been done) should use HashCode instead of own hash code. So let's refactor them to use HashCode.

@0crat 0crat added the 0crat/new label Sep 6, 2020
@fabriciofx fabriciofx changed the title IterableOf, MapEntry and MapEnvelope should use HashCode instead of own hash code IterableOf, MapEntry and MapEnvelope should use HashCode instead of own hash code algorithm Sep 6, 2020
@victornoel
Copy link
Collaborator

@fabriciofx that seems a good idea. Can you just edit the description to say:

IterableOf, MapEntry and MapEnvelope (or MapOf if #1384 has been done)...

@fabriciofx
Copy link
Contributor Author

@victornoel is it better now?

@victornoel
Copy link
Collaborator

@0crat in

@victornoel
Copy link
Collaborator

@fabriciofx thanks :)

@fabriciofx
Copy link
Contributor Author

@victornoel welcome!

@victornoel
Copy link
Collaborator

@0crat status

@0crat
Copy link
Collaborator

0crat commented Jun 12, 2021

@0crat status (here)

@victornoel This is what I know about this job in C63314D6Z, as in §32:

  • The job #1440 is in scope for 9mon
  • The role is DEV
  • The job is not assigned to anyone
  • The budget is not set yet
  • These users are banned and won't be assigned:
  • Job footprint (restricted area)

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

3 participants