You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by re...@apache.org on 2021/03/17 09:42:47 UTC

[uima-uimaj] branch bugfix/UIMA-6342-Moving-iterator-beyond-bounds-does-not-invalidate-it updated: [UIMA-6342] Moving iterator beyond bounds does not invalidate it

This is an automated email from the ASF dual-hosted git repository.

rec pushed a commit to branch bugfix/UIMA-6342-Moving-iterator-beyond-bounds-does-not-invalidate-it
in repository https://gitbox.apache.org/repos/asf/uima-uimaj.git


The following commit(s) were added to refs/heads/bugfix/UIMA-6342-Moving-iterator-beyond-bounds-does-not-invalidate-it by this push:
     new d6b8bc3  [UIMA-6342] Moving iterator beyond bounds does not invalidate it
d6b8bc3 is described below

commit d6b8bc39f247524f772cccc7e2ac089f1ef7d3ae
Author: Richard Eckart de Castilho <re...@apache.org>
AuthorDate: Wed Mar 17 10:43:01 2021 +0100

    [UIMA-6342] Moving iterator beyond bounds does not invalidate it
    
    - Added missing break statements in switch
---
 uimaj-core/src/main/java/org/apache/uima/cas/impl/Subiterator.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/uimaj-core/src/main/java/org/apache/uima/cas/impl/Subiterator.java b/uimaj-core/src/main/java/org/apache/uima/cas/impl/Subiterator.java
index 0969fd5..9339afc 100644
--- a/uimaj-core/src/main/java/org/apache/uima/cas/impl/Subiterator.java
+++ b/uimaj-core/src/main/java/org/apache/uima/cas/impl/Subiterator.java
@@ -879,14 +879,17 @@ public class Subiterator<T extends AnnotationFS> implements LowLevelIterator<T>
         if (is_beyond_bounds_chk_coveringNvc()) {
           return;
         }
+        break;
       case coveredBy:
         if (is_beyond_bounds_chk_coveredByNvc()) {
           return;
         }
+        break;
       case sameBeginEnd:
         if (is_beyond_bounds_chk_sameBeginEnd()) {
           return;
         }
+        break;
       case notBounded:
       default:
         // No check necessary