You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by sc...@apache.org on 2015/01/20 22:44:17 UTC

svn commit: r1653377 - /uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java

Author: schor
Date: Tue Jan 20 21:44:16 2015
New Revision: 1653377

URL: http://svn.apache.org/r1653377
Log:
no Jira, fix comment

Modified:
    uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java

Modified: uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java
URL: http://svn.apache.org/viewvc/uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java?rev=1653377&r1=1653376&r2=1653377&view=diff
==============================================================================
--- uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java (original)
+++ uima/uimaj/trunk/uimaj-core/src/main/java/org/apache/uima/cas/impl/FSBagIndex.java Tue Jan 20 21:44:16 2015
@@ -135,10 +135,10 @@ public class FSBagIndex extends FSLeafIn
     }
 
     /**
-     * Strictly speaking, moving to a FS in a bag index isn't a valid operation because there are no keys
-     *   to define "equals".
+     * Although you might think that moving to a FS in a bag index isn't a valid operation because there are no key
+     *   to define "equals", the copy() operation uses this to move to the same spot.
      *   
-     * However, we implement the following:  If the FS is == (identical), we return that position.
+     * This requires moving to the == (identical) position.
      * If the FS is not found, we mark the iterator as invalid. 
      * @see org.apache.uima.internal.util.IntPointerIterator#moveTo(int)
      */