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 2014/04/03 10:02:14 UTC

svn commit: r1584277 - /opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java

Author: joern
Date: Thu Apr  3 08:02:14 2014
New Revision: 1584277

URL: http://svn.apache.org/r1584277
Log:
OPENNLP-670 seqCodec is not initialized in the default constructor which causes a NPE if trained with default seq codec. Thanks  to Vinh Khuc for providing a patch.

Modified:
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java?rev=1584277&r1=1584276&r2=1584277&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/namefind/TokenNameFinderFactory.java Thu Apr  3 08:02:14 2014
@@ -54,8 +54,8 @@ public class TokenNameFinderFactory exte
    * of the resources.
    */
   public TokenNameFinderFactory() {
+    this.seqCodec = new BioCodec();
   }
-
   
   public TokenNameFinderFactory(byte[] featureGeneratorBytes, final Map<String, Object> resources,
       SequenceCodec<String> seqCodec) {