From fd3f0e6d2570c5a5a7461dfc1a204ad77958b4b5 Mon Sep 17 00:00:00 2001
From: Christoph Atteneder <christoph.atteneder@gmail.com>
Date: Mon, 1 Feb 2021 10:52:06 +0100
Subject: [PATCH] Minor cleanup

---
 .../pendingtrades/PendingTradesView.java          | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java b/desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java
index c2dfcea6c6..0e82a2bd1e 100644
--- a/desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java
+++ b/desktop/src/main/java/bisq/desktop/main/portfolio/pendingtrades/PendingTradesView.java
@@ -126,16 +126,16 @@ public class PendingTradesView extends ActivatableViewAndModel<VBox, PendingTrad
     private Subscription selectedItemSubscription;
     private Stage chatPopupStage;
     private ListChangeListener<PendingTradesListItem> tradesListChangeListener;
-    private Map<String, Long> newChatMessagesByTradeMap = new HashMap<>();
+    private final Map<String, Long> newChatMessagesByTradeMap = new HashMap<>();
     private String tradeIdOfOpenChat;
     private double chatPopupStageXPosition = -1;
     private double chatPopupStageYPosition = -1;
     private ChangeListener<Number> xPositionListener;
     private ChangeListener<Number> yPositionListener;
 
-    private Map<String, Button> buttonByTrade = new HashMap<>();
-    private Map<String, JFXBadge> badgeByTrade = new HashMap<>();
-    private Map<String, ListChangeListener<ChatMessage>> listenerByTrade = new HashMap<>();
+    private final Map<String, Button> buttonByTrade = new HashMap<>();
+    private final Map<String, JFXBadge> badgeByTrade = new HashMap<>();
+    private final Map<String, ListChangeListener<ChatMessage>> listenerByTrade = new HashMap<>();
     private ChangeListener<Trade.State> tradeStateListener;
     private ChangeListener<Trade.DisputeState> disputeStateListener;
     private ChangeListener<MediationResultState> mediationResultStateListener;
@@ -536,9 +536,7 @@ public class PendingTradesView extends ActivatableViewAndModel<VBox, PendingTrad
         if (selectedItemFromModel != null) {
             // Select and focus selectedItem from model
             int index = tableView.getItems().indexOf(selectedItemFromModel);
-            UserThread.execute(() -> {
-                tableView.getSelectionModel().select(index);
-            });
+            UserThread.execute(() -> tableView.getSelectionModel().select(index));
         }
     }
 
@@ -864,7 +862,7 @@ public class PendingTradesView extends ActivatableViewAndModel<VBox, PendingTrad
         return chatColumn;
     }
 
-    private TableColumn<PendingTradesListItem, PendingTradesListItem> setRemoveTradeColumnCellFactory() {
+    private void setRemoveTradeColumnCellFactory() {
         moveTradeToFailedColumn.setCellValueFactory((trade) -> new ReadOnlyObjectWrapper<>(trade.getValue()));
         moveTradeToFailedColumn.setCellFactory(
                 new Callback<>() {
@@ -937,6 +935,5 @@ public class PendingTradesView extends ActivatableViewAndModel<VBox, PendingTrad
                         };
                     }
                 });
-        return moveTradeToFailedColumn;
     }
 }