suppress parser error dialog when refreshing automatically

Author: chewiebug

src/main/java/com/tagtraum/perf/gcviewer/ChartPanelView.java

@@ -91,13 +91,17 @@ public ChartPanelView(GCDocument gcDocument, URL url) throws DataReaderException
     }
 
     /**
-     * @return true, if the files has been reloaded
+     * Reloads the model displayed in this chart panel if it has changed. Using the parameter
+     * the parser error dialog can be suppressed.
+     * 
+     * @param showParserErrors if <code>true</code> parser errors will be shown
+     * @return <code>true</code>, if the file has been reloaded
      * @throws DataReaderException if something went wrong reading the file
      */
-    public boolean reloadModel() throws DataReaderException {
+    public boolean reloadModel(boolean showParserErrors) throws DataReaderException {
         if (model.getURL() == null) return false;
         if (model.isDifferent(model.getURL())) {
-            setModel(dataReaderFacade.loadModel(this.model.getURL(), true, gcDocument));
+            setModel(dataReaderFacade.loadModel(this.model.getURL(), showParserErrors, gcDocument));
             return true;
         }
         return false;

src/main/java/com/tagtraum/perf/gcviewer/GCDocument.java

@@ -127,7 +127,7 @@ public void setShowModelPanel(boolean showModelPanel) {
     public boolean reloadModels(boolean background) throws DataReaderException {
         boolean reloaded = false;
         for (ChartPanelView chartPanelView : chartPanelViews) {
-            reloaded |= chartPanelView.reloadModel();
+            reloaded |= chartPanelView.reloadModel(!refreshWatchDog.isRunning());
         }
         if (!background) {
             relayout();

src/main/java/com/tagtraum/perf/gcviewer/RefreshWatchDog.java

@@ -1,78 +1,94 @@
 package com.tagtraum.perf.gcviewer;
 
-import javax.swing.*;
-import java.awt.*;
-import java.awt.event.ActionEvent;
-import java.awt.event.ActionListener;
+import java.awt.Toolkit;
 import java.util.TimerTask;
 
+import javax.swing.Action;
+import javax.swing.ImageIcon;
+import javax.swing.SwingUtilities;
+
 /**
+ * Timer to trigger refresh of changed log files.
  *
- * Date: May 26, 2005
- * Time: 2:04:38 PM
+ * <p>Date: May 26, 2005</p>
+ * <p>Time: 2:04:38 PM</p>
  * @author <a href="mailto:[email protected]">Hendrik Schreiber</a>
  */
 public class RefreshWatchDog {
     private GCDocument gcDocument;
-    private javax.swing.Timer animationTimer;
     private java.util.Timer reloadTimer;
     private Action action;
+    private boolean isRunning;
     public static final ImageIcon WATCH_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewerGui.class.getResource("images/watch.png")));
     private static final ImageIcon CLOCK_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewerGui.class.getResource("images/clock.png")));
-    /*
-    private static final ImageIcon CLOCK_N_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_n.gif")));
-    private static final ImageIcon CLOCK_NE_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_ne.gif")));
-    private static final ImageIcon CLOCK_E_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_e.gif")));
-    private static final ImageIcon CLOCK_SE_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_se.gif")));
-    private static final ImageIcon CLOCK_S_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_s.gif")));
-    private static final ImageIcon CLOCK_SW_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_sw.gif")));
-    private static final ImageIcon CLOCK_W_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_w.gif")));
-    private static final ImageIcon CLOCK_NW_ICON = new ImageIcon(Toolkit.getDefaultToolkit().getImage(GCViewer.class.getResource("images/clock_nw.gif")));
-    private static final ImageIcon[] CLOCK_ICONS = new ImageIcon[] {CLOCK_N_ICON, CLOCK_NE_ICON, CLOCK_E_ICON, CLOCK_SE_ICON, CLOCK_S_ICON, CLOCK_SW_ICON, CLOCK_W_ICON, CLOCK_NW_ICON};
-    */
-    private static final int ANIMATION_DELAY = 250;
     private static final int RELOAD_DELAY = 1000;
 
+    /**
+     * Get the {@link GCDocument} this watchdog is used for.
+     * @return GCDocument for this watchdog
+     */
     public GCDocument getGcDocument() {
         return gcDocument;
     }
 
+    /**
+     * Set the {@link GCDocument} this watchdog should trigger refreshes for.
+     * @param gcDocument to be refreshed
+     */
     public void setGcDocument(GCDocument gcDocument) {
         this.gcDocument = gcDocument;
     }
 
+    /**
+     * Get action that is triggered by the watchdog.
+     * @return Action to be triggered
+     */
     public Action getAction() {
         return action;
     }
 
+    /**
+     * Set action to be triggered by this watchdog.
+     * @param action Action to be triggered
+     */
     public void setAction(Action action) {
         this.action = action;
     }
 
+    /**
+     * Start the watchdog.
+     */
     public void start() {
-        animationTimer = new javax.swing.Timer(ANIMATION_DELAY, new ClockAnimation());
         reloadTimer = new java.util.Timer(true);
         reloadTimer.schedule(new ModelReloader(), 0, RELOAD_DELAY);
         action.putValue(Action.SMALL_ICON, CLOCK_ICON);
-        animationTimer.start();
+        isRunning = true;
     }
 
+    /**
+     * Stop the watchdog.
+     */
     public void stop() {
         if (reloadTimer != null) reloadTimer.cancel();
-        if (animationTimer != null && animationTimer.isRunning()) animationTimer.stop();
         if (action != null) action.putValue(Action.SMALL_ICON, WATCH_ICON);
+        isRunning = false;
     }
-
-    private class ClockAnimation implements ActionListener {
-        private int clockIndex;
-        public void actionPerformed(final ActionEvent e) {
-            if (action != null) {
-                //clockIndex = (clockIndex+1) % CLOCK_ICONS.length;
-                //action.putValue(Action.SMALL_ICON, CLOCK_ICONS[clockIndex]);
-            }
-        }
+    
+    /**
+     * Is the watchdog timer currently running and triggering refreshes of the logs?
+     * @return <code>true</code> if it is running, <code>false</code> otherwise.
+     */
+    public boolean isRunning() {
+        return isRunning;
     }
 
+    /**
+     * Implementation of the task to reload the model.
+     * 
+     * <p>Date: May 26, 2005</p>
+     * <p>Time: 2:04:38 PM</p>
+     * @author <a href="mailto:[email protected]">Hendrik Schreiber</a>
+     */
     private class ModelReloader extends TimerTask {
         public void run() {
             try {
@@ -81,8 +97,6 @@ public void run() {
                     SwingUtilities.invokeAndWait(new Runnable() {
                         public void run() {
                             gcDocument.relayout();
-                            //gcDocument.scrollToRightEdge();
-                            //gcDocument.repaint();
                         }
                     });
                 }