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

Julian Sedding commented on SLING-2641:
---------------------------------------

The patch looks good, except some minor nitpicks:
* the patch includes changes to the QueriableResourceProvider interface
* javadoc for Resource#getChildren states "Returns an iterator ...", it should 
be "Returns an iterable ..."
* javadoc for ResourceResolver#getChildren states "Returns an 
<code>Iterator</code> ...", it should be "Returns an <code>Iterable</code> ..."

                
> Make listChildren() Iterable
> ----------------------------
>
>                 Key: SLING-2641
>                 URL: https://issues.apache.org/jira/browse/SLING-2641
>             Project: Sling
>          Issue Type: Improvement
>          Components: API, General, ResourceResolver, Testing
>    Affects Versions: API 2.2.4
>            Reporter: Dan Klco
>            Assignee: Carsten Ziegeler
>              Labels: features, noob, patch
>         Attachments: iterable-patch.txt, SLING-2641-Resource-Iterator.diff
>
>
> When you call Resource.listChildren() or resourceResolver.listChildren() it 
> returns a Iterator<Resource>, this is fine if you want to just iterate 
> through the results using old-style while loops, but if you want to use 
> enhanced loops, you are out of luck.  
> I'm proposing adding an interface to return from these methods which extends 
> both Iterator<Resource> and Iterable<Resource>.  This will allow for using 
> enhanced loops with the results of listChildren().

--
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