Hi Martin - so does this mean my proposal about EPSG_DATA_SOURCE hint is accepted then? Perhaps I should just remove the proposal .... your guidence is apreciated.
I was mostly setting up a couple developers today - I will be back on the CRSAuthority case tomorrow. > Jody did a nice work of putting global configuration in a central place > (org.geotools.factory.GeoTools). I continued on his track. Now we have: > > * System properties String defined in org.geotools.factory.GeoTools, > instead of various places in Geotools code base. For now: > > - CRS_AUTHORITY_EXTRA_DIRECTORY > - EPSG_DATA_SOURCE > - FORCE_LONGITUDE_FIRST_AXIS_ORDER > Thanks Martin - good idea to put the useful strings in one spot. > * A Hints.putSystemDefault(Key,Object) method added. Now the following > codes should have the same effect: > > System.setProperty(GeoTools.FORCE_LONGITUDE_FIRST_AXIS_ORDER, "true"); > > or > > Hints.putSystemDefault(Hints.FORCE_LONGITUDE_FIRST_AXIS_ORDER, > Boolean.TRUE); > > The former has the following advantages: > * Can be specified on the command line. > > The later has the following advantages: > * No security constrains (or is it an inconvenient?) > * Any hint defined in org.geotools.factory.Hints can be set as a > system default, instead of only a few system property strings. > 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). > Currents factories in gt2-main ignore hints. I don't know if we plan to use > them > in the future. But if there is such a possibility, it may be worth to make the > following classes extend org.geotools.factory.AbstractFactory: > > org/geotools/data/FeatureLockFactory.java > org/geotools/data/AbstractDataStoreFactory.java > org/geotools/styling/AbstractStyleFactory.java > I tried to hunt down as many things as I could - let me clean up the above three you mentioned. > The cost is an initially empty HashMap instance inherited by all those > factories. Should I make such change (and add a Foo(Hints) constructor)? > > If not hints were provided I expected to use GeoTools.getDefaultHints() - is this the cost you were thinking of? Cheers, Jody ------------------------------------------------------------------------- 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
