[ 
https://issues.apache.org/jira/browse/JENA-393?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Claude Warren reopened JENA-393:
--------------------------------


Dave Reynolds <[email protected]>
2:31 PM (4 hours ago)

to dev 
On 10/02/13 19:12, Claude Warren wrote:

I propose that we mark the following as deprecated with the noted substitution

ArrayIterator : Arrays.asList( x[] ).iterator();

ConcatenatedIterator : NiceIterator.andThen()

IteratorIterator : add a method to WrappedIterator to create an
ExtendedIterator that does the same thing.

UniqueExtendedIterator :  I am not sure about this one, but I think
that it should be replaced with a UniqueFilter that can be applied to
any ExtendedIterator.

I haven't been following this but happened to spot that last para which rang 
alarm bells. :)

UniqueExtendedIterator is used heavily in the code base and, I suspect outside 
it. Changing its implementation would be one thing but deprecating the class 
itself would need some justification.
                
> Deprecate redundant iterator classes.
> -------------------------------------
>
>                 Key: JENA-393
>                 URL: https://issues.apache.org/jira/browse/JENA-393
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: Jena
>    Affects Versions: Jena 2.10.0
>            Reporter: Claude Warren
>            Assignee: Claude Warren
>
> I propose that we mark the following as deprecated with the noted substitution
> ArrayIterator : Arrays.asList( x[] ).iterator();
> ConcatenatedIterator : NiceIterator.andThen()
> IteratorIterator : add a method to WrappedIterator to create an
> ExtendedIterator that does the same thing.
> UniqueExtendedIterator :  I am not sure about this one, but I think
> that it should be replaced with a UniqueFilter that can be applied to
> any ExtendedIterator.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to