You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2015/12/22 12:30:11 UTC

svn commit: r1721362 - /jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java

Author: fschumacher
Date: Tue Dec 22 11:30:11 2015
New Revision: 1721362

URL: http://svn.apache.org/viewvc?rev=1721362&view=rev
Log:
Use isEmpty to check for an empty list, since it can be implemented more efficiently than size().

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java?rev=1721362&r1=1721361&r2=1721362&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/report/dashboard/ReportGenerator.java Tue Dec 22 11:30:11 2015
@@ -243,7 +243,7 @@ public class ReportGenerator {
                         .getFilteredSamples();
                 // Sample is kept if none filter is set or if the filter
                 // contains its name
-                return filteredSamples.size() == 0
+                return filteredSamples.isEmpty()
                         || filteredSamples.contains(sample.getName());
             }
         });