On 6/19/2016 4:52 PM, Thomas Mundt wrote: > James Almer schrieb am So, 19.6.2016: >> On 2/7/2016 8:39 PM, Thomas Mundt wrote: >>> + at item deint >>> +Specify which frames to deinterlace. Accept one of the following >>> +values: >>> + >>> + at table @option >>> + at item 0, all >>> +Deinterlace all frames. >>> + at item 1, interlaced >>> +Only deinterlace frames marked as interlaced. >>> + at end table >>> + >>> +The default value is @code{all}. >> >> This... >> >> [...] >> >>> + { "deint", "specify which frames to deinterlace", OFFSET(deint), >>> AV_OPT_TYPE_INT, {.i64=BWDIF_DEINT_INTERLACED}, 0, 1, FLAGS, "deint" }, >> >> ...and this don't match. So either fix the docs line or change the >> default here. >> Bump lavfi micro version of you change the default. >> >> For the record, every other deinterlace filter seems to default to >> "all". >> >>> + CONST("all", "deinterlace all frames", >>> BWDIF_DEINT_ALL, "deint"), >>> + CONST("interlaced", "only deinterlace frames marked as interlaced", >>> BWDIF_DEINT_INTERLACED, "deint"), > > Oops, I also placed the documentation at the wrong alphabetical position. > Patches attached. > Sorry about that!! > > Thomas
Both applied, thanks! _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel