Sorry for the late review. But I am +0 as that is one of the apis I have not really used in geotools much. So I delegate to the experts on this one.
On 10-06-14 11:20 PM, Jody Garnett wrote: > So I have a few votes in - and a few reviews (thanks!) > > • Andrea Aime > • +1 Ben Caradoc-Davies > • Christian Mueller > • Ian Turton > • Justin Deoliveira > • +1 Jody Garnett > • Michael Bedward > • +0 Simone Giannecchini > > The proposal page [1] has been updated with pictures showing the final class > diagram. > > Discussion: > - Andrea thanks for sorting out listener and dispose methods on IRC > yesterday; the changes we talked about are now included (ie was unable to use > weak references, dispose is required; dispose should not dispose grid > coverage reader) > - Map is renamed to MapContent > > If you would like to review the work is available as: > - a patch against the http://jira.codehaus.org/browse/GEOT-3136 > - ... or later today on trunk > > [1] http://docs.codehaus.org/display/GEOTOOLS/MapContext+Refactor > ------------------------------------------------------------------------------ > 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 > [email protected] > https://lists.sourceforge.net/lists/listinfo/geotools-devel -- Justin Deoliveira OpenGeo - http://opengeo.org Enterprise support for open source geospatial. ------------------------------------------------------------------------------ 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 [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
