[kaffe] Re: LinkedList.subList severely broken

[Dalibor Topic]

Salut Daniel,

--- Daniel Bonniot <Daniel.Bonniot@inria.fr> wrote:
> 
> > 	LinkedListIterator(LinkedList list, int index) {
> > 		super(list, index);
> > 		elem = list.head;
> >+		// Position elem to the index'th element.
> >+		// We know that index is a valid index, because that was 
> >+		// checked in the constructor in AbstractListIterator.
> >+		for (; index > 0; index--)
> >+		  elem = elem.next;
> >
> 
> Maybe it would be simpler and more robust (but a little bit less 
> efficient?) to just have:
> 
>  		super(list, index);
>  		elem = list.get(index);
> 
> Which one is considered better?

simpler code is always better in my opinion. Easier to write, easier to
understand, easier to change.

> Should I send a changelog entry too?

that would be nice, thanks.

cheers,
dalibor topic

__________________________________
Do you Yahoo!?
The New Yahoo! Search - Faster. Easier. Bingo.
http://search.yahoo.com