On 12 Oct 2022, at 19:39, Anton Khirnov wrote:
> Quoting Marvin Scholz (2022-09-28 12:21:00)
>> Currently it would essentially change the find_stream_info setting for
>> the file it was specified for and all following files, which is unusual
>> and somewhat unexpected behaviour for a per-file option and not even
>> documented to behave like this.
>
> Well, it's not documented at all.
>
>> ---
>> fftools/ffmpeg.h | 1 +
>> fftools/ffmpeg_opt.c | 7 ++++---
>> 2 files changed, 5 insertions(+), 3 deletions(-)
>>
>> diff --git a/fftools/ffmpeg.h b/fftools/ffmpeg.h
>> index ede0b2bd96..3389ce5edc 100644
>> --- a/fftools/ffmpeg.h
>> +++ b/fftools/ffmpeg.h
>> @@ -126,6 +126,7 @@ typedef struct OptionsContext {
>> int accurate_seek;
>> int thread_queue_size;
>> int input_sync_ref;
>> + int find_stream_info;
>>
>> SpecifierOpt *ts_scale;
>> int nb_ts_scale;
>> diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
>> index 5febe319e4..df1c8df454 100644
>> --- a/fftools/ffmpeg_opt.c
>> +++ b/fftools/ffmpeg_opt.c
>> @@ -190,7 +190,6 @@ static int input_stream_potentially_available = 0;
>> static int ignore_unknown_streams = 0;
>> static int copy_unknown_streams = 0;
>> static int recast_media = 0;
>> -static int find_stream_info = 1;
>>
>> static void uninit_options(OptionsContext *o)
>> {
>> @@ -240,6 +239,7 @@ static void init_options(OptionsContext *o)
>> o->accurate_seek = 1;
>> o->thread_queue_size = -1;
>> o->input_sync_ref = -1;
>> + o->find_stream_info = 1;
>> o->shortest_buf_duration = 10.f;
>> }
>>
>> @@ -1307,7 +1307,7 @@ static int open_input_file(OptionsContext *o, const
>> char *filename)
>> for (i = 0; i < ic->nb_streams; i++)
>> choose_decoder(o, ic, ic->streams[i], HWACCEL_NONE,
>> AV_HWDEVICE_TYPE_NONE);
>>
>> - if (find_stream_info) {
>> + if (o->find_stream_info) {
>> AVDictionary **opts = setup_find_stream_info_opts(ic,
>> o->g->codec_opts);
>> int orig_nb_streams = ic->nb_streams;
>>
>> @@ -4000,7 +4000,8 @@ const OptionDef options[] = {
>> { "thread_queue_size", HAS_ARG | OPT_INT | OPT_OFFSET | OPT_EXPERT |
>> OPT_INPUT | OPT_OUTPUT,
>> { .off
>> = OFFSET(thread_queue_size) },
>> "set the maximum number of queued packets from the demuxer" },
>> - { "find_stream_info", OPT_BOOL | OPT_PERFILE | OPT_INPUT | OPT_EXPERT,
>> { &find_stream_info },
>> + { "find_stream_info", OPT_BOOL | OPT_PERFILE | OPT_INPUT | OPT_EXPERT |
>> OPT_OFFSET,
>> + { .off
>> = OFFSET(find_stream_info) },
>
> You can drop OPT_PERFILE, since it's implied by OPT_OFFSET.
>
> Otherwise patch LGTM, I'll do the change myself and push if you don't
> object.
>
Thanks for the review, feel free to do the change yourself.
> --
> Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
