Author: fschumacher
Date: Sat Jan 31 10:59:42 2015
New Revision: 1656177

URL: http://svn.apache.org/r1656177
Log:
Bug: 57110
Reformat code, no functional change. Remove empty javadoc comments.
Bugzilla Id: 57110

Modified:
    
jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java
    
jmeter/trunk/src/components/org/apache/jmeter/visualizers/StatVisualizer.java
    jmeter/trunk/src/core/org/apache/jmeter/config/gui/RowDetailDialog.java
    jmeter/trunk/src/core/org/apache/jmeter/control/gui/TestPlanGui.java
    jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddToTree.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/Load.java
    jmeter/trunk/src/core/org/apache/jmeter/gui/action/SearchTreeDialog.java
    
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/AccessLogSampler.java
    jmeter/trunk/test/src/org/apache/jmeter/visualizers/GenerateTreeGui.java

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=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java
 (original)
+++ 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/PropertyControlGui.java
 Sat Jan 31 10:59:42 2015
@@ -53,8 +53,8 @@ import org.apache.jmeter.util.JMeterUtil
 import org.apache.jorphan.gui.ObjectTableModel;
 import org.apache.jorphan.reflect.Functor;
 
-public class PropertyControlGui extends AbstractConfigGui
-    implements ActionListener, UnsharedComponent {
+public class PropertyControlGui extends AbstractConfigGui implements
+        ActionListener, UnsharedComponent {
 
     private static final long serialVersionUID = 1L;
 
@@ -72,7 +72,7 @@ public class PropertyControlGui extends
     private static final String JMETER = "jmeter"; // $NON-NLS-1$
 
     private final JCheckBox systemButton = new JCheckBox("System");
-    
+
     private final JCheckBox jmeterButton = new JCheckBox("JMeter");
 
     private final JLabel tableLabel = new JLabel("Properties");
@@ -138,7 +138,7 @@ public class PropertyControlGui extends
 
     private void setUpData(){
         tableModel.clearData();
-        Properties p=null;
+        Properties p = null;
         if (systemButton.isSelected()){
             p = System.getProperties();
         }
@@ -153,14 +153,14 @@ public class PropertyControlGui extends
         Collections.sort(al, new Comparator<Map.Entry<Object, Object>>(){
             @Override
             public int compare(Map.Entry<Object, Object> o1, Map.Entry<Object, 
Object> o2) {
-                String m1,m2;
-                m1=(String)o1.getKey();
-                m2=(String)o2.getKey();
+                String m1, m2;
+                m1 = (String) o1.getKey();
+                m2 = (String) o2.getKey();
                 return m1.compareTo(m2);
             }
         });
         Iterator<Map.Entry<Object, Object>> i = al.iterator();
-        while(i.hasNext()){
+        while (i.hasNext()) {
             tableModel.addRow(i.next());
         }
 
@@ -170,6 +170,7 @@ public class PropertyControlGui extends
     public void modifyTestElement(TestElement element) {
         configureTestElement(element);
     }
+
     private Component makeMainPanel() {
         initializeTableModel();
         table = new JTable(tableModel);
@@ -228,10 +229,10 @@ public class PropertyControlGui extends
     private void init() {
         JPanel p = this;
 
-            setLayout(new BorderLayout(0, 5));
-            setBorder(makeBorder());
-            add(makeTitlePanel(), BorderLayout.NORTH);
-            p = new JPanel();
+        setLayout(new BorderLayout(0, 5));
+        setBorder(makeBorder());
+        add(makeTitlePanel(), BorderLayout.NORTH);
+        p = new JPanel();
 
         p.setLayout(new BorderLayout());
 
@@ -247,12 +248,13 @@ public class PropertyControlGui extends
     private void initializeTableModel() {
         tableModel = new ObjectTableModel(new String[] { COLUMN_NAMES_0, 
COLUMN_NAMES_1 },
                 new Functor[] {
-                new Functor(Map.Entry.class, "getKey"), // $NON-NLS-1$
-                new Functor(Map.Entry.class, "getValue") },  // $NON-NLS-1$
+                    new Functor(Map.Entry.class, "getKey"), // $NON-NLS-1$
+                    new Functor(Map.Entry.class, "getValue") // $NON-NLS-1$
+                },
                 new Functor[] {
-                null, //new Functor("setName"), // $NON-NLS-1$
-                new Functor(Map.Entry.class,"setValue", new Class[] { 
Object.class }) // $NON-NLS-1$
-            },
+                    null, //new Functor("setName"), // $NON-NLS-1$
+                    new Functor(Map.Entry.class,"setValue", new Class[] { 
Object.class }) // $NON-NLS-1$
+                },
                 new Class[] { String.class, String.class });
     }
 }

Modified: 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/StatVisualizer.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/visualizers/StatVisualizer.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/StatVisualizer.java 
(original)
+++ 
jmeter/trunk/src/components/org/apache/jmeter/visualizers/StatVisualizer.java 
Sat Jan 31 10:59:42 2015
@@ -63,23 +63,24 @@ public class StatVisualizer extends Abst
 
     private static final String USE_GROUP_NAME = "useGroupName"; //$NON-NLS-1$
 
-    private static final String SAVE_HEADERS   = "saveHeaders"; //$NON-NLS-1$
+    private static final String SAVE_HEADERS = "saveHeaders"; //$NON-NLS-1$
 
-    private final String TOTAL_ROW_LABEL
-        = JMeterUtils.getResString("aggregate_report_total_label");  
//$NON-NLS-1$
+    private final String TOTAL_ROW_LABEL = JMeterUtils
+            .getResString("aggregate_report_total_label"); //$NON-NLS-1$
 
     private JTable myJTable;
 
     private JScrollPane myScrollPane;
 
-    private final JButton saveTable =
-        new JButton(JMeterUtils.getResString("aggregate_graph_save_table"));   
         //$NON-NLS-1$
+    private final JButton saveTable = new JButton(
+            JMeterUtils.getResString("aggregate_graph_save_table")); 
//$NON-NLS-1$
 
-    private final JCheckBox saveHeaders = // should header be saved with the 
data?
-        new 
JCheckBox(JMeterUtils.getResString("aggregate_graph_save_table_header"),true);  
  //$NON-NLS-1$
+    // should header be saved with the data?
+    private final JCheckBox saveHeaders = new JCheckBox(
+            JMeterUtils.getResString("aggregate_graph_save_table_header"), 
true); //$NON-NLS-1$
 
-    private final JCheckBox useGroupName =
-        new 
JCheckBox(JMeterUtils.getResString("aggregate_graph_use_group_name"));          
  //$NON-NLS-1$
+    private final JCheckBox useGroupName = new JCheckBox(
+            JMeterUtils.getResString("aggregate_graph_use_group_name")); 
//$NON-NLS-1$
 
     private transient ObjectTableModel model;
 
@@ -138,7 +139,7 @@ public class StatVisualizer extends Abst
                 synchronized(tot) {
                     tot.addSample(res);
                 }
-                model.fireTableDataChanged();                
+                model.fireTableDataChanged();
             }
         });
     }

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/config/gui/RowDetailDialog.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/config/gui/RowDetailDialog.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/config/gui/RowDetailDialog.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/config/gui/RowDetailDialog.java Sat 
Jan 31 10:59:42 2015
@@ -76,13 +76,13 @@ public class RowDetailDialog extends JDi
     private JButton nextButton;
 
     private JButton previousButton;
-    
+
     private JButton closeButton;
 
     private ObjectTableModel tableModel;
 
     private int selectedRow;
-    
+
     private boolean textChanged = true; // change to false after the first 
insert
 
 
@@ -102,9 +102,7 @@ public class RowDetailDialog extends JDi
         JRootPane rootPane = new JRootPane();
         // Hide Window on ESC
         Action escapeAction = new AbstractAction("ESCAPE") {
-            /**
-             *
-             */
+
             private static final long serialVersionUID = -8699034338969407625L;
 
             @Override
@@ -114,9 +112,7 @@ public class RowDetailDialog extends JDi
         };
         // Do update on Enter
         Action enterAction = new AbstractAction("ENTER") {
-            /**
-             *
-             */
+
             private static final long serialVersionUID = -1529005452976176873L;
 
             @Override
@@ -198,22 +194,22 @@ public class RowDetailDialog extends JDi
     @Override
     public void actionPerformed(ActionEvent e) {
         String action = e.getActionCommand();
-        if(action.equals(CLOSE)) {
+        if (action.equals(CLOSE)) {
             this.setVisible(false);
         }
-        else if(action.equals(NEXT)) {
+        else if (action.equals(NEXT)) {
             selectedRow++;
             previousButton.setEnabled(true);
             nextButton.setEnabled(selectedRow < tableModel.getRowCount()-1);
             setValues(selectedRow);
         }
-        else if(action.equals(PREVIOUS)) {
+        else if (action.equals(PREVIOUS)) {
             selectedRow--;
             nextButton.setEnabled(true);
             previousButton.setEnabled(selectedRow > 0);
             setValues(selectedRow);
         }
-        else if(action.equals(UPDATE)) {
+        else if (action.equals(UPDATE)) {
             doUpdate(e);
         }
     }
@@ -265,5 +261,5 @@ public class RowDetailDialog extends JDi
     public void changedUpdate(DocumentEvent e) {
         changeLabelButton();
     }
-    
+
 }

Modified: jmeter/trunk/src/core/org/apache/jmeter/control/gui/TestPlanGui.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/control/gui/TestPlanGui.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/control/gui/TestPlanGui.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/control/gui/TestPlanGui.java Sat 
Jan 31 10:59:42 2015
@@ -158,14 +158,14 @@ public class TestPlanGui extends Abstrac
         super.configure(el);
         if (el instanceof TestPlan) {
             TestPlan tp = (TestPlan) el;
-        functionalMode.setSelected(tp.isFunctionalMode());
-        serializedMode.setSelected(tp.isSerialized());
-        tearDownOnShutdown.setSelected(tp.isTearDownOnShutdown());
-        final JMeterProperty udv = tp.getUserDefinedVariablesAsProperty();
-        if (udv != null) {
-            argsPanel.configure((Arguments) udv.getObjectValue());
-        }
-        browseJar.setFiles(tp.getTestPlanClasspathArray());
+            functionalMode.setSelected(tp.isFunctionalMode());
+            serializedMode.setSelected(tp.isSerialized());
+            tearDownOnShutdown.setSelected(tp.isTearDownOnShutdown());
+            final JMeterProperty udv = tp.getUserDefinedVariablesAsProperty();
+            if (udv != null) {
+                argsPanel.configure((Arguments) udv.getObjectValue());
+            }
+            browseJar.setFiles(tp.getTestPlanClasspathArray());
         }
     }
 

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/functions/gui/FunctionHelper.java 
Sat Jan 31 10:59:42 2015
@@ -69,7 +69,7 @@ public class FunctionHelper extends JDia
         init();
         JMeterUtils.addLocaleChangeListener(this);
     }
-    
+
     /**
      * Allow Dialog to be closed by ESC key
      */
@@ -78,13 +78,11 @@ public class FunctionHelper extends JDia
         JRootPane rootPane = new JRootPane();
         KeyStroke stroke = KeyStrokes.ESC;
         javax.swing.Action escapeAction = new AbstractAction("ESCAPE") { 
-            /**
-             * 
-             */
+
             private static final long serialVersionUID = -4036804004190858925L;
 
             @Override
-            public void actionPerformed(ActionEvent actionEvent) { 
+            public void actionPerformed(ActionEvent actionEvent) {
                 setVisible(false);
             } 
         };
@@ -93,7 +91,7 @@ public class FunctionHelper extends JDia
         inputMap.put(stroke, escapeAction.getValue(Action.NAME));
         return rootPane;
     }
-    
+
     private void init() {
         parameterPanel = new 
ArgumentsPanel(JMeterUtils.getResString("function_params"), false); 
//$NON-NLS-1$
         initializeFunctionList();
@@ -112,7 +110,7 @@ public class FunctionHelper extends JDia
         generateButton.addActionListener(this);
         resultsPanel.add(generateButton);
         this.getContentPane().add(resultsPanel, BorderLayout.SOUTH);
-        
+
         this.pack();
         ComponentUtil.centerComponentInWindow(this);
     }

Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/MainFrame.java Sat Jan 31 
10:59:42 2015
@@ -196,6 +196,7 @@ public class MainFrame extends JFrame im
 
         totalThreads = new JLabel("0"); // $NON-NLS-1$
         
totalThreads.setToolTipText(JMeterUtils.getResString("total_threads_tooltip")); 
// $NON-NLS-1$
+
         activeThreads = new JLabel("0"); // $NON-NLS-1$
         
activeThreads.setToolTipText(JMeterUtils.getResString("active_threads_tooltip"));
 // $NON-NLS-1$
 
@@ -206,9 +207,9 @@ public class MainFrame extends JFrame im
         warnIndicator.setContentAreaFilled(false);
         warnIndicator.setBorderPainted(false);
         warnIndicator.setCursor(new Cursor(Cursor.HAND_CURSOR));
-        
         
warnIndicator.setToolTipText(JMeterUtils.getResString("error_indicator_tooltip"));
 // $NON-NLS-1$
         warnIndicator.addActionListener(this);
+
         errorsOrFatalsLabel = new JLabel("0"); // $NON-NLS-1$
         
errorsOrFatalsLabel.setToolTipText(JMeterUtils.getResString("error_indicator_tooltip"));
 // $NON-NLS-1$
 
@@ -345,7 +346,7 @@ public class MainFrame extends JFrame im
         }
         stoppingMessage = new EscapeDialog(this, 
JMeterUtils.getResString("stopping_test_title"), true); //$NON-NLS-1$
         String label = JMeterUtils.getResString("stopping_test"); //$NON-NLS-1
-        if(!StringUtils.isEmpty(host)) {
+        if (!StringUtils.isEmpty(host)) {
             label = label + JMeterUtils.getResString("stopping_test_host")+ ": 
" + host;
         }
         JLabel stopLabel = new JLabel(label); //$NON-NLS-1$$NON-NLS-2$
@@ -356,7 +357,7 @@ public class MainFrame extends JFrame im
         SwingUtilities.invokeLater(new Runnable() {
             @Override
             public void run() {
-                if (stoppingMessage != null) {// TODO - how can this be null?
+                if (stoppingMessage != null) { // TODO - how can this be null?
                     stoppingMessage.setVisible(true);
                 }
             }
@@ -409,7 +410,7 @@ public class MainFrame extends JFrame im
         activeThreads.setText("0"); // $NON-NLS-1$
         totalThreads.setText("0"); // $NON-NLS-1$
         menuBar.setRunning(true, host);
-        if(LOCAL.equals(host)) {
+        if (LOCAL.equals(host)) {
             toolbar.setLocalTestStarted(true);
         } else {
             toolbar.setRemoteTestStarted(true);
@@ -442,7 +443,7 @@ public class MainFrame extends JFrame im
             JMeterContextService.endTest();
         }
         menuBar.setRunning(false, host);
-        if(LOCAL.equals(host)) {
+        if (LOCAL.equals(host)) {
             toolbar.setLocalTestStarted(false);
         } else {
             toolbar.setRemoteTestStarted(false);
@@ -640,7 +641,7 @@ public class MainFrame extends JFrame im
         treevar.addTreeSelectionListener(treeListener);
         treevar.addMouseListener(treeListener);
         treevar.addKeyListener(treeListener);
-        
+
         // enable drag&drop, install a custom transfer handler
         treevar.setDragEnabled(true);
         treevar.setDropMode(DropMode.ON_OR_INSERT);
@@ -725,18 +726,18 @@ public class MainFrame extends JFrame im
         @SuppressWarnings("unchecked")
         List<File> files = (List<File>)
                 tr.getTransferData(DataFlavor.javaFileListFlavor);
-        if(files.isEmpty()) {
+        if (files.isEmpty()) {
             return false;
         }
         File file = files.get(0);
-        if(!file.getName().endsWith(".jmx")) {
+        if (!file.getName().endsWith(".jmx")) {
             log.warn("Importing file:" + file.getName()+ "from DnD failed 
because file extension does not end with .jmx");
             return false;
         }
 
         ActionEvent fakeEvent = new ActionEvent(this, 
ActionEvent.ACTION_PERFORMED, ActionNames.OPEN);
         LoadDraggedFile.loadProject(fakeEvent, file);
-        
+
         return true;
     }
 
@@ -781,7 +782,7 @@ public class MainFrame extends JFrame im
     @Override
     public void clearData() {
         logPanel.clear();
-        if(DISPLAY_ERROR_FATAL_COUNTER) {
+        if (DISPLAY_ERROR_FATAL_COUNTER) {
             errorsAndFatalsCounterLogTarget.clearData();
         }
     }
@@ -791,7 +792,7 @@ public class MainFrame extends JFrame im
      */
     @Override
     public void actionPerformed(ActionEvent event) {
-        if(event.getSource()==warnIndicator) {
+        if (event.getSource() == warnIndicator) {
             ActionRouter.getInstance().doActionNow(new 
ActionEvent(event.getSource(), event.getID(), 
ActionNames.LOGGER_PANEL_ENABLE_DISABLE));
         }
     }

Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddToTree.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddToTree.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddToTree.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/action/AddToTree.java Sat Jan 
31 10:59:42 2015
@@ -45,7 +45,6 @@ public class AddToTree implements Comman
         commandSet = Collections.unmodifiableSet(commands);
     }
 
-
     public AddToTree() {
     }
 
@@ -76,7 +75,7 @@ public class AddToTree implements Comman
             log.error("", err); // $NON-NLS-1$
             String msg = err.getMessage();
             if (msg == null) {
-                msg=err.toString();
+                msg = err.toString();
             }
             JMeterUtils.reportErrorToUser(msg);
         }
@@ -84,7 +83,7 @@ public class AddToTree implements Comman
             log.error("", err); // $NON-NLS-1$
             String msg = err.getMessage();
             if (msg == null) {
-                msg=err.toString();
+                msg = err.toString();
             }
             JMeterUtils.reportErrorToUser(msg);
         }

Modified: jmeter/trunk/src/core/org/apache/jmeter/gui/action/Load.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/action/Load.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/action/Load.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/action/Load.java Sat Jan 31 
10:59:42 2015
@@ -81,10 +81,10 @@ public class Load implements Command {
             return;
         }
         final File selectedFile = chooser.getSelectedFile();
-        if(selectedFile != null) {
+        if (selectedFile != null) {
             final boolean merging = 
e.getActionCommand().equals(ActionNames.MERGE);
             // We must ask the user if it is ok to close current project
-            if(!merging) { // i.e. it is OPEN
+            if (!merging) { // i.e. it is OPEN
                 if (!Close.performAction(e)) {
                     return;
                 }
@@ -121,25 +121,26 @@ public class Load implements Command {
         if (f != null) {
             InputStream reader = null;
             try {
-                    if (merging) {
-                        log.info("Merging file: " + f);
-                    } else {
-                        log.info("Loading file: " + f);
-                        // TODO should this be done even if not a full test 
plan?
-                        // and what if load fails?
-                        if(setDetails) {
-                            FileServer.getFileServer().setBaseForScript(f);
-                        }
-                    }
-                    reader = new FileInputStream(f);
-                    final HashTree tree = SaveService.loadTree(reader);
-                    final boolean isTestPlan = insertLoadedTree(e.getID(), 
tree, merging);
-    
-                    // don't change name if merging
-                    if (!merging && isTestPlan && setDetails) {
-                        // TODO should setBaseForScript be called here rather 
than above?
-                        guiPackage.setTestPlanFile(f.getAbsolutePath());
+                if (merging) {
+                    log.info("Merging file: " + f);
+                } else {
+                    log.info("Loading file: " + f);
+                    // TODO should this be done even if not a full test plan?
+                    // and what if load fails?
+                    if (setDetails) {
+                        FileServer.getFileServer().setBaseForScript(f);
                     }
+                }
+                reader = new FileInputStream(f);
+                final HashTree tree = SaveService.loadTree(reader);
+                final boolean isTestPlan = insertLoadedTree(e.getID(), tree, 
merging);
+
+                // don't change name if merging
+                if (!merging && isTestPlan && setDetails) {
+                    // TODO should setBaseForScript be called here rather than
+                    // above?
+                    guiPackage.setTestPlanFile(f.getAbsolutePath());
+                }
             } catch (NoClassDefFoundError ex) {// Allow for missing optional 
jars
                 reportError("Missing jar file", ex, true);
             } catch (ConversionException ex) {
@@ -204,7 +205,7 @@ public class Load implements Command {
         // Send different event wether we are merging a test plan into another 
test plan,
         // or loading a testplan from scratch
         ActionEvent actionEvent =
-            new ActionEvent(subTree.get(subTree.getArray()[subTree.size() - 
1]), id, 
+            new ActionEvent(subTree.get(subTree.getArray()[subTree.size() - 
1]), id,
                     merging ? ActionNames.SUB_TREE_MERGED : 
ActionNames.SUB_TREE_LOADED);
 
         ActionRouter.getInstance().actionPerformed(actionEvent);

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/gui/action/SearchTreeDialog.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/gui/action/SearchTreeDialog.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/gui/action/SearchTreeDialog.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/gui/action/SearchTreeDialog.java 
Sat Jan 31 10:59:42 2015
@@ -59,9 +59,7 @@ import org.apache.log.Logger;
  * FIXME Why is searchTF not getting focus correctly after having been 
setVisible(false) once
  */
 public class SearchTreeDialog extends JDialog implements ActionListener {
-    /**
-     *
-     */
+
     private static final long serialVersionUID = -4436834972710248247L;
 
     private static final Logger logger = LoggingManager.getLoggerForClass();
@@ -93,9 +91,7 @@ public class SearchTreeDialog extends JD
         JRootPane rootPane = new JRootPane();
         // Hide Window on ESC
         Action escapeAction = new AbstractAction("ESCAPE") {
-            /**
-             *
-             */
+
             private static final long serialVersionUID = -6543764044868772971L;
 
             @Override
@@ -105,9 +101,7 @@ public class SearchTreeDialog extends JD
         };
         // Do search on Enter
         Action enterAction = new AbstractAction("ENTER") {
-            /**
-             *
-             */
+
             private static final long serialVersionUID = -3661361497864527363L;
 
             @Override
@@ -172,7 +166,7 @@ public class SearchTreeDialog extends JD
      */
     @Override
     public void actionPerformed(ActionEvent e) {
-        if(e.getSource()==cancelButton) {
+        if (e.getSource()==cancelButton) {
             searchTF.requestFocusInWindow();
             this.setVisible(false);
             return;
@@ -186,7 +180,7 @@ public class SearchTreeDialog extends JD
     private void doSearch(ActionEvent e) {
         boolean expand = e.getSource()==searchAndExpandButton;
         String wordToSearch = searchTF.getText();
-        if(StringUtils.isEmpty(wordToSearch)) {
+        if (StringUtils.isEmpty(wordToSearch)) {
             return;
         } else {
             this.lastSearch = wordToSearch;
@@ -196,7 +190,7 @@ public class SearchTreeDialog extends JD
         ActionRouter.getInstance().doActionNow(new ActionEvent(e.getSource(), 
e.getID(), ActionNames.SEARCH_RESET));
         // do search
         Searcher searcher = null;
-        if(isRegexpCB.isSelected()) {
+        if (isRegexpCB.isSelected()) {
             searcher = new RegexpSearcher(isCaseSensitiveCB.isSelected(), 
searchTF.getText());
         } else {
             searcher = new RawTextSearcher(isCaseSensitiveCB.isSelected(), 
searchTF.getText());
@@ -211,7 +205,7 @@ public class SearchTreeDialog extends JD
                     List<JMeterTreeNode> matchingNodes = 
jMeterTreeNode.getPathToThreadGroup();
                     List<String> searchableTokens = 
searchable.getSearchableTokens();
                     boolean result = searcher.search(searchableTokens);
-                    if(result) {
+                    if (result) {
                         nodes.addAll(matchingNodes);
                     }
                 }

Modified: 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/AccessLogSampler.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/AccessLogSampler.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/AccessLogSampler.java
 (original)
+++ 
jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/sampler/AccessLogSampler.java
 Sat Jan 31 10:59:42 2015
@@ -308,11 +308,11 @@ public class AccessLogSampler extends HT
                         initFilter();
                         s.filter = (Filter) ((TestCloneable) filter).clone();
                     }
-                    
if(TestCloneable.class.isAssignableFrom(Class.forName(parserClassName)))
+                    if 
(TestCloneable.class.isAssignableFrom(Class.forName(parserClassName)))
                     {
                         instantiateParser();
                         s.PARSER = (LogParser)((TestCloneable)PARSER).clone();
-                        if(filter != null)
+                        if (filter != null)
                         {
                             s.PARSER.setFilter(s.filter);
                         }

Modified: 
jmeter/trunk/test/src/org/apache/jmeter/visualizers/GenerateTreeGui.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/test/src/org/apache/jmeter/visualizers/GenerateTreeGui.java?rev=1656177&r1=1656176&r2=1656177&view=diff
==============================================================================
--- jmeter/trunk/test/src/org/apache/jmeter/visualizers/GenerateTreeGui.java 
(original)
+++ jmeter/trunk/test/src/org/apache/jmeter/visualizers/GenerateTreeGui.java 
Sat Jan 31 10:59:42 2015
@@ -59,8 +59,8 @@ import org.apache.jmeter.util.JMeterUtil
  * Note: the code currently runs on all versions of JMeter back to 2.2.
  * Beware of making changes that rely on more recent APIs.
  */
-public class GenerateTreeGui extends AbstractConfigGui
-    implements ActionListener, UnsharedComponent {
+public class GenerateTreeGui extends AbstractConfigGui implements
+        ActionListener, UnsharedComponent {
 
     private static final long serialVersionUID = 1L;
 
@@ -119,7 +119,7 @@ public class GenerateTreeGui extends Abs
             JMeterTreeNode myTarget) {
         myTarget = addSimpleController(treeModel, myTarget, title);
         JPopupMenu jp = MenuFactory.makeMenu(menuKey, "").getPopupMenu();
-        for(Component comp : jp.getComponents()) {
+        for (Component comp : jp.getComponents()) {
             JMenuItem jmi = (JMenuItem) comp;
             try {
                 TestElement testElement = 
guiPackage.createTestElement(jmi.getName());
@@ -156,17 +156,16 @@ public class GenerateTreeGui extends Abs
         return labelPanel;
     }
 
-
     /**
      * Initialize the components and layout of this component.
      */
     private void init() {
         JPanel p = this;
 
-            setLayout(new BorderLayout(0, 5));
-            setBorder(makeBorder());
-            add(makeTitlePanel(), BorderLayout.NORTH);
-            p = new JPanel();
+        setLayout(new BorderLayout(0, 5));
+        setBorder(makeBorder());
+        add(makeTitlePanel(), BorderLayout.NORTH);
+        p = new JPanel();
 
         p.setLayout(new BorderLayout());
 
@@ -208,14 +207,14 @@ public class GenerateTreeGui extends Abs
         }
 
         volatile JMeterTreeNode newNode;
-        
+
         @Override
         public void run() {
             try {
                 newNode = model.addComponent(testElement, node);
             } catch (IllegalUserActionException e) {
                  throw new Error(e);
-            }               
+            }
         }
     }
 


Reply via email to