So this makes two things to clean up? - FeatureCollection.iterator() to be deprecated - FeatureSource.setTransaction( transaction ) to be added
Jody On 04/06/2010, at 4:21 PM, Andrea Aime wrote: > Michael Bedward ha scritto: >> Hi folks, >> >> For my (occasional) javadoc editing I'm trying to get my head around >> these two iterator lineages, but not with much success so far. >> Essentially I don't understand why we have both and there seem to be >> places in the library where one is disguised as another. Is this a >> historical thing or do they have distinct purposes that I should >> understand and document ? > > It is an accident due to some thinking that you can access data > using an API that does not have resource handling (close()) and > does not throw exceptions. > > SimpleFeatureIterator is the sane way to access data, > Iterator<SimpleFeature> is for people actively looking for trouble > (it's still there just for backwards compatibility, but we should > really, really deprecate it). > > Cheers > Andrea > > > -- > Andrea Aime > OpenGeo - http://opengeo.org > Expert service straight from the developers. > > ------------------------------------------------------------------------------ > ThinkGeek and WIRED's GeekDad team up for the Ultimate > GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the > lucky parental unit. See the prize list and enter to win: > http://p.sf.net/sfu/thinkgeek-promo > _______________________________________________ > Geotools-devel mailing list > Geotools-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel ------------------------------------------------------------------------------ ThinkGeek and WIRED's GeekDad team up for the Ultimate GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the lucky parental unit. See the prize list and enter to win: http://p.sf.net/sfu/thinkgeek-promo _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
