Re: [Ccrtp-devel] Bug report RTPDataQueue::getCurrentTimestamp()

[Federico Montesino Pouzols]

        You are right, it is a bug. Probably we will make a new
release after this weekend that will have this fixed.

On Thu, Jan 09, 2003 at 04:18:39PM +0100, Guillaume Fraysse wrote:
> Hello everyone,
> 
> While having a go at ccRTP with a co-worker of mine, we had a little 
> synchronization problem which led us to find a bug in the 
> uint32 RTPDataQueue::getCurrentTimestamp() const (in queue.cpp) method :
> 
> The following line :
> uint32 result = now.tv_usec - getInitialTime().tv_usec;
> 
> gives an incorrect result if getInitialTime().tv_usec > now.tv_usec as you 
> expect to get an unsigned integer. 
> 
> We fixed it by removing the unsigned :
> int32 result = now.tv_usec - getInitialTime().tv_usec;
> 
> And now it works better.
> 
> Hope this helps, please tell us if we got something wrong,
> Cheers
> 
> Guillaume & Guillaume
> 
> 
> _______________________________________________
> Ccrtp-devel mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/ccrtp-devel