From: Martin D. (JIRA) <ji...@co...> - 2007-07-07 10:55:11
|
[ http://jira.codehaus.org/browse/GEOT-1300?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Desruisseaux reopened GEOT-1300: --------------------------------------- > Isolate CanonicalSet as a subclass of WeakHashSet > ------------------------------------------------- > > Key: GEOT-1300 > URL: http://jira.codehaus.org/browse/GEOT-1300 > Project: GeoTools > Issue Type: Sub-task > Components: core referencing > Affects Versions: 2.4-M3 > Reporter: Jody Garnett > Assignee: Jody Garnett > Fix For: 2.4-M4 > > > WeakHashSet is currently pulling double duty: > - It functions as a normal set that happens to store its entries using WeakReferences > - it has additional methods (ie canonicalize) in order to function in a manner similar to String.intern > We should seperate these two concerns - primarily so what is going on can be understood. > - WeakHashSet - remains as set of weak references, methods providing additional functionality will be deprecated > - CanonicalSet cerated as an extension to WeakHashSet and proivdes a *unique* method > This bug report is done when we have updated the code and provided user docs on the result. > Sample use: > {code} > class FooFactory { > public Foo create(String definition) { > Foo created = new Foo(definition); > return (Foo) canionicalSet.unique(created); > } > } > {code} -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira |