Re: [Geotools-devel] DefaultMap

[Andrea A. <and...@al...>]

On Wednesday 07 April 2004 19:46, David Zwiers wrote:
> When the old code read :
>
> DefaultMap map;
> ...
> map.render(renderer, env);
>
> my understanding was to make it read
>
> DefaultMapContext map;
> ...
>
> But what is the second line?

StyledMapRenderer renderer = new StyledMapRenderer();
renderer.setMapContext(map);
renderer.paint(...)

or use the equivalent methods in LiteRenderer if you don't want to
use the stateful renderer.

> More to the point: Why was one class deleted, when it's peer was
> deprecated?  Please try to be consistent so as to avoid wasting people's
> time, we don't all like fixing code we don't understand to test systems,
> especially when it's because someone goofed and forgot there was such a
> thing as deprecation ...
>

David,
according to the IRC meetings my understanding was that no-one was still
using those classes and that many felt that they where cumbersome to use.
Map should not be in use for a year or so, and I forgot to delete it. The deprecation
road would have led us to 3 map related classes, MapContext, Context and Map: that's
what I call a mess. I decided to go the delete road to avoid confusion before we started
to have many users, and forgot to delete the Map class, it's not easy to avoid errors 
when you only have time to work on gt2 on Sunday.
Anyway, we have CVS, if you feel like I did a mistake, resurrect those classes back and
deprecate them.

Best regards
Andrea Aime