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