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
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel
