[FFmpeg-devel] [PATCH 2/4] ffmpeg: move AVFrame time base adjustment into a function

Michael Niedermayer michael at niedermayer.cc
Sun Sep 13 22:43:42 EEST 2020


On Sun, Sep 13, 2020 at 09:07:09PM +0300, Jan Ekström wrote:
> On Sun, Sep 13, 2020 at 7:14 PM Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> >
> > On Sun, Sep 13, 2020 at 01:26:20PM +0300, Jan Ekström wrote:
> > > This will have to be called later for video down the line.
> > > ---
> > >  fftools/ffmpeg.c | 77 ++++++++++++++++++++++++++++++++----------------
> > >  1 file changed, 52 insertions(+), 25 deletions(-)
> >
> > This affects the output, example:
> > ./ffmpeg -i mm-small.mpg -vf idet  -v 50 -bitexact -f null - 2>&1 | grep idet
> >
> 
> Apparently this was due to the leftover logging I left there. This
> also happens with vanilla FFmpeg if you add such logging to
> reap_filters.
> 
> In other words, just this following diff appears to fix it.

That was not what i expected to cause this but in fact you are correct.
this is a issue with log repeation removial changes from interspaced
debug messages. So this is resolved

for the record as others might run into this
use "-v repeat+50" instead of -v 50 to avoid this

thx

[...]
-- 
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: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200913/9a1696f0/attachment.sig>


More information about the ffmpeg-devel mailing list