From: James M. <jma...@gm...> - 2005-04-05 15:20:56
|
On Apr 5, 2005 4:52 AM, Martin Desruisseaux <mar...@no...> wrote: > While tracking down org.geotools.factory.Factory dependencies for > factory-hints branch merge, I realized that a lot of classes implement > this Factory interfaces, including some surprising classes. For example > > FunctionExpression > > Is FunctionExpression really intented to be a Factory? It doesn't seems > an object for creating other objects to me. > > I suspect that it implements Factory in order to be recognizable by > org.geotools.factory.FactoryFinder. It should not be needed. The > proposed replacement (FactoryRegistry) can work directly on Object. > Implementing Factory is optional, and should be done only for objects > that are real factories. > You are exactly right, the only reason it implements factory is to be picked up by the finder. Feel free to remove this once the new proposed replacment in in place. James |