[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Ccrtp-devel] Bug report RTPDataQueue::getCurrentTimestamp()
From: |
Federico Montesino Pouzols |
Subject: |
Re: [Ccrtp-devel] Bug report RTPDataQueue::getCurrentTimestamp() |
Date: |
Fri, 10 Jan 2003 14:46:57 +0100 |
User-agent: |
Mutt/1.4i |
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