use Objects.hash instead of concat hashcode

Author: 317787106

chainbase/src/main/java/org/tron/core/db2/core/SnapshotManager.java

@@ -423,6 +423,7 @@ public List<String> getCheckpointList() {
 
   private void deleteCheckpoint() {
     if(checkTmpStore == null) {
+      //only occurs in mock test. TODO fix test
       return;
     }
     try {

framework/src/main/java/org/tron/common/logsfilter/EventPluginLoader.java

@@ -546,6 +546,7 @@ public boolean isBusy() {
     }
     int queueSize = 0;
     if (eventListeners == null || eventListeners.isEmpty()) {
+      //only occurs in mock test. TODO fix test
       return false;
     }
     for (IPluginEventListener listener : eventListeners) {

framework/src/main/java/org/tron/core/services/jsonrpc/TronJsonRpc.java

@@ -24,7 +24,6 @@
 import org.tron.core.exception.jsonrpc.JsonRpcInvalidRequestException;
 import org.tron.core.exception.jsonrpc.JsonRpcMethodNotFoundException;
 import org.tron.core.exception.jsonrpc.JsonRpcTooManyResultException;
-import org.tron.core.net.peer.Item;
 import org.tron.core.services.jsonrpc.types.BlockResult;
 import org.tron.core.services.jsonrpc.types.BuildArguments;
 import org.tron.core.services.jsonrpc.types.CallArguments;
@@ -501,11 +500,7 @@ public boolean equals(Object o) {
 
     @Override
     public int hashCode() {
-      int result = blockHash != null ? blockHash.hashCode() : 0;
-      result = 31 * result + (transactionHash != null ? transactionHash.hashCode() : 0);
-      result = 31 * result + (transactionIndex != null ? transactionIndex.hashCode() : 0);
-      result = 31 * result + (logIndex != null ? logIndex.hashCode() : 0);
-      return result + (removed ? 1 : 0);
+      return Objects.hash(blockHash, transactionHash, transactionIndex, logIndex, removed);
     }
 
   }

framework/src/test/java/org/tron/core/jsonrpc/LogMatchExactlyTest.java

@@ -4,6 +4,7 @@
 
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Objects;
 import org.junit.Assert;
 import org.junit.Test;
 import org.tron.common.runtime.vm.DataWord;
@@ -220,6 +221,18 @@ public void testMatchBlock() {
       List<LogFilterElement> elementList =
           matchBlock(logFilter, 100, null, transactionInfoList, false);
       Assert.assertEquals(1, elementList.size());
+
+      //test LogFilterElement
+      List<LogFilterElement> elementList2 =
+          matchBlock(logFilter, 100, null, transactionInfoList, false);
+      Assert.assertEquals(1, elementList2.size());
+
+      LogFilterElement logFilterElement1 = elementList.get(0);
+      LogFilterElement logFilterElement2 = elementList2.get(0);
+
+      Assert.assertEquals(logFilterElement1.hashCode(), logFilterElement2.hashCode());
+      Assert.assertEquals(logFilterElement1, logFilterElement2);
+
     } catch (JsonRpcInvalidParamsException e) {
       Assert.fail();
     }