Apply suggestions from code review

Signed-off-by: Fabio Di Fabio <[email protected]>
macfarla · Nov 30, 2023 · f278102 · f278102
1 parent a750b66
commit f278102
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/...n/java/org/hyperledger/besu/ethereum/eth/manager/task/GetHeadersFromPeerByNumberTask.java b/...n/java/org/hyperledger/besu/ethereum/eth/manager/task/GetHeadersFromPeerByNumberTask.java
@@ -81,7 +81,7 @@ protected PendingPeerRequest sendRequest() {
               .setMessage("Requesting {} headers (blockNumber {}) from peer {}.")
               .addArgument(count)
               .addArgument(blockNumber)
-              .addArgument(peer.getShortNodeId())
+              .addArgument(peer::getShortNodeId)
               .log();
           return peer.getHeadersByNumber(blockNumber, count, skip, reverse);
         },

diff --git a/...src/main/java/org/hyperledger/besu/ethereum/eth/manager/task/GetNodeDataFromPeerTask.java b/...src/main/java/org/hyperledger/besu/ethereum/eth/manager/task/GetNodeDataFromPeerTask.java
@@ -68,8 +68,8 @@ protected PendingPeerRequest sendRequest() {
         peer -> {
           LOG.atTrace()
               .setMessage("Requesting {} node data entries from peer {}...")
-              .addArgument(hashes.size())
-              .addArgument(peer.getShortNodeId())
+              .addArgument(hashes::size)
+              .addArgument(peer::getShortNodeId)
               .log();
           return peer.getNodeData(hashes);
         },