Browse Source

Optimized code in Synchronizer.uniqueCommonBlocks()

prioritize-peers
CalDescent 3 years ago
parent
commit
f532dbe7b4
  1. 6
      src/main/java/org/qortal/controller/Synchronizer.java

6
src/main/java/org/qortal/controller/Synchronizer.java

@ -403,10 +403,7 @@ public class Synchronizer {
private List<BlockSummaryData> uniqueCommonBlocks(List<Peer> peers) { private List<BlockSummaryData> uniqueCommonBlocks(List<Peer> peers) {
List<BlockSummaryData> commonBlocks = new ArrayList<>(); List<BlockSummaryData> commonBlocks = new ArrayList<>();
Iterator<Peer> iterator = peers.iterator(); for (Peer peer : peers) {
while (iterator.hasNext()) {
Peer peer = iterator.next();
if (peer.getCommonBlockData() != null && peer.getCommonBlockData().getCommonBlockSummary() != null) { if (peer.getCommonBlockData() != null && peer.getCommonBlockData().getCommonBlockSummary() != null) {
LOGGER.debug(String.format("Peer %s has common block %.8s", peer, Base58.encode(peer.getCommonBlockData().getCommonBlockSummary().getSignature()))); LOGGER.debug(String.format("Peer %s has common block %.8s", peer, Base58.encode(peer.getCommonBlockData().getCommonBlockSummary().getSignature())));
@ -416,7 +413,6 @@ public class Synchronizer {
} }
else { else {
LOGGER.debug(String.format("Peer %s has no common block data. Skipping...", peer)); LOGGER.debug(String.format("Peer %s has no common block data. Skipping...", peer));
iterator.remove();
} }
} }

Loading…
Cancel
Save