Fix List.VNode.removeAfter() bug#2030
Conversation
|
Wow, this is an incredibly specific bug ! 😲 For the record I did not write that code, and I do not fully understand this method and the left shift operator. If you did understand this:
|
|
Hello @jdeniau ! |
|
@alexvictoor great, I sent you an invite. |
|
@alexvictoor For the record, I did add the same test as |
Tested on 10 000, and the last failing number was 1023
|
released in 5.0.3 |
Hello!
I have added a failing test. I was not sure where to write it, let me know if it should be located elsewhere. I guess the bug comes from a bad copy/paste between VNode.removeBefore() & VNode.removeAfter().
Let me know if there is something missing in this PR.