Thanks Martin ... so back to the actual bug then? It seems you fixed the problem at line 576: > assertSame(((Conversion) operation).getMethod(), ((Conversion) > projection).getMethod()); What was the fix? What did I mess up - was it hints like I suspected? Jody
> Jody Garnett a écrit : >> Okay I understand; I will break this up into two classes then... >> - BufferedAuthorityFactory which is abstract >> - DelegateAuthorityFactory which wraps up a single delegate > > May I suggest to just completly ignore BufferedAuthorityFactory for > now, and see at the end of the project how it could retrofit in the > new picture? > > About WKT-based factory: > >> I am still interested in using object pool in this case; so we can >> evict the wkt reader when it is not used (so we can free up all that >> room taken up in memory by the property file). > > This is what DeferredAuthorityFactory already do right now. I admit > that ObjectPool containing a singleton could do the same. > >> I really wished we started doing these renames on day 3 of the >> proposal as this has been a very useful discussion. Perhaps we should >> narrow the proposal acceptance period down from the current 15 days? > > I realize that getting a proposal accepted first, then apply it is the > way to perform well organized work, but I don't fit in this schema - > maybe I'm a very disorganized guy. There is no way I can explore all > avenue before a work begin. For me, idea popup when a work is in > progress. > > There is an other idea that popup yesterday. This is a proposal to > bring into consideration for the *end* of the project. Please ignore > it for now - I don't want that you lost time doing the diagrams again, > since other idea way still popup. My proposal is to rename every > > ThreadedFooFactory > into ConcurrentFooFactory > > Rational: The old classes were already thread-safe, so "Threaded" may > be not quite accurate. What the new classes bring is concurrency, not > thread safety. In addition, "Concurrent" prefix is consistent with > Java 5 usage, as in ConcurrentHashMap and its friend from the > java.util.concurrent package. > > 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 Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
