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 2016/12/21 17:01:06 UTC

svn commit: r1775508 - /jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java

Author: pmouawad
Date: Wed Dec 21 17:01:06 2016
New Revision: 1775508

URL: http://svn.apache.org/viewvc?rev=1775508&view=rev
Log:
Sonar : Fix code smells
Use Java8 lambda

Modified:
    jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java

Modified: jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java?rev=1775508&r1=1775507&r2=1775508&view=diff
==============================================================================
--- jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java (original)
+++ jmeter/trunk/src/components/org/apache/jmeter/sampler/DebugSampler.java Wed Dec 21 17:01:06 2016
@@ -20,8 +20,6 @@ package org.apache.jmeter.sampler;
 
 import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.Collections;
-import java.util.Comparator;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
@@ -92,21 +90,14 @@ public class DebugSampler extends Abstra
     private void formatSet(StringBuilder sb, @SuppressWarnings("rawtypes") Set s) {
         @SuppressWarnings("unchecked")
         List<Map.Entry<Object, Object>> al = new ArrayList<>(s);
-        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();
+        al.sort((Map.Entry<Object, Object> o1, Map.Entry<Object, Object> o2) -> {
+                String m1 = (String)o1.getKey();
+                String m2 = (String)o2.getKey();
                 return m1.compareTo(m2);
-            }
         });
-        for(Map.Entry<Object, Object> me : al){
-            sb.append(me.getKey());
-            sb.append("=");
-            sb.append(me.getValue());
-            sb.append("\n");
-        }
+        al.forEach(me -> {
+            sb.append(me.getKey()).append("=").append(me.getValue()).append("\n");
+        });
     }
 
     public boolean isDisplayJMeterVariables() {