I'd be grateful for a quick review on the PR (
https://github.com/geotools/geotools/pull/1171) for this one as it is in
gt-main, its a simple fix (famous last words), if everyone is happy I'd
also like to back port it to 14 before the next release there
Ian
On 14 April 2016 at 14:28, Ian Turton (JIRA) <[email protected]>
wrote:
> Ian Turton
> <https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?name=ianturton>
> *created* an issue
>
> GeoTools <https://osgeo-org.atlassian.net/browse/GEOT> / [image: Bug]
> <https://osgeo-org.atlassian.net/browse/GEOT-5403> GEOT-5403
> <https://osgeo-org.atlassian.net/browse/GEOT-5403>
> FilteringFeatureIterators throw NPE if you call next() before hasNext()
> <https://osgeo-org.atlassian.net/browse/GEOT-5403>
> Issue Type: [image: Bug] Bug
> Affects Versions: 15-beta, 14.3
> Assignee: Ian Turton
> <https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?name=ianturton>
> Components: main
> Created: 14/Apr/16 3:27 PM
> Priority: [image: High] High
> Reporter: Ian Turton
> <https://osgeo-org.atlassian.net/secure/ViewProfile.jspa?name=ianturton>
>
> If you call next() on a FilteringFeatureIterator (or it's child
> FilteringSimpleFeatureIterator) before you have called hasNext() it returns
> a null.
>
> I noticed this because UnionFeatureCollection calls next on its inputs and
> sometimes they are FilteringSimpleFeatureCollections.
>
> This discussion
> https://stackoverflow.com/questions/2176212/java-iterator-implementation-next-and-hasnext-enforcing-order
> on Stack Overflow suggests that this behaviour breaks the iterator contract.
> [image: Add Comment]
> <https://osgeo-org.atlassian.net/browse/GEOT-5403#add-comment> Add Comment
> <https://osgeo-org.atlassian.net/browse/GEOT-5403#add-comment>
>
> This message was sent by Atlassian JIRA (v7.2.0-OD-05-030#72002-
> sha1:7e8526a)
> [image: Atlassian logo]
>
>
> ------------------------------------------------------------------------------
> Find and fix application performance issues faster with Applications
> Manager
> Applications Manager provides deep performance insights into multiple
> tiers of
> your business applications. It resolves application problems quickly and
> reduces your MTTR. Get your free trial!
> https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
> _______________________________________________
> GeoTools-Devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>
>
--
Ian Turton
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
