[FFmpeg-devel] [PATCH] ffmpeg: correct units for raw pts in -progress report

Michael Niedermayer michael at niedermayer.cc
Mon Aug 27 22:59:00 EEST 2018


On Sun, Aug 26, 2018 at 11:53:16AM +0530, Gyan Doshi wrote:
> On 09-08-2018 10:09 AM, Gyan Doshi wrote:
> >
> >
> >On 08-08-2018 12:47 AM, Nicolas George wrote:
> >>Gyan Doshi (2018-08-08):
> >>>That will just defer the breaking change.
> >>
> >>That will leave people time to notice the change and allow old and new
> >>scripts to work during the transition.
> >
> >Will do it this way.
> 
> Revised patch attached.
> 
> Gyan

>  ffmpeg.c |    2 ++
>  1 file changed, 2 insertions(+)
> cfd96a95728f4dd9c873985b872be949b0495aea  0001-ffmpeg-add-correct-field-for-raw-pts-in-progress-rep.patch
> From 397591ae236cd19f2c50772ea8f7288b80705c89 Mon Sep 17 00:00:00 2001
> From: Gyan Doshi <ffmpeg at gyani.pro>
> Date: Sun, 5 Aug 2018 12:34:21 +0530
> Subject: [PATCH] ffmpeg: add correct field for raw pts in -progress report
> 
> PTS is in microseconds, so correct field name is out_time_us.
> 
> Old field out_time_ms kept for now - will be removed after a suitable transition
> period.
> 
> Fixes #7345
> ---
>  fftools/ffmpeg.c | 2 ++
>  1 file changed, 2 insertions(+)

LGTM

thanks

PS: these fields seem not documented anywhere, neither the new nor the old or
iam looking at the wrong places

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

You can kill me, but you cannot change the truth.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180827/87ddc33d/attachment.sig>


More information about the ffmpeg-devel mailing list