OLD | NEW |
---|---|
1 /** | 1 /** |
2 * An iterator that allows the user to move forward and backward though | 2 * An iterator that allows the user to move forward and backward though |
3 * a set of items. (Bi-directional) | 3 * a set of items. (Bi-directional) |
4 */ | 4 */ |
5 class BiIterator<E> { | 5 class BiIterator<E> { |
6 | 6 |
7 /** | 7 /** |
8 * Provides forward and backward iterator functionality to keep track | 8 * Provides forward and backward iterator functionality to keep track |
9 * which item is currently selected. | 9 * which item is currently selected. |
10 */ | 10 */ |
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
51 currentIndex.value -= 1; | 51 currentIndex.value -= 1; |
52 } | 52 } |
53 return list[currentIndex.value]; | 53 return list[currentIndex.value]; |
54 } | 54 } |
55 | 55 |
56 /** | 56 /** |
57 * Move the iterator pointer over so that it points to a given list item. | 57 * Move the iterator pointer over so that it points to a given list item. |
58 */ | 58 */ |
59 void jumpToValue(E val) { | 59 void jumpToValue(E val) { |
60 for (int i = 0; i < list.length; i++) { | 60 for (int i = 0; i < list.length; i++) { |
61 if (list[i] === val) { | 61 if (identical(list[i], val)) { |
Lasse Reichstein Nielsen
2012/11/12 13:10:41
Arguably this should never have been === to begin
floitsch
2012/11/12 22:18:43
Maybe, but I don't know if they depend on it. Keep
| |
62 currentIndex.value = i; | 62 currentIndex.value = i; |
63 break; | 63 break; |
64 } | 64 } |
65 } | 65 } |
66 } | 66 } |
67 } | 67 } |
OLD | NEW |