On Sat, Nov 3, 2012 at 3:36 PM, Jody Garnett <jody.garn...@gmail.com> wrote: > Updated the pull request in response to feedback from Andrea and Micheal. > > - DataUtilities.close( Iterator ) > - DataUtilities.first( SimpleFeatureCollection ): SimpleFeature > - DataUtilities.first( FeatureCollection<?,F> ): F > - DataUtilities.list( FeatureCollection<?,F> ): List<F> // Already existed > - DataUtilities.list( FeatureCollection<?,F>, int ): List<F> > - DataUtilities.iterator( FeatureIterator ): Iterator // also Closable > - DataUtilities.collectionCast( FeatureCollection ): Collection
Cool, that should make writing a lot of tests easier (getting the first few items out of a collection is normally what testing code does anyways) > A few other fixes: > - Code that can make use of the above methods has been updated > - RasterAsPointFeatureCollection now extends BaseFeatureCollection in order > to avoid the use of WrappingIterator Hum... ok, Wondering what level of overlap there is now between BaseFeatureCollection and that processing oriented feature collection base I've made some time ago It would be nice to see how the whole looks now, but it seems you are making changes in another branch. If you had committed them in the same branch where you made the original pull request github would have updated the pull request with it. Cheers Andrea > -- > Jody Garnett > > > ------------------------------------------------------------------------------ > LogMeIn Central: Instant, anywhere, Remote PC access and management. > Stay in control, update software, and manage PCs from one command center > Diagnose problems and improve visibility into emerging IT issues > Automate, monitor and manage. Do more in less time with Central > http://p.sf.net/sfu/logmein12331_d2d > _______________________________________________ > GeoTools-Devel mailing list > GeoTools-Devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel > -- == Our support, Your Success! Visit http://opensdi.geo-solutions.it for more information. == Ing. Andrea Aime @geowolf Technical Lead GeoSolutions S.A.S. Via Poggio alle Viti 1187 55054 Massarosa (LU) Italy phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- ------------------------------------------------------------------------------ LogMeIn Central: Instant, anywhere, Remote PC access and management. Stay in control, update software, and manage PCs from one command center Diagnose problems and improve visibility into emerging IT issues Automate, monitor and manage. Do more in less time with Central http://p.sf.net/sfu/logmein12331_d2d _______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
