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

svn commit: r1364057 - in /opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools: dictionary/serializer/DictionarySerializer.java formats/frenchtreebank/ConstitParseSampleStream.java

Author: autayeu
Date: Sat Jul 21 10:07:52 2012
New Revision: 1364057

URL: http://svn.apache.org/viewvc?rev=1364057&view=rev
Log:
OPENNLP-525: Exception cleanup in opennlp-tools. Thanks to Daniel Naber for the patch. Cause initialization workaround for Java5.

Modified:
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/dictionary/serializer/DictionarySerializer.java
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/frenchtreebank/ConstitParseSampleStream.java

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/dictionary/serializer/DictionarySerializer.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/dictionary/serializer/DictionarySerializer.java?rev=1364057&r1=1364056&r2=1364057&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/dictionary/serializer/DictionarySerializer.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/dictionary/serializer/DictionarySerializer.java Sat Jul 21 10:07:52 2012
@@ -299,7 +299,8 @@ public class DictionarySerializer {
       hd.endDocument();
     }
     catch (SAXException e) {
-      throw new IOException("Error during serialization: " + e.getMessage());
+      //TODO update after Java6 upgrade
+      throw (IOException) new IOException("Error during serialization: " + e.getMessage()).initCause(e);
     }
   }
 

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/frenchtreebank/ConstitParseSampleStream.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/frenchtreebank/ConstitParseSampleStream.java?rev=1364057&r1=1364056&r2=1364057&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/frenchtreebank/ConstitParseSampleStream.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/formats/frenchtreebank/ConstitParseSampleStream.java Sat Jul 21 10:07:52 2012
@@ -63,7 +63,8 @@ public class ConstitParseSampleStream ex
         try {
           saxParser.parse(new ByteArrayInputStream(xmlbytes), new ConstitDocumentHandler(producedParses));
         } catch (SAXException e) {
-          throw new IOException(e.getMessage());
+          //TODO update after Java6 upgrade
+          throw (IOException) new IOException(e.getMessage()).initCause(e);
         }
         
         parses.addAll(producedParses);