Stuart McCulloch wrote:
couple of thoughts, you could make it:

   extends AbstractAssociation, Iterable<T>

because Iterable<T> only has one method "iterator()" and you'd then
get the benefit of being able to use the many association in a foreach
without having to do "toIterable()", which looks a bit clunky

also, the wrapper methods should really be called "asList", "asSet", etc.
because they return views onto the original object (see "Arrays.asList")
rather than copying the contents, while "to..." usually suggests copying

Exellent suggestions! Will do.

/Rickard


_______________________________________________
qi4j-dev mailing list
[email protected]
http://lists.ops4j.org/mailman/listinfo/qi4j-dev

Reply via email to