| | 1 | = [wiki:ReviewHeuristicLab3.3.0Code HeuristicLab 3.3.0 Code Review] - !HeuristicLab.Collections-3.3 = |
| | 2 | |
| | 3 | == Reviewer: swinkler == |
| | 4 | |
| | 5 | * The years mentioned in all copyright headers should be corrected (2002-2008 -> 2002-2010) |
| | 6 | * swagner: Corrected in r2790. |
| | 7 | * Could it be more performant not to use a `StringBuilder` in the `ToString()` method of `IndexedItem<T>`? (I'm not sure here) |
| | 8 | * swagner: Removed `StringBuilder` in r2790. |
| | 9 | * Using `System.Linq` and partially also `System.Linq` is not necessary in these public interfaces |
| | 10 | * swagner: Removed unused usings in r2790. |
| | 11 | * "Item[]", "Length", "Capacity", and other constant strings are used several times - it could be safer to define static strings that are used in the whole project |
