Jody Garnett a écrit :
> The "DefaultAuthorityFactory" (the one that
> acts as a dispatch system and delegates out work to all authority
> factories available on the classpath) is making the incorrect assumption
> that we are extending AbstractAuthorityFactory.
Unless I'm missing something, it work on the plain CRSAuthorityFactory
interface
for everything except the "Finder" stuff. Even this part is protected by "if (x
instanceof AbstractAuthorityFactory)", so it is not an assumption: we check if
we really have AbstractAuthorityFactory.
I suggest to NOT create an interface. The "Finder" stuff is a very
Geotools-specific, not planned for GeoAPI, may change in any future version,
and
abstract classes are much easier to change than interfaces. We may add more
geotools-specific functionalities in the future, and this work take place more
easily in AbstractAuthorityFactory (at least until we fell it mature enough to
move elswhere), so the need for "if (x instanceof AbstractAuthorityFactory)"
may
still there anyway.
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
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
