You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by pm...@apache.org on 2012/09/04 13:43:26 UTC

svn commit: r1380575 - in /jmeter/trunk/src/reports/org/apache/jmeter: JMeterReport.java gui/ReportMainFrame.java

Author: pmouawad
Date: Tue Sep  4 11:43:25 2012
New Revision: 1380575

URL: http://svn.apache.org/viewvc?rev=1380575&view=rev
Log:
Remove unused parameter
Make field final

Modified:
    jmeter/trunk/src/reports/org/apache/jmeter/JMeterReport.java
    jmeter/trunk/src/reports/org/apache/jmeter/gui/ReportMainFrame.java

Modified: jmeter/trunk/src/reports/org/apache/jmeter/JMeterReport.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/reports/org/apache/jmeter/JMeterReport.java?rev=1380575&r1=1380574&r2=1380575&view=diff
==============================================================================
--- jmeter/trunk/src/reports/org/apache/jmeter/JMeterReport.java (original)
+++ jmeter/trunk/src/reports/org/apache/jmeter/JMeterReport.java Tue Sep  4 11:43:25 2012
@@ -194,7 +194,7 @@ public class JMeterReport implements JMe
         treeLis.setActionHandler(ReportActionRouter.getInstance());
         ReportGuiPackage.getInstance(treeLis, treeModel);
         org.apache.jmeter.gui.ReportMainFrame main =
-            new org.apache.jmeter.gui.ReportMainFrame(ReportActionRouter.getInstance(),
+            new org.apache.jmeter.gui.ReportMainFrame(
                 treeModel, treeLis);
         ComponentUtil.centerComponentInWindow(main, 80);
         main.setVisible(true);

Modified: jmeter/trunk/src/reports/org/apache/jmeter/gui/ReportMainFrame.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/reports/org/apache/jmeter/gui/ReportMainFrame.java?rev=1380575&r1=1380574&r2=1380575&view=diff
==============================================================================
--- jmeter/trunk/src/reports/org/apache/jmeter/gui/ReportMainFrame.java (original)
+++ jmeter/trunk/src/reports/org/apache/jmeter/gui/ReportMainFrame.java Tue Sep  4 11:43:25 2012
@@ -89,7 +89,7 @@ public class ReportMainFrame extends JFr
     //private ImageIcon runningIcon = JMeterUtils.getImage("thread.enabled.gif");
 
     /** An image which is displayed when a test is not currently running. */
-    private ImageIcon stoppedIcon = JMeterUtils.getImage("thread.disabled.gif");// $NON-NLS-1$
+    private final ImageIcon stoppedIcon = JMeterUtils.getImage("thread.disabled.gif");// $NON-NLS-1$
 
     /** The x coordinate of the last location where a component was dragged. */
     private int previousDragXLocation = 0;
@@ -116,14 +116,12 @@ public class ReportMainFrame extends JFr
     /**
      * Create a new JMeter frame.
      *
-     * @param actionHandler
-     *            this parameter is not used
      * @param treeModel
      *            the model for the test tree
      * @param treeListener
      *            the listener for the test tree
      */
-    public ReportMainFrame(ActionListener actionHandler, TreeModel treeModel,
+    public ReportMainFrame(TreeModel treeModel,
             ReportTreeListener treeListener) {
         runningIndicator = new JButton(stoppedIcon);
         runningIndicator.setMargin(new Insets(0, 0, 0, 0));