svn commit: r1035160 - in /logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw: ./ color/ dnd/ favourites/ help/ icons/ layout/ prefs/ receivers/ zeroconf/

[email protected]
Newsgroups gmane.comp.apache.logging
Message-ID <[email protected]>
Author: sdeboy
Date: Mon Nov 15 07:17:14 2010
New Revision: 1035160

URL: http://svn.apache.org/viewvc?rev=1035160&view=rev
Log:
PMD cleanup and removing unused favourites package & classes

Removed:
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/favourites/
Modified:
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawToolBarAndMenus.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ExpressionRuleContext.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogFilePatternLayoutBuilder.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelLoggerTreeModel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogUI.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggingEventWrapper.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/WelcomePanel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/color/ColorPanel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/dnd/FileDnDTarget.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/help/HelpManager.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/icons/LineIconFactory.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/MRUFileList.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/SettingsManager.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfDeviceModel.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java
    logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPreferenceModel.java

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawCyclicBufferTableModel.java Mon Nov 15 07:17:14 2010
@@ -17,7 +17,6 @@
 
 package org.apache.log4j.chainsaw;
 
-import java.awt.EventQueue;
 import java.beans.PropertyChangeEvent;
 import java.beans.PropertyChangeListener;
 import java.beans.PropertyChangeSupport;

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawToolBarAndMenus.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawToolBarAndMenus.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawToolBarAndMenus.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ChainsawToolBarAndMenus.java Mon Nov 15 07:17:14 2010
@@ -251,7 +251,7 @@ class ChainsawToolBarAndMenus implements
         new AbstractAction("Scroll to top") {
           public void actionPerformed(ActionEvent e) {
             if (logui.getCurrentLogPanel() != null) {
-              logui.getCurrentLogPanel().scrollToTop();;
+              logui.getCurrentLogPanel().scrollToTop();
             }
           }
         };

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ExpressionRuleContext.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ExpressionRuleContext.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ExpressionRuleContext.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/ExpressionRuleContext.java Mon Nov 15 07:17:14 2010
@@ -28,7 +28,6 @@ import javax.swing.DefaultListModel;
 import javax.swing.JList;
 import javax.swing.JPopupMenu;
 import javax.swing.JScrollPane;
-import javax.swing.JTextField;
 import javax.swing.ListModel;
 import javax.swing.text.JTextComponent;
 

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogFilePatternLayoutBuilder.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogFilePatternLayoutBuilder.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogFilePatternLayoutBuilder.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogFilePatternLayoutBuilder.java Mon Nov 15 07:17:14 2010
@@ -133,7 +133,7 @@ public class LogFilePatternLayoutBuilder
             } else if (converter instanceof FileLocationPatternConverter) {
                 buffer.append("FILE");
             } else if (converter instanceof PropertiesPatternConverter) {
-                PropertiesPatternConverter propertiesConverter = (PropertiesPatternConverter) converter;
+//                PropertiesPatternConverter propertiesConverter = (PropertiesPatternConverter) converter;
 //                String option = propertiesConverter.getOption();
 //                if (option != null && option.length() > 0) {
 //                    buffer.append("PROP(" + option + ")");

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanel.java Mon Nov 15 07:17:14 2010
@@ -3851,7 +3851,7 @@ public class LogPanel extends DockablePa
 
     private class EventTimeDeltaMatchThumbnail extends AbstractEventMatchThumbnail {
         public EventTimeDeltaMatchThumbnail() {
-            super("timedelta");
+            super();
             initializeLists();
         }
 
@@ -3931,7 +3931,7 @@ public class LogPanel extends DockablePa
   //a listener receiving color updates needs to call configureColors on this class
     private class ColorizedEventAndSearchMatchThumbnail extends AbstractEventMatchThumbnail {
         public ColorizedEventAndSearchMatchThumbnail() {
-            super("colors");
+            super();
             configureColors();
         }
 
@@ -4042,7 +4042,7 @@ public class LogPanel extends DockablePa
         protected List secondaryList = new ArrayList();
         protected final int maxEventHeight = 6;
 
-        AbstractEventMatchThumbnail(final String name) {
+        AbstractEventMatchThumbnail() {
             super();
             addMouseMotionListener(new MouseMotionAdapter() {
               public void mouseMoved(MouseEvent e) {

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelLoggerTreeModel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelLoggerTreeModel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelLoggerTreeModel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelLoggerTreeModel.java Mon Nov 15 07:17:14 2010
@@ -33,7 +33,6 @@ import javax.swing.tree.MutableTreeNode;
 
 import org.apache.log4j.LogManager;
 import org.apache.log4j.Logger;
-import org.apache.log4j.chainsaw.helper.SwingHelper;
 
 
 /**
@@ -186,10 +185,6 @@ outerFor: 
         public int compare(Object o1, Object o2) {
           return o1.toString().compareToIgnoreCase(o2.toString());
         }
-
-        public boolean equals(Object obj) {
-          return false;
-        }
       };
 
     private LogPanelTreeNode(String logName) {

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java Mon Nov 15 07:17:14 2010
@@ -33,8 +33,6 @@ import java.util.Properties;
 
 import javax.swing.table.TableColumn;
 
-import org.apache.log4j.LogManager;
-import org.apache.log4j.Logger;
 import org.apache.log4j.chainsaw.prefs.SettingsManager;
 import org.apache.log4j.helpers.Constants;
 
@@ -46,7 +44,6 @@ import org.apache.log4j.helpers.Constant
 public class LogPanelPreferenceModel implements Serializable{
   public static final String ISO8601 = "ISO8601";
   public static final Collection DATE_FORMATS;
-  private static final Logger logger = LogManager.getLogger(LogPanelPreferenceModel.class);
 
  private static final long serialVersionUID = 7526472295622776147L;
   static {

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogUI.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogUI.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogUI.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LogUI.java Mon Nov 15 07:17:14 2010
@@ -84,7 +84,6 @@ import javax.swing.event.ChangeListener;
 import javax.swing.event.EventListenerList;
 import javax.swing.event.HyperlinkEvent;
 import javax.swing.event.HyperlinkListener;
-import javax.xml.parsers.ParserConfigurationException;
 
 import org.apache.log4j.Appender;
 import org.apache.log4j.AppenderSkeleton;
@@ -127,7 +126,6 @@ import org.apache.log4j.spi.LoggingEvent
 import org.apache.log4j.spi.RepositorySelector;
 import org.apache.log4j.xml.DOMConfigurator;
 import org.apache.log4j.xml.XMLDecoder;
-import org.xml.sax.SAXException;
 
 
 /**
@@ -1815,8 +1813,6 @@ public class LogUI extends JFrame implem
 
     if (selectedTab instanceof LogPanel) {
       return (LogPanel) selectedTab;
-    } else {
-      //      System.out.println(selectedTab);
     }
 
     return null;
@@ -1856,33 +1852,6 @@ public class LogUI extends JFrame implem
   }
 
   /**
-   * Changes the currently used Look And Feel of the App
-   *
-   * @param lookAndFeelClassName
-   *                    The FQN of the LookANdFeel
-   */
-  private static void applyLookAndFeel(String lookAndFeelClassName) {
-    UIManager.put("swing.boldMetal", Boolean.FALSE);
-    if (
-      UIManager.getLookAndFeel().getClass().getName().equals(
-          lookAndFeelClassName)) {
-      return;
-    }
-
-    if (
-      (lookAndFeelClassName == null) || lookAndFeelClassName.trim().equals("")) {
-      lookAndFeelClassName = UIManager.getSystemLookAndFeelClassName();
-    }
-
-    try {
-      UIManager.setLookAndFeel(lookAndFeelClassName);
-
-    } catch (Exception e) {
-      e.printStackTrace();
-    }
-  }
-
-  /**
    * Causes the Welcome Panel to become visible, and shows the URL specified as
    * it's contents
    *

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggerNameTreePanel.java Mon Nov 15 07:17:14 2010
@@ -159,7 +159,6 @@ final class LoggerNameTreePanel extends 
   private final LogPanel logPanel;
   private final RuleColorizer colorizer;
   private Rule ignoreExpressionRule;
-  private FilterModel filterModel;
   private boolean expandRootLatch = false;
   private String currentlySelectedLoggerName;
 
@@ -177,7 +176,6 @@ final class LoggerNameTreePanel extends 
     this.preferenceModel = preferenceModel;
     this.logPanel = logPanel;
     this.colorizer = colorizer;
-    this.filterModel = filterModel;
 
     setLayout(new BorderLayout());
     ignoreExpressionEntryField.setPreferredSize(new Dimension(300, 150));
@@ -1831,11 +1829,6 @@ final class LoggerNameTreePanel extends 
         {
             super.firePropertyChange(propertyName, oldVal, newVal);
         }
-
-        public void firePropertyChange(PropertyChangeEvent evt)
-        {
-            super.firePropertyChange(evt);
-        }
     }
 
 }

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggingEventWrapper.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggingEventWrapper.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggingEventWrapper.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/LoggingEventWrapper.java Mon Nov 15 07:17:14 2010
@@ -18,7 +18,6 @@ package org.apache.log4j.chainsaw;
 
 import java.awt.Color;
 import java.util.HashMap;
-import java.util.Iterator;
 import java.util.Map;
 import java.util.Set;
 
@@ -67,10 +66,8 @@ public class LoggingEventWrapper {
 
   public void setProperty(String propName, String propValue) {
     loggingEvent.setProperty(propName, propValue);
-    if (id == 0) {
-      if (propName.equals(Constants.LOG4J_ID_KEY)) {
-        id = Integer.parseInt(propValue);
-      }
+    if (id == 0 && propName.equals(Constants.LOG4J_ID_KEY)) {
+      id = Integer.parseInt(propValue);
     }
     if (syncWrapper != null && !propName.equals(ChainsawConstants.MILLIS_DELTA_COL_NAME_LOWERCASE)) {
       syncWrapper.getLoggingEvent().setProperty(propName, propValue);

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/TableColorizingRenderer.java Mon Nov 15 07:17:14 2010
@@ -22,11 +22,6 @@ import java.awt.Color;
 import java.awt.Component;
 import java.awt.Dimension;
 import java.awt.Toolkit;
-import java.awt.font.FontRenderContext;
-import java.awt.font.LineBreakMeasurer;
-import java.awt.font.TextLayout;
-import java.text.AttributedCharacterIterator;
-import java.text.AttributedString;
 import java.text.DateFormat;
 import java.text.SimpleDateFormat;
 import java.util.Date;
@@ -443,11 +438,9 @@ public class TableColorizingRenderer ext
     /**
      * Colourize background based on row striping if the event still has default foreground and background color
      */
-    if (background.equals(ChainsawConstants.COLOR_DEFAULT_BACKGROUND) && foreground.equals(ChainsawConstants.COLOR_DEFAULT_FOREGROUND)) {
-      if ((row % 2) != 0) {
-        background = applicationPreferenceModel.getAlternatingColorBackgroundColor();
-        foreground = applicationPreferenceModel.getAlternatingColorForegroundColor();
-      }
+    if (background.equals(ChainsawConstants.COLOR_DEFAULT_BACKGROUND) && foreground.equals(ChainsawConstants.COLOR_DEFAULT_FOREGROUND) && (row % 2) != 0) {
+      background = applicationPreferenceModel.getAlternatingColorBackgroundColor();
+      foreground = applicationPreferenceModel.getAlternatingColorForegroundColor();
     }
 
     component.setBackground(background);
@@ -630,6 +623,7 @@ public class TableColorizingRenderer ext
     useRelativeTimesToPrevious = false;
   }
 
+  /*
    private int calculateHeight(String string, int width, Map paramMap) {
      if (string.trim().length() == 0) {
          return ChainsawConstants.DEFAULT_ROW_HEIGHT;
@@ -646,6 +640,7 @@ public class TableColorizingRenderer ext
      }
      return Math.max(ChainsawConstants.DEFAULT_ROW_HEIGHT, (int) height);
     }
+    */
 
     private void setHighlightAttributesInternal(Object matchSet, StyledDocument styledDocument) {
         if (!highlightSearchMatchText) {

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/WelcomePanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/WelcomePanel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/WelcomePanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/WelcomePanel.java Mon Nov 15 07:17:14 2010
@@ -20,7 +20,6 @@ package org.apache.log4j.chainsaw;
 import java.awt.BorderLayout;
 import java.awt.Color;
 import java.awt.Dimension;
-import java.awt.Font;
 import java.awt.event.ActionEvent;
 import java.io.IOException;
 import java.net.URL;
@@ -36,7 +35,6 @@ import javax.swing.JPanel;
 import javax.swing.JScrollPane;
 import javax.swing.JToolBar;
 import javax.swing.SwingUtilities;
-import javax.swing.UIManager;
 import javax.swing.event.HyperlinkEvent;
 import javax.swing.event.HyperlinkListener;
 
@@ -64,7 +62,6 @@ public class WelcomePanel extends JPanel
 
     if (helpURL != null) {
       textInfo.setBorder(BorderFactory.createEmptyBorder(0, 5, 0, 5));
-      Font font = UIManager.getFont("Label.font");
 
       JScrollPane pane = new JScrollPane(textInfo);
       pane.setBorder(null);

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/color/ColorPanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/color/ColorPanel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/color/ColorPanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/color/ColorPanel.java Mon Nov 15 07:17:14 2010
@@ -309,10 +309,8 @@ public class ColorPanel extends JPanel
       }
       for (Iterator iter = allLogPanelColorizers.entrySet().iterator();iter.hasNext();) {
           Map.Entry entry = (Map.Entry)iter.next();
-          if (!entry.getValue().equals(currentLogPanelColorizer)) {
-              if (logPanelColorizersModel.getIndexOf(entry.getKey()) == -1) {
-                logPanelColorizersModel.addElement(entry.getKey());
-              }
+          if (!entry.getValue().equals(currentLogPanelColorizer) && (logPanelColorizersModel.getIndexOf(entry.getKey()) == -1)) {
+              logPanelColorizersModel.addElement(entry.getKey());
           }
       }
       //update search and alternating colors, since they may have changed from another color panel

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/dnd/FileDnDTarget.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/dnd/FileDnDTarget.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/dnd/FileDnDTarget.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/dnd/FileDnDTarget.java Mon Nov 15 07:17:14 2010
@@ -55,7 +55,6 @@ public class FileDnDTarget implements Dr
 
     protected int acceptableActions = DnDConstants.ACTION_COPY;
 
-    private DropTarget dropTarget;
     private List fileList;
 
     private JComponent guiTarget;
@@ -68,7 +67,6 @@ public class FileDnDTarget implements Dr
      */
     public FileDnDTarget(JComponent c) {
         this.guiTarget = c;
-        dropTarget = new DropTarget(this.guiTarget, this);
     }
     
     public void addDropTargetToComponent(JComponent c){

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/help/HelpManager.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/help/HelpManager.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/help/HelpManager.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/help/HelpManager.java Mon Nov 15 07:17:14 2010
@@ -74,14 +74,6 @@ public final class HelpManager {
     }
 
     /**
-     * @return URL
-     */
-    public final URL getHelpURL() {
-
-        return helpURL;
-    }
-
-    /**
      * The current Help URL that should be displayed, and is
      * a PropertyChangeListener supported property.
      *
@@ -90,7 +82,7 @@ public final class HelpManager {
      * of the event will be null)
      * @param helpURL
      */
-    public final void setHelpURL(URL helpURL) {
+    public void setHelpURL(URL helpURL) {
         this.helpURL = helpURL;
         firePropertyChange("helpURL", null, this.helpURL);
     }

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/icons/LineIconFactory.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/icons/LineIconFactory.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/icons/LineIconFactory.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/icons/LineIconFactory.java Mon Nov 15 07:17:14 2010
@@ -48,7 +48,7 @@ public final class LineIconFactory {
   private LineIconFactory() {
   }
 
-  public static final Icon createExpandIcon() {
+  public static Icon createExpandIcon() {
       int size = 8;
       int xOffSet = 0;
       int yOffSet = 0;
@@ -80,7 +80,7 @@ public final class LineIconFactory {
 
     return null;
   }
-  public static final Icon createCollapseIcon() {
+  public static Icon createCollapseIcon() {
       int size = 8;
       int xOffSet = 0;
       int yOffSet = 0;
@@ -109,11 +109,11 @@ public final class LineIconFactory {
     return null;
   }
 
-  public static final Icon createCloseIcon() {
+  public static Icon createCloseIcon() {
     return new CloseIcon(8, 0, 0);
   }
 
-  public static final Icon createBlankIcon() {
+  public static Icon createBlankIcon() {
     return new BlankIcon(16);
   }
 

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java Mon Nov 15 07:17:14 2010
@@ -177,7 +177,6 @@ public class EventDetailLayout extends L
    * @return new LoggingEvent
    */
   private static LoggingEvent copyForHTML(LoggingEvent event) {
-    String fqnCategory = escape(event.getFQNOfLoggerClass());
     Logger logger = Logger.getLogger(event.getLoggerName());
     String threadName = event.getThreadName();
     Object msg = escape(event.getMessage().toString());

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/MRUFileList.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/MRUFileList.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/MRUFileList.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/MRUFileList.java Mon Nov 15 07:17:14 2010
@@ -33,7 +33,6 @@ import com.thoughtworks.xstream.io.xml.D
 public class MRUFileList{
 
     private static MRUFileList log4jList = new MRUFileList();
-    private static MRUFileList jdk14List = new MRUFileList();
     private static final int DEFAULT_MRU_SIZE = 5;
     
     private List fileList = new ArrayList();

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/SettingsManager.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/SettingsManager.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/SettingsManager.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/prefs/SettingsManager.java Mon Nov 15 07:17:14 2010
@@ -79,7 +79,7 @@ public final class SettingsManager {
      * Returns the singleton instance of the SettingsManager
      * @return settings manager
      */
-    public static final SettingsManager getInstance() {
+    public static SettingsManager getInstance() {
         return instance;
     }
 

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/receivers/PluginPropertyEditorPanel.java Mon Nov 15 07:17:14 2010
@@ -18,7 +18,6 @@ package org.apache.log4j.chainsaw.receiv
 
 import java.awt.BorderLayout;
 import java.awt.Component;
-import java.awt.Dimension;
 import java.awt.event.WindowAdapter;
 import java.awt.event.WindowEvent;
 import java.beans.BeanInfo;

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfDeviceModel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfDeviceModel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfDeviceModel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfDeviceModel.java Mon Nov 15 07:17:14 2010
@@ -23,11 +23,8 @@ import java.util.List;
 import javax.jmdns.ServiceEvent;
 import javax.jmdns.ServiceInfo;
 import javax.jmdns.ServiceListener;
-import javax.swing.ImageIcon;
 import javax.swing.table.AbstractTableModel;
 
-import org.apache.log4j.chainsaw.icons.ChainsawIcons;
-
 public class ZeroConfDeviceModel extends AbstractTableModel implements ServiceListener {
 
     private List deviceList = new ArrayList();

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java Mon Nov 15 07:17:14 2010
@@ -18,7 +18,6 @@ package org.apache.log4j.chainsaw.zeroco
 
 import java.awt.BorderLayout;
 import java.awt.Component;
-import java.awt.Container;
 import java.awt.event.ActionEvent;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
@@ -36,7 +35,6 @@ import javax.jmdns.ServiceEvent;
 import javax.jmdns.ServiceInfo;
 import javax.jmdns.ServiceListener;
 import javax.swing.AbstractAction;
-import javax.swing.Icon;
 import javax.swing.ImageIcon;
 import javax.swing.JFrame;
 import javax.swing.JMenu;
@@ -44,7 +42,6 @@ import javax.swing.JMenuBar;
 import javax.swing.JMenuItem;
 import javax.swing.JPopupMenu;
 import javax.swing.JScrollPane;
-import javax.swing.JTabbedPane;
 import javax.swing.JTable;
 import javax.swing.JToolBar;
 import javax.swing.SwingUtilities;
@@ -93,8 +90,6 @@ public class ZeroConfPlugin extends GUIP
 
     private static final Logger LOG = Logger.getLogger(ZeroConfPlugin.class);
 
-    private static final Icon DEVICE_DISCOVERED_ICON = new ImageIcon(ChainsawIcons.ANIM_RADIO_TOWER);
-
     private ZeroConfDeviceModel discoveredDevices = new ZeroConfDeviceModel();
 
     private JTable deviceTable = new JTable(discoveredDevices);
@@ -226,21 +221,6 @@ public class ZeroConfPlugin extends GUIP
     }
 
     /**
-     * Sets the icon of this parent container (a JTabbedPane, we hope
-     *
-     */
-    private void setIconIfNeeded() {
-        Container container = this.getParent();
-        if(container instanceof JTabbedPane) {
-            JTabbedPane tabbedPane = (JTabbedPane) container;
-            Icon icon = discoveredDevices.getRowCount()==0?null:DEVICE_DISCOVERED_ICON;
-            tabbedPane.setIconAt(tabbedPane.indexOfTab(getName()), icon);
-        }else {
-            LOG.warn("Parent is not a TabbedPane, not setting icon: " + container.getClass().getName());
-        }
-    }
-
-    /**
      * Attempts to find a JFrame container as a parent,and addse a "Connect to" menu
      *
      */

Modified: logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPreferenceModel.java
URL: http://svn.apache.org/viewvc/logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPreferenceModel.java?rev=1035160&r1=1035159&r2=1035160&view=diff
==============================================================================
--- logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPreferenceModel.java (original)
+++ logging/chainsaw/trunk/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPreferenceModel.java Mon Nov 15 07:17:14 2010
@@ -16,7 +16,6 @@
  */
 package org.apache.log4j.chainsaw.zeroconf;
 
-import java.beans.PropertyChangeSupport;
 import java.util.ArrayList;
 import java.util.HashSet;
 import java.util.List;
@@ -27,7 +26,6 @@ public class ZeroConfPreferenceModel {
     private List monitoredZones = new ArrayList();
     private Set autoConnectDevices = new HashSet();
     
-    private transient PropertyChangeSupport propertySupport = new PropertyChangeSupport(this);
 //   TODO expose addPropertyChangeListener
     
     public void addAutoConnectDevice(String deviceName) {
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.