You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@opennlp.apache.org by jo...@apache.org on 2011/11/11 12:44:21 UTC

svn commit: r1200827 - /incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java

Author: joern
Date: Fri Nov 11 11:44:20 2011
New Revision: 1200827

URL: http://svn.apache.org/viewvc?rev=1200827&view=rev
Log:
OPENNLP-369 Now using for each loop. Thanks to Aliaksandr Autayeu for providing patch.

Modified:
    incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java

Modified: incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java
URL: http://svn.apache.org/viewvc/incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java?rev=1200827&r1=1200826&r2=1200827&view=diff
==============================================================================
--- incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java (original)
+++ incubator/opennlp/trunk/opennlp-maxent/src/main/java/opennlp/model/AbstractDataIndexer.java Fri Nov 11 11:44:20 2011
@@ -138,18 +138,18 @@ public abstract class AbstractDataIndexe
    * @param cutoff The cutoff which determines whether a predicate is included.
    */
    protected static void update(String[] ec, Set<String> predicateSet, Map<String,Integer> counter, int cutoff) {
-    for (int j=0; j<ec.length; j++) {
-      Integer i = counter.get(ec[j]);
-      if (i == null) {
-        counter.put(ec[j], 1);
-      }
-      else {
-        counter.put(ec[j], i+1);
-      }
-      if (!predicateSet.contains(ec[j]) && counter.get(ec[j]) >= cutoff) {
-        predicateSet.add(ec[j]);
-      }
-    }
+     for (String s : ec) {
+       Integer i = counter.get(s);
+       if (i == null) {
+         counter.put(s, 1);
+       }
+       else {
+         counter.put(s, i + 1);
+       }
+       if (!predicateSet.contains(s) && counter.get(s) >= cutoff) {
+         predicateSet.add(s);
+       }
+     }
   }
 
   /**
@@ -173,6 +173,4 @@ public abstract class AbstractDataIndexe
   public float[][] getValues() {
     return null;
   }
-  
-  
-}
+}
\ No newline at end of file