When I used MoveObjects() with an index of 0 with a collection of objects that includes the first object (at index 0), I get an IndexOutOfRangeException because it will try to insert objects at index -1.
I narrowed it down to this block of code in MoveObjects():
|
foreach (object modelObject in modelObjects) { |
|
int i = this.IndexOf(modelObject); |
|
if (i >= 0 && i <= index) |
|
displacedObjectCount++; |
|
} |
|
index -= displacedObjectCount; |
Since, the first object (at index 0) will satisfy the if condition, the index will be decreased from 0 to -1.
Should this condition be changed to if (i >= 0 && i < index) instead?
When I used
MoveObjects()with an index of 0 with a collection of objects that includes the first object (at index 0), I get anIndexOutOfRangeExceptionbecause it will try to insert objects at index -1.I narrowed it down to this block of code in
MoveObjects():ObjectListView/ObjectListView/FastObjectListView.cs
Lines 123 to 128 in 8a10cbd
Since, the first object (at index 0) will satisfy the
ifcondition, the index will be decreased from 0 to -1.Should this condition be changed to
if (i >= 0 && i < index)instead?