Martin Desruisseaux ha scritto: > Andrea Aime a écrit : >> If this sets up things, who does the work? I know I've been hitting you >> quite a bit lately, so I'm here ready to do the changes and committing > > > Please go :). And thanks,
Done. As far as FactoryUsingWKT is concerned, the commit is at revision 24684. Summary of changes in referencing: * moved FactoryUsingWKT to referencing and performed the suggested visibility changes, made it throw a FactoryNotFoundException if the file is not there * registered factory in meta-inf * copied epsg.properties among referecing test resources, with a single definition, and copied the FactoryUsingWKT test among referencing tests to check SPI registration and factory are working as expected Summary of changes in epsg-extension * renamed espg.properties to epsg-extension.properties * created FactoryEPSGExtension * modified FactoryUsingWKT test to test FactoryEPSGExtension (and renamed it). Concerning the fallback mechanisms, the commit is at revision 24689 Here I modified the various create(xxx) methods in FallbackAuthorityFactory, then added a test with an extra factory that gets registered programmaticaly and has a broken wkt definition. That factory is before FactoryUsingWKT in the fallback chain, and I make sure the error I get is about WKT parsing, not a code not found exception. Working on the latter, I noticed that if I register programmaticaly a factory using FactoryFinder.addAuthorityFactory I then have to force a scan for plugins, or the SPI subsystem won't kick in. Is this intented? If so, maybe it's worth explaining that in the addAuthorityFactory method. Cheers Andrea ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
