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 <ffm...@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.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
