[ 
https://issues.apache.org/jira/browse/JENA-966?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14596588#comment-14596588
 ] 

A. Soroka edited comment on JENA-966 at 6/22/15 8:26 PM:
---------------------------------------------------------

[~claudenw], would it be useful to you for me to send a PR showing my proposal 
(adding {{andThen(Supplier<Iterator<T>>)}} to {{ExtendedIterator}}? It would 
look something like:

{code}

ExtendedIterator andThen(final Supplier<Iterator<T>> nextIterator) {
    return andThen(new Iterator<T>() {

        public T next() { return nextIterator.get().next(); }

        public boolean hasNext() { return nextIterator.get().hasNext(); }

        public void remove() { nextIterator.get().remove(); }

} );
}

{code}

and you would invoke it like {{ExtendedIterator joined = first.andThen(() -> 
second);}}.


was (Author: ajs6f):
[~claudenw], would it be useful to you for me to send a PR showing my proposal 
(adding {{andThen(Supplier<Iterator<T>>)}} to {{ExtendedIterator}}? It would 
look something like:

{code}

ExtendedIterator andThen(final Supplier<Iterator<T>> nextIterator) {
    return andThen(new Iterator<T>() {

        public T next() { return nextIterator.get().next(); }

        public boolean hasNext() { return nextIterator.get().hasNext(); }

        public void remove() { nextIterator.get().remove); }

} );
}

{code}

and you would invoke it like {{ExtendedIterator joined = first.andThen(() -> 
second);}}.

> LazyIterator
> ------------
>
>                 Key: JENA-966
>                 URL: https://issues.apache.org/jira/browse/JENA-966
>             Project: Apache Jena
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: Jena 3.0.0
>            Reporter: Claude Warren
>            Assignee: Claude Warren
>
> LazyIterator is an abstract class.  The documentation indicates that the 
> create() method needs to be overridden to create an instance.  From this I 
> would expect that 
> now LazyIterator(){
> @Override
> public ExtendedIterator<Model> create() {
>                       ...
> }};
> Would work however LazyIterator does not override:
> remoteNext(), andThen(), toList(), and toSet().
> I believe these should be implemented in the class.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to