| gergel | |
| Aug 13 2009, 10:24 AM |
| F284: ChangeRequest.txt | |
| Aug 13 2009, 10:31 AM |
Swaping a point upwards removes the former point on the position.
Swaping a point downwards removes the next point on the position.
To fix, we have to keep the former/next point and write it back. If swap is not possible (former/next index does not exist) the method should do nothing and return false.
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | None | T2394 Calling SwapPointPosition in PointSet deletes one of the points | ||
| Restricted Maniphest Task |
I understand that you want to enhance PointSetTest to check correct swapping, right? (because right now the test does not test much)
Revision 18547:
Corrected implementation. Both points are swapped now, if swapping is not possible the method returns false.