-
Notifications
You must be signed in to change notification settings - Fork 393
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
Calculate hashCode with arguments that are consistent with equals #1097
Conversation
@AniketNS thanks for the pull request. Please do the following:
|
Hello @MarkEWaite, could you please provide some info on the current status of this PR? I'm wondering if there are any specific improvements or additional changes I should make. I appreciate your feedback. |
Please take the actions that I listed in the checklist of the earlier comment. those actions include:
An example pull request that uses the pull request template is: |
Hey @MarkEWaite, I've changed the Pull request name as you suggested but can you please tell me now how I can restore the pull request template? Even if I copy the content from this pull req template then how can I add that checklist? Or should I close this pull request and open a new one with the default pull request template and with a clear pull request description as you told me before? |
One way is to copy the source text of the pull request template and paste it into the description of this pull request. The source text is also on your local copy of the repository if you prefer to copy it from a local text file. |
Thank you so much @MarkEWaite. I've made the changes as requested. Please check and let me know if I've missed anything. |
Calculate hashCode with arguments that are consistent with equals
I used
String.format()
function for better consistency and it is more understandable.Instead of relying on the
hashCode
of the superclass I've used theObject.hash()
method to generate a hashcode. I think it is easier to understand and more explicit.Checklist
Types of changes
What types of changes does your code introduce?