Mirrors/overte
3
0
Fork 0
mirror of https://github.com/overte-org/overte.git synced 2025-08-11 14:37:30 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

removed magic number 65536 from OctreeInboundPacketProcessor

Browse source
This commit is contained in:
wangyix 2014-06-16 16:08:48 -07:00
parent 5e37704772
commit 95b2524784
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
assignment-client/src/octree/OctreeInboundPacketProcessor.cpp
View file

@ -286,7 +286,7 @@ void SingleSenderStats::trackInboundPacket(unsigned short int incomingSequence,
printf("\t\t tracked seq %d\n", incomingSequence); printf("\t\t tracked seq %d\n", incomingSequence);
const int UINT16_RANGE = 65536; const int UINT16_RANGE = UINT16_MAX + 1;
const int MAX_REASONABLE_SEQUENCE_GAP = 1000; // this must be less than UINT16_RANGE / 2 for rollover handling to work const int MAX_REASONABLE_SEQUENCE_GAP = 1000; // this must be less than UINT16_RANGE / 2 for rollover handling to work
const int MAX_MISSING_SEQUENCE_SIZE = 100; const int MAX_MISSING_SEQUENCE_SIZE = 100;
@ -305,7 +305,7 @@ printf("\t\t tracked seq %d\n", incomingSequence);
int absGap = std::abs(incoming - expected); int absGap = std::abs(incoming - expected);
if (absGap >= UINT16_RANGE - MAX_REASONABLE_SEQUENCE_GAP) { if (absGap >= UINT16_RANGE - MAX_REASONABLE_SEQUENCE_GAP) {
// rollover likely occurred between incoming and expected. // rollover likely occurred between incoming and expected.
// correct the larger of the two so that it's within [-65536, -1] while the other remains within [0, 65535] // correct the larger of the two so that it's within [-UINT16_RANGE, -1] while the other remains within [0, UINT16_RANGE-1]
if (incoming > expected) { if (incoming > expected) {
incoming -= UINT16_RANGE; incoming -= UINT16_RANGE;
} else { } else {