Hi Jody

Jody Garnett a écrit :
> Hi Martin - so does this mean my proposal about EPSG_DATA_SOURCE hint is
> accepted then?

It was accepted since many weeks ago :). It was a perfectly legitimate wish from
the begining and I don't see any controversial issue with it.


> I am thinking hard on the later one ... I had expected people to make a
> call to GeoTools.setDefaultHints( hints ) - after setting things up the
> way they want. I also expected that the get/set default hints could be
> made per thread group in the future (so applets do not mess with each
> other).

There is a GeoTools.init(Hints) - this method name was already there (maybe by
you?), I though about setDefaultHints(...) but finally keep the name as it was.

Hints.putSystemDefault(...) is just an other was to set default hints, on a
hint-by-hint basis instead of as a whole.


> If not hints were provided I expected to use GeoTools.getDefaultHints()
> - is this the cost you were thinking of?

No. The following factory classes:

    org.geotools.data.FeatureLockFactory
    org.geotools.data.AbstractDataStoreFactory
    org.geotools.styling.AbstractStyleFactory

currently extend java.lang.Object. We could make them extend
org.geotools.factory.AbstractFactory. But they would then inherit a private
HashMap field for Factory.getDefaultImplementationHints() method implementation.
Since factories are usually singleton, I assume that it is not a big deal.
Advantage is more room left open for future Hints to be taken in consideration
by those factories, if it is of any use.

        Martin

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to