[FFmpeg-devel] [PATCH 1/4] avutil: add av_format_option_for_user() callback system

Michael Niedermayer michaelni at gmx.at
Mon May 11 14:16:15 CEST 2015


On Mon, May 11, 2015 at 02:26:24AM -0300, James Almer wrote:
> On 10/05/15 10:18 PM, Michael Niedermayer wrote:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavutil/opt.c |   32 ++++++++++++++++++++++++++++++++
> >  libavutil/opt.h |   21 +++++++++++++++++++++
> >  2 files changed, 53 insertions(+)
> > 
> 
> If this is meant to solve the whole debate from the "Tell users about -use_absolute_path"
> thread, then i think it's a bit overkill.

yes i felt similarly, but


> "Set <AVOption> to <value>" is generic enough
> that works in any situation, be it for API user or ffmpeg.c, and could be used in every
> av_log message.

it works, but it does not work well

consider the normal/generic case of a user application using
libavcodec and maybe libavformat amongth probably other libs

the user sees a message saying something along the lines of:
to do this set option foo to bar

The user most likly does not know that this message comes from
libavcodec or libavformat, another lib or the application itself
and likely the user doesnt even know the difference anyway

but the way to set the option can and does sometimes depend on for
what the option is

in mplayer, its one of
-lavdopts o=XXX=YYY
-lavcopts o=XXX=YYY
-lavfopts o=XXX=YYY
-lavdopts XXX=YYY
-lavcopts XXX=YYY
-lavfopts XXX=YYY

in ffmpeg/ffplay/ffprobe its
-XXX YYY

then theres
--demuxer-lavf-o=XXX=YYY

and many more

also the message may originate from a libavcodec used by another lib
instead of the user application

I think the average user would benefit from having the option dispayed
with a exactly useable as is syntax

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

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150511/5cd28ec2/attachment.asc>


More information about the ffmpeg-devel mailing list