[Live-devel] scheduleNextQOSMeasurement() bug?

Thu Jan 15 05:41:40 PST 2009

Hi Ross,I would like to report a possible bug in scheduleNextQOSMeasurement():
unsigned usecsToDelay = nextQOSMeasurementUSecs < timeNowUSecs ? 0    : nextQOSMeasurementUSecs - timeNowUSecs;
I think that should be so:
unsigned usecsToDelay = nextQOSMeasurementUSecs < timeNowUSecs ? qosMeasurementIntervalMS*1000    : nextQOSMeasurementUSecs - timeNowUSecs;

because after several minutes, nextQOSMeasurementUSecs exceeds the size of sizeof (unsigned long) that is 4294967295.This makes qosMeasurementTimerTask = env->taskScheduler().scheduleDelayedTask(usecsToDelay,(TaskFunc*)periodicQOSMeasurement, (void*)NULL);
to don't delay periodicQOSMeasurement (usecsToDelay=0).
Regards,
Gabriele De Luca
_________________________________________________________________
25 GB di spazio online per archiviare i tuoi file. Gratis!
http://skydrive.live.com/?showunauth=1