avoid branching by using %= operator

This commit is contained in:
Stephen Birarda 2015-09-15 15:15:26 -07:00
parent 6756d5364b
commit 8454cb916c

View file

@ -102,9 +102,7 @@ void PacketTimeWindow::onPacketArrival() {
_packetIntervals[_currentPacketInterval++] = duration_cast<microseconds>(now - _lastPacketTime).count(); _packetIntervals[_currentPacketInterval++] = duration_cast<microseconds>(now - _lastPacketTime).count();
// reset the currentPacketInterval index when it wraps // reset the currentPacketInterval index when it wraps
if (_currentPacketInterval == _numPacketIntervals) { _currentPacketInterval %= _numPacketIntervals;
_currentPacketInterval = 0;
}
} }
// remember this as the last packet arrival time // remember this as the last packet arrival time
@ -123,7 +121,5 @@ void PacketTimeWindow::onProbePair2Arrival() {
_probeIntervals[_currentProbeInterval++] = duration_cast<microseconds>(now - _firstProbeTime).count(); _probeIntervals[_currentProbeInterval++] = duration_cast<microseconds>(now - _firstProbeTime).count();
// reset the currentProbeInterval index when it wraps // reset the currentProbeInterval index when it wraps
if (_currentProbeInterval == _numProbeIntervals) { _currentProbeInterval %= _numProbeIntervals;
_currentProbeInterval = 0;
}
} }