[FFmpeg-devel] [PATCH v2] Add option to log timing

Moritz Barsnick barsnick at gmx.net
Wed Sep 18 16:47:01 EEST 2019


On Wed, Sep 18, 2019 at 00:38:13 +0000, Soft Works wrote:
> You're correct, It's a superset. I found it more convenient to do it like this because:
> - It would make little sense printing only the date for each line
> - I wanted to avoid requiring a user to specify two separate flags to get both date and time
>
> But I wouldn't mind changing this to independent flags

It's okay by me, I was just pointing it out. Others may want to
comment.

> > You also need to add documentation for these options to doc/fftools-
> > common-opts.texi (section "@item -loglevel").
>
> Damn, this is going to get more work than expected ;-)

But it's easy just to add a few words. :-) (Be sure to rebuild the docs
as well, to ensure that your edits don't break the markup syntax.)

> Thanks for looking into this, I'll wait for other comments before updating the patch.

I always get a final date/time line without a terminating line break,
which is quite ugly:

barsnick at sunshine:/usr/new/tools/video/ffmpeg/ffmpeg-snapshot-2019-09-17 > ffmpeg -loglevel +datetiming -f lavfi -i testsrc2=g=x
2019-09-18 15:43:16.455 ffmpeg version N-94957-g65589ad Copyright (c) 2000-2019 the FFmpeg developers
2019-09-18 15:43:16.457   built with icc (ICC) 14.0.3 20140422
2019-09-18 15:43:16.457   configuration: --prefix=/usr/new/tools/video/install/ffmpeg/2019-09-17 --cc=icc --cxx=icpc --enable-gpl --enable-version3 --enable-nonfree --dis
able-shared --enable-gnutls --enable-libcdio --enable-libfreetype --enable-libx264 --enable-libvpx --enable-libmp3lame --enable-openal --enable-libopencore-amrnb --enable
-libopencore-amrwb --enable-libtwolame --enable-librtmp --enable-libass --enable-libv4l2 --enable-libvidstab --enable-libfdk-aac --enable-libsmbclient --enable-libzvbi --
enable-libtesseract --enable-libzmq --enable-libopus --enable-libcodec2 --enable-libxml2 --enable-libopencv
2019-09-18 15:43:16.458   libavutil      56. 35.100 / 56. 35.100
2019-09-18 15:43:16.459   libavcodec     58. 57.100 / 58. 57.100
2019-09-18 15:43:16.459   libavformat    58. 33.100 / 58. 33.100
2019-09-18 15:43:16.460   libavdevice    58.  9.100 / 58.  9.100
2019-09-18 15:43:16.461   libavfilter     7. 58.102 /  7. 58.102
2019-09-18 15:43:16.461   libswscale      5.  6.100 /  5.  6.100
2019-09-18 15:43:16.462   libswresample   3.  6.100 /  3.  6.100
2019-09-18 15:43:16.463   libpostproc    55.  6.100 / 55.  6.100
2019-09-18 15:43:16.465 [Parsed_testsrc2_0 @ 0xb24fa40] Option 'g' not found
2019-09-18 15:43:16.466 [lavfi @ 0xb24ea80] Error initializing filter 'testsrc2' with args 'g=x'
2019-09-18 15:43:16.468 testsrc2=g=x: Option not found
2019-09-18 15:43:16.469 barsnick at sunshine:/usr/new/tools/video/ffmpeg/ffmpeg-snapshot-2019-09-17 >

(See my new shell prompt appearing on the same line as the final
timestamp.)

Please try to find out why that is and whether it can be fixed without
a hack.

Moritz


More information about the ffmpeg-devel mailing list