amiiigh
commented
5 years ago I squashed all the commits into one
Open amiiigh opened 5 years ago
amiiigh
commented
5 years ago I squashed all the commits into one
Hey this pull request includes:
Making Connection a Duplex Stream instead of an EventEmitter
Fixing the problem with getting corrupted data
CurrentNode of the LinkedList will point to the node with the lowest value, not the first node added -- Imagine adding numbers in this order into the list: 9, 1, 5, 3, 4, 5, 8. The currentNode in the current implementation would be 9 and if you use the seek() function you would always get 9 because there is no node after 9 since it is the largest and also making current node to point to the node with the lowest value makes more sense
Sending the ACK for packets that have sequence number lower than what we expected again.