simplify rollover logic for SafeLanding sequence size calculation

This commit is contained in:
Andrew Meadows 2019-07-17 13:57:08 -07:00
parent b0f297e696
commit 63b3d666f9

View file

@ -138,9 +138,7 @@ void SafeLanding::updateTracking() {
bool shouldStop = false; bool shouldStop = false;
{ {
Locker lock(_lock); Locker lock(_lock);
int32_t sequenceSize = _sequenceStart < _sequenceEnd ? auto sequenceSize = _sequenceEnd - _sequenceStart; // this works even in rollover case
(int32_t)(_sequenceEnd - _sequenceStart) :
(int32_t)((SafeLanding::MAX_SEQUENCE - _sequenceStart) + _sequenceEnd + 1); // with rollover
auto startIter = _sequenceNumbers.find(_sequenceStart); auto startIter = _sequenceNumbers.find(_sequenceStart);
auto endIter = _sequenceNumbers.find(_sequenceEnd - 1); auto endIter = _sequenceNumbers.find(_sequenceEnd - 1);