[FFmpeg-devel] [PATCH] libavformat: Add FIFO pseudo-muxer

[Moritz Barsnick]

On Mon, Jul 04, 2016 at 11:28:55 +0200, Jan Sebechlebsky wrote:
> >> +        {"recovery_wait_time", "Waiting time between recovery attempts (seconds)", OFFSET(recovery_wait_time),
> >> +         AV_OPT_TYPE_DURATION, {.i64 = FIFO_DEFAULT_RECOVERY_WAIT_TIME}, 0, INT64_MAX, AV_OPT_FLAG_ENCODING_PARAM},
> > So the default you are setting happens to be 1000000 seconds. Or am I
> > missing something? (I could check by applying your patch. Sorry, didn't
> > do so.)
> I agree this is confusing - The type of the recovery_wait_time option is 
> "duration" which allows you to enter duration in several formats and 
> converts it to the microseconds.

I totally missed AV_OPT_TYPE_DURATION, which makes this quite obvious
(I do know a bit of av_opts). Sorry!

Perhaps drop "(seconds)", as "ffmpeg -h full" will mark it as
"<duration>". And the texi source could refer to duration syntax, as
often expressed thus:
as @ref{time duration syntax,,the Time duration section in the ffmpeg-utils(1) manual,ffmpeg-utils}

Moritz