svn commit: r1776765 - in /jmeter/trunk/src/components/org/apache/jmeter/visualizers: Graph.java PropertyControlGui.java RenderAsCssJQuery.java

[pm...@apache.org]

Author: pmouawad
Date: Sat Dec 31 14:52:33 2016
New Revision: 1776765

URL: http://svn.apache.org/viewvc?rev=1776765&view=rev
Log:
Sonar : fix squid:CommentedOutCodeLine "This block of commented-out lines of code should be removed. "

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java
    jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsCssJQuery.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java?rev=1776765&r1=1776764&r2=1776765&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/Graph.java Sat Dec 31 14:52:33 2016
@@ -85,7 +85,6 @@ public class Graph extends JComponent im
     @Override
     public Dimension getPreferredScrollableViewportSize() {
         return this.getPreferredSize();
-        // return new Dimension(width, 400);
     }
 
     /**
@@ -208,7 +207,6 @@ public class Graph extends JComponent im
     }
 
     private void drawSample(long x, Sample oneSample, Graphics g) {
-        // int width = getWidth();
         int height = getHeight();
         log.debug("Drawing a sample at " + x);
         int adjustedWidth = (int)(x % WIDTH); // will always be within range of an int: as must be < width

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java?rev=1776765&r1=1776764&r2=1776765&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java Sat Dec 31 14:52:33 2016
@@ -33,10 +33,8 @@ import java.util.Map;
 import java.util.Properties;
 import java.util.Set;
 
-//import javax.swing.BorderFactory;
 import javax.swing.Box;
 import javax.swing.ButtonGroup;
-//import javax.swing.JButton;
 import javax.swing.JCheckBox;
 import javax.swing.JLabel;
 import javax.swing.JPanel;

Modified: jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsCssJQuery.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsCssJQuery.java?rev=1776765&r1=1776764&r2=1776765&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsCssJQuery.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/visualizers/RenderAsCssJQuery.java Sat Dec 31 14:52:33 2016
@@ -78,13 +78,10 @@ public class RenderAsCssJQuery implement
     @Override
     public void clearData() {
         this.cssJqueryDataField.setText(""); // $NON-NLS-1$
-        // don't set empty to keep cssJquery
-        // cssJqueryField.setText(""); // $NON-NLS-1$
         this.cssJqueryResultField.setText(""); // $NON-NLS-1$
-        // don't set empty to keep attribute
-        // this.attributeField.setText(""); // $NON-NLS-1$
+        // don't set cssJqueryField to empty to keep it
+        // don't set attribute to empty to keep it 
         // don't change impl
-        // this.cssJqueryLabeledChoice.setText(HtmlExtractor.DEFAULT_EXTRACTOR);
     }
 
     /** {@inheritDoc} */