[FFmpeg-devel] [PATCH 1/2] fftools/cmdutils: add logflags option

Michael Niedermayer michael at niedermayer.cc
Wed Mar 21 01:04:57 EET 2018


On Tue, Mar 20, 2018 at 09:36:51AM +0100, Tobias Rapp wrote:
> On 20.03.2018 02:44, Michael Niedermayer wrote:
> >On Mon, Mar 19, 2018 at 09:31:44AM +0100, Tobias Rapp wrote:
> >>On 14.03.2018 09:55, Tobias Rapp wrote:
> >>>Allows to set the AV_LOG_PRINT_LEVEL and AV_LOG_SKIP_REPEATED flags
> >>>using a distinct command-line option, similar to other flag options.
> >>>Previously only the AV_LOG_SKIP_REPEATED flag was supported as a prefix
> >>>to the "loglevel" option value.
> >>>
> >>>Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
> >>>---
> >>>  doc/fftools-common-opts.texi | 13 +++++++++++++
> >>>  fftools/cmdutils.c           | 31 +++++++++++++++++++++++++++++++
> >>>  fftools/cmdutils.h           |  6 ++++++
> >>>  3 files changed, 50 insertions(+)
> >>>
> >>>[...]
> >>
> >>Any opinions? My motivation was adding support for AV_LOG_PRINT_LEVEL on the
> >>CLI. Using the existing flags option string parsing functions seemed easier
> >>and more consistent that extending the custom loglevel string parser.
> >
> >this means the feature would require every user app to add similar code
> >if one wants to support it.
> >
> >it would be better if apps do not need changes for added options
> 
> Not sure I understand. Do you mean user apps that link to libav* libraries
> and want to mimic the command-line options of ffmpeg/ffprobe/ffplay? 

no, i mean that a user app doesnt want to support each option by specific code
most user apps just want things to work without changes

imagine each option for each codec would need a change to each user app
... unpractical ...


> What
> alternative would you suggest for managing AV_LOG_PRINT_LEVEL via
> command-line options?

whatever parses the log level could also do all options
or am i missing something ?

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

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- 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/20180321/d31b786a/attachment.sig>


More information about the ffmpeg-devel mailing list