use Objects.hash()

Vladimir Kotal · Vladimir Kotal · commit 3f0c984cf749 · 2021-04-16T21:46:48.000+02:00
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/history/History.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/history/History.java
@@ -139,7 +139,7 @@ public boolean equals(Object o) {
 
     @Override
     public int hashCode() {
-        return getHistoryEntries().hashCode() * getRenamedFiles().hashCode();
+        return Objects.hash(getHistoryEntries().hashCode(), getRenamedFiles().hashCode());
     }
 
     @Override
diff --git a/opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryEntry.java b/opengrok-indexer/src/main/java/org/opengrok/indexer/history/HistoryEntry.java
@@ -246,7 +246,7 @@ public boolean equals(Object o) {
 
     @Override
     public int hashCode() {
-        return getHashCode(getAuthor()) + getHashCode(getRevision()) + getHashCode(getDate()) +
-                getHashCode(getMessage()) + getHashCode(getFiles()) + getHashCode(getTags());
+        return Objects.hash(getHashCode(getAuthor()), getHashCode(getRevision()), getHashCode(getDate()),
+                getHashCode(getMessage()), getHashCode(getFiles()), getHashCode(getTags()));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ public boolean equals(Object o) {`
`139`	`139`
`140`	`140`	`@Override`
`141`	`141`	`public int hashCode() {`
`142`		`- return getHistoryEntries().hashCode() * getRenamedFiles().hashCode();`
	`142`	`+ return Objects.hash(getHistoryEntries().hashCode(), getRenamedFiles().hashCode());`
`143`	`143`	`}`
`144`	`144`
`145`	`145`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -246,7 +246,7 @@ public boolean equals(Object o) {`
`246`	`246`
`247`	`247`	`@Override`
`248`	`248`	`public int hashCode() {`
`249`		`- return getHashCode(getAuthor()) + getHashCode(getRevision()) + getHashCode(getDate()) +`
`250`		`- getHashCode(getMessage()) + getHashCode(getFiles()) + getHashCode(getTags());`
	`249`	`+ return Objects.hash(getHashCode(getAuthor()), getHashCode(getRevision()), getHashCode(getDate()),`
	`250`	`+ getHashCode(getMessage()), getHashCode(getFiles()), getHashCode(getTags()));`
`251`	`251`	`}`
`252`	`252`	`}`