You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@uima.apache.org by "Marshall Schor (JIRA)" <de...@uima.apache.org> on 2014/11/06 21:56:34 UTC
[jira] [Updated] (UIMA-4094) moveTo(fs) where fs > all items in
index is broken
[ https://issues.apache.org/jira/browse/UIMA-4094?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Marshall Schor updated UIMA-4094:
---------------------------------
Description:
Using moveTo(fs) where the fs is beyond the last element in the index incorrectly sets the iterator position to the 1st element. It should set the iterator to (from the Javadocs) ""insertion point" for fs, i.e., to a point where the feature structure at that position is greater than fs and the fs at the previous position (if it exists) is less than fs
When the fs is > all the elements in the index should conceptually set the iterator to 1 past the end of the index, which isValid() will return "false" for.
Because this change may break user code (in case users have worked around this), I think this change should be removable under a JVM property flag, something like uima.keep_wrong_moveTo_behavior.
UIMA-1601 introduced this behavior (8/19/2011)
was:
Using moveTo(fs) where the fs is beyond the last element in the index incorrectly sets the iterator position to the 1st element. It should set the iterator to (from the Javadocs) ""insertion point" for fs, i.e., to a point where the feature structure at that position is greater than fs and the fs at the previous position (if it exists) is less than fs
When the fs is > all the elements in the index should conceptually set the iterator to 1 past the end of the index, which isValid() will return "false" for.
Because this change may break user code (in case users have worked around this), I think this change should be removable under a JVM property flag, something like uima.keep_wrong_moveTo_behavior.
> moveTo(fs) where fs > all items in index is broken
> --------------------------------------------------
>
> Key: UIMA-4094
> URL: https://issues.apache.org/jira/browse/UIMA-4094
> Project: UIMA
> Issue Type: Bug
> Components: Core Java Framework
> Affects Versions: 2.6.0SDK
> Reporter: Marshall Schor
> Assignee: Marshall Schor
> Fix For: 2.6.1SDK
>
>
> Using moveTo(fs) where the fs is beyond the last element in the index incorrectly sets the iterator position to the 1st element. It should set the iterator to (from the Javadocs) ""insertion point" for fs, i.e., to a point where the feature structure at that position is greater than fs and the fs at the previous position (if it exists) is less than fs
> When the fs is > all the elements in the index should conceptually set the iterator to 1 past the end of the index, which isValid() will return "false" for.
> Because this change may break user code (in case users have worked around this), I think this change should be removable under a JVM property flag, something like uima.keep_wrong_moveTo_behavior.
> UIMA-1601 introduced this behavior (8/19/2011)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)