[FFmpeg-devel] RTCP packets issue

[Luca Abeni]

On 07/22/2010 09:05 AM, Luca Abeni wrote:
[...]
>> 2. The current code puts a value of 0 for 'delay_since_last' value.
>
> Last time I checked, this value seemed to be computed correctly. It is
> set to 0 only before the first RTCP SR packet is received (which makes
> sense).

Sorry, I can now see the problem... Don't know what happened, but I think
I've seen this working correctly in the past.


>> I
>> have changed uint64_t ntp_time= s->last_rtcp_ntp_time to uint64_t
>> ntp_time= av_gettime()/1000000 to get a reasonable value.
>
> This is wrong. This value should be the delay between receiving the
> last SR packet and sending this RR packet.

Sorrry again, I misread the code. I now think you are right; ntp_time
is not currently set in the correct way. What you do is equivalent to
"ntp_time= ff_ntp_time();", but I am not sure it is correct... In this
way, you compare a time received in the SR packet (last_rtcp_ntp_time)
with the local time... I am going to check what should be done, here.


				Luca