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