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;