Hi guys, and sorry I'm late but there's always the timezone/working time delay :P
The mix with T/E is explained because CursorableLinkedList implements List, where there are 2 different signatures of toArray() method, the one that uses Object[] and the other one that interests us[1]. That's why I didn't use T directly. The mix with T/Object is my fault, it will be fixed ASAP. Agreed that since the interface declares List<E>, we should use E as well, it will be fixed ASAP too. Thanks a lot for reviewing it, Simo [1] http://download.oracle.com/javase/6/docs/api/java/util/List.html#toArray(T[]) http://people.apache.org/~simonetripodi/ http://www.99soft.org/ On Wed, Oct 13, 2010 at 2:54 AM, sebb <[email protected]> wrote: > On 13 October 2010 01:15, Paul Benedict <[email protected]> wrote: >> The whole <E> convention comes from java.util where there are >> "elements" of a collection. > > Yes, I know. > >> I don't like to mix things. I think type T >> would be find everywhere unless you have multiple types and need a >> memorable letter. > > As far as I can tell, the collection only supports a single type, > similar to List<E>, and one would not expect to see other types used > in such a List class. > > But I could be wrong which is why I was asking. > > Since j.u.List uses <E>, I think it would make sense use <E> here as well. > >> On Tue, Oct 12, 2010 at 6:21 PM, sebb <[email protected]> wrote: >>> CursorableLinkedList is based on the type T, and removeFirst() returns >>> T, yet getFirst() returns Object. >>> >>> Also, toArray() returns Object[] or E[]. >>> >>> Not sure I follow why there is this mixture? >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: [email protected] >>> For additional commands, e-mail: [email protected] >>> >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
