I see; and since none of my implementations extend AbstractAuthorityFactory factory they are not getting picked up ... by the DefaultAuthorityFactory find operation. I will take your advice and create an abstract class rather than an interface.
Cheers, Jody > 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
