Re: [FFmpeg-devel] [PATCH] avformat/rtpdec: Fix prft wallclock time.

2021-03-30 Thread Carl Eugen Hoyos
Am Di., 30. März 2021 um 17:02 Uhr schrieb James Almer : > > On 3/25/2021 1:46 AM, Alok Priyadarshi wrote: > > Timestamp difference is available in media timebase (1/90K) where as > > rtcp time is in the default microseconds timebase. This patch fixes > > the calculated prft wallclock time by resca

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec: Fix prft wallclock time.

2021-03-30 Thread James Almer
On 3/25/2021 1:46 AM, Alok Priyadarshi wrote: Timestamp difference is available in media timebase (1/90K) where as rtcp time is in the default microseconds timebase. This patch fixes the calculated prft wallclock time by rescaling the timestamp delta to the microseconds timebase. --- libavforma

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec: Fix prft wallclock time.

2021-03-30 Thread Alok Priyadarshi
Ping. Could someone please review this patch. Thanks. On Thu, Mar 25, 2021, 6:40 AM Alok Priyadarshi wrote: > In effect the patch does replace that one line. But it also adds the steps > to illustrate how the wallclock is calculated. Adding all the calculations > in a single line will make it to

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec: Fix prft wallclock time.

2021-03-25 Thread Alok Priyadarshi
In effect the patch does replace that one line. But it also adds the steps to illustrate how the wallclock is calculated. Adding all the calculations in a single line will make it too long and hard to read. Note that delta_timestamp can be negative. It typically happens after rtcp SR is received a

Re: [FFmpeg-devel] [PATCH] avformat/rtpdec: Fix prft wallclock time.

2021-03-25 Thread Carl Eugen Hoyos
Am Do., 25. März 2021 um 05:47 Uhr schrieb Alok Priyadarshi : > > Timestamp difference is available in media timebase (1/90K) where as > rtcp time is in the default microseconds timebase. This patch fixes > the calculated prft wallclock time by rescaling the timestamp delta > to the microseconds ti