svn commit: r1523700 - /opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java

[jo...@apache.org]

Author: joern
Date: Mon Sep 16 15:20:18 2013
New Revision: 1523700

URL: http://svn.apache.org/r1523700
Log:
OPENNLP-594 Fixed a bug when the last child from a parse was removed. Thanks to Ioan Barbulescu for providing a patch.

Modified:
    opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java

Modified: opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java
URL: http://svn.apache.org/viewvc/opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java?rev=1523700&r1=1523699&r2=1523700&view=diff
==============================================================================
--- opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java (original)
+++ opennlp/trunk/opennlp-tools/src/main/java/opennlp/tools/parser/Parse.java Mon Sep 16 15:20:18 2013
@@ -518,8 +518,10 @@ public class Parse implements Cloneable,
 
   public void remove(int index) {
     parts.remove(index);
-    if (index == 0 || index == parts.size()) { //size is orig last element
-      span = new Span((parts.get(0)).span.getStart(),(parts.get(parts.size()-1)).span.getEnd());
+    if(! parts.isEmpty()) {
+        if (index == 0 || index == parts.size()) { //size is orig last element
+            span = new Span((parts.get(0)).span.getStart(),(parts.get(parts.size()-1)).span.getEnd());
+          }
     }
   }