Thanks Martin comments inline .. and I am updating the page to match (oh
I see you have too - thanks!).
> But I don't think that the following ones are improvements...
>
> * AbstractAuthorityFactory --> AbstractCRSAuthority
> * DirectAuthorityFactory --> DirectCRSAuthority
> Replacing "Authority" by "CRS" is not accurate since
> those factories are not just about CRS. They are about
> CS, Datum, Parameters, CoordinateOperation as well.
>
Okay understood ... I would like to remove the word Factory (as these
classes are builders and are responsible for "definition" not creation).
Would any of the following be of interest to you: AbstractAuthority,
AbstractReferencingAuthority, ReferencingObjectBuilder,
ReferencingDefinitionAuthority. AbstractAuthority is the shortest ...
I have also changed the text to reflect your above statement:
| {{AbstractAuthorityFactory}} | {{AbstractAuthority}} | Authority
providing the definition of referencing objects for a code |
> * WeakHashSet --> CanonicalSet
> See http://jira.codehaus.org/browse/GEOT-1300
>
Thanks - I reopened the bug and added some notes.
> I'm unsure about
>
> * BufferedAuthorityFactory --> ThreadedCRSAuthority
>
> As above, the "CRS" part is innacurate. In addition this factory would be both
> buffered and threaded, so "Buffered" may be a name as good as "Threaded"...
>
I am sure about the Threaded part - BufferedAuthorityFactory did three
things; built referencing objects based on definitions, cached, and
handled multiple threads. These concerns are being isolated into three
different classes now ... and threaded is the one being handled here.
| {{BufferedAuthorityFactory}} | {{ThreadedAuthority}} | Handles shared
cache and worker dispatch for multiple threads |
Sweet,
Jody
-------------------------------------------------------------------------
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
