From: Frédéric Perrin <frederic.perrin@att.com>
Date: Wed, 29 Jan 2020 15:23:56 +0000 (+0000)
Subject: Don't informSubscriber() after launching files
X-Git-Tag: v19.2.fp6~2^2
X-Git-Url: http://gitweb.fperrin.net/?p=GpsPrune.git;a=commitdiff_plain;h=4663754ebc9b97c9aa79365047ec1c5533c29a46

Don't informSubscriber() after launching files

It doesn't seem necessary (the recent files list is populater even
without), and creates a deadlock between:

AWT-EventQueue-0:
  [1] tim.prune.gui.profile.ProfileChart.makePopup (ProfileChart.java:381)
  [2] tim.prune.gui.profile.ProfileChart.dataUpdated (ProfileChart.java:459)
  [3] tim.prune.UpdateMessageBroker.informSubscribers (UpdateMessageBroker.java:84)
  [4] tim.prune.UpdateMessageBroker.informSubscribers (UpdateMessageBroker.java:68)
  [5] tim.prune.gui.map.MapCanvas.itemStateChanged (MapCanvas.java:149)
  [ ... ]
  [10] javax.swing.AbstractButton.setSelected (AbstractButton.java:348)
  [11] tim.prune.gui.map.MapCanvas.paintMapContents (MapCanvas.java:484)
  [12] tim.prune.gui.map.MapCanvas.paint (MapCanvas.java:380)

main:
  [ ... ]
  [11] javax.swing.JPopupMenu.<init> (JPopupMenu.java:178)
  [12] tim.prune.gui.profile.ProfileChart.makePopup (ProfileChart.java:381)
  [13] tim.prune.gui.profile.ProfileChart.dataUpdated (ProfileChart.java:459)
  [14] tim.prune.UpdateMessageBroker.informSubscribers (UpdateMessageBroker.java:84)
  [15] tim.prune.UpdateMessageBroker.informSubscribers (UpdateMessageBroker.java:68)
  [16] tim.prune.GpsPrune.launch (GpsPrune.java:266)
  [17] tim.prune.GpsPrune.main (GpsPrune.java:153)
---

diff --git a/src/tim/prune/GpsPrune.java b/src/tim/prune/GpsPrune.java
index f396e9e..a9b53f4 100644
--- a/src/tim/prune/GpsPrune.java
+++ b/src/tim/prune/GpsPrune.java
@@ -262,8 +262,6 @@ public class GpsPrune
 		frame.setVisible(true);
 		// Set position of map/profile splitter
 		midSplit.setDividerLocation(0.75);
-		// Update menu (only needed for recent file list)
-		UpdateMessageBroker.informSubscribers();
 
 		// Make a full screen toggler
 		SidebarController fsc = new SidebarController(new Component[] {leftPanel, profileDisp, rightPanel},