+0, until now I did not use MapContext directly, but a cleanup sounds good Quoting Jody Garnett <jody.garn...@gmail.com>:
> 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 > Geotools-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/geotools-devel > ---------------------------------------------------------------- This message was sent using IMP, the Internet Messaging Program. ------------------------------------------------------------------------------ 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