[FFmpeg-devel] [PATCH] avfilter: add BobWeaver deinterlacing filter

Thomas Mundt loudmax at yahoo.de
Sun Jun 19 21:52:41 CEST 2016


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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avfilter-vf_bwdif-Change-default-to-deinterlace-all-.patch
Type: application/octet-stream
Size: 1754 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160619/1ab3ab44/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-doc-filters.texi-Move-bwdif-to-correct-alphabetical-.patch
Type: application/octet-stream
Size: 3448 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160619/1ab3ab44/attachment-0001.obj>


More information about the ffmpeg-devel mailing list