You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/07/15 17:47:21 UTC
svn commit: r1503308 -
/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java
Author: sebb
Date: Mon Jul 15 15:47:21 2013
New Revision: 1503308
URL: http://svn.apache.org/r1503308
Log:
Drop the alternate commented approach
Modified:
commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java
Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java?rev=1503308&r1=1503307&r2=1503308&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math3/stat/Frequency.java Mon Jul 15 15:47:21 2013
@@ -518,12 +518,6 @@ public class Frequency implements Serial
long frequency = ent.getValue().longValue();
if (frequency == mostPopular) {
modeList.add(ent.getKey());
-// Alternatively, to avoid scanning the entries twice, keep recreating the set
-// To use this approach, comment out the values() scan loop above and uncomment below
-// } else if (frequency > mostPopular) {
-// modeList.clear(); // the previous List is obsolete
-// modeList.add(ent.getKey());
-// mostPopular = frequency;
}
}
return modeList;