CSSTUDIO-2880 Remove "synchronized" from the definition of refresh(), since the function now always runs on the UI thread.

abrahamwolk · abrahamwolk · commit 39a65fbf99d6 · 2024-12-18T12:38:05.000+01:00
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
@@ -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());

Original file line number	Diff line number	Diff line change
`@@ -435,7 +435,7 @@ public String getQuery() {`
`435`	`435`	`return query.getValue().getQuery();`
`436`	`436`	`}`
`437`	`437`
`438`		`- private synchronized void refresh() {`
	`438`	`+ private void refresh() {`
`439`	`439`	`Runnable refreshRunnable = () -> {`
`440`	`440`	`if (this.searchResult != null) {`
`441`	`441`	`List<TableViewListItem> selectedLogEntries = new ArrayList<>(tableView.getSelectionModel().getSelectedItems());`