From 39a65fbf99d6586963512c7d12609391789821a4 Mon Sep 17 00:00:00 2001
From: Abraham Wolk <abraham.wolk@ess.eu>
Date: Wed, 18 Dec 2024 12:38:05 +0100
Subject: [PATCH] CSSTUDIO-2880 Remove "synchronized" from the definition of
 refresh(), since the function now always runs on the UI thread.

---
 .../phoebus/logbook/olog/ui/LogEntryTableViewController.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryTableViewController.java b/app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryTableViewController.java
index b89dba75d..8c692f72b 100644
--- a/app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryTableViewController.java
+++ b/app/logbook/olog/ui/src/main/java/org/phoebus/logbook/olog/ui/LogEntryTableViewController.java
@@ -435,7 +435,7 @@ public String getQuery() {
         return query.getValue().getQuery();
     }
 
-    private synchronized void refresh() {
+    private void refresh() {
         Runnable refreshRunnable = () -> {
             if (this.searchResult != null) {
                 List<TableViewListItem> selectedLogEntries = new ArrayList<>(tableView.getSelectionModel().getSelectedItems());