[FFmpeg-devel] [PATCH] ffprobe: Add option to allow unknown format private AVOptions

[Michael Niedermayer]

On Fri, Jun 26, 2020 at 02:15:00PM +0100, Derek Buitenhuis wrote:
> This useful, because by ffprobe's very nature, you use it to probe
> a file and find out what it is. Requiring every format private option
> to be known to the demuxer forces one to run ffprobe twice, if one
> wants to use ffprobe in a generic way.
> 
> For example, say one wants to probe all user-uploaded files, while
> also ignoring edit lists for any MP4s that are uploaded. Currently,
> you'd have to run ffprobe twice: once to identify the format, and
> once again to actually probe the metadata you want. After this
> patch, you could set -ignore_editlist 1 on every call and only
> probe once.
> 
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  fftools/ffprobe.c | 11 +++++++++--
>  1 file changed, 9 insertions(+), 2 deletions(-)

You explain why allow_unknown_format_opts = 1 is needed

does allow_unknown_format_opts = 0 have a use case too ?
because if not, this can be simplified

thx

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

Those who are best at talking, realize last or never when they are wrong.
-------------- 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/20200627/b1ef72a5/attachment.sig>