[FFmpeg-devel] [PATCH 1/2 v3] avfilter: add readvitc filter

Tobias Rapp t.rapp at noa-archive.com
Fri Apr 8 14:40:23 CEST 2016


On 08.04.2016 12:48, Carl Eugen Hoyos wrote:
> Tobias Rapp <t.rapp <at> noa-archive.com> writes:
>
>>> AV_PIX_FMT_YUV440P? Also J variants...
>>
>> Good catch, I was lazy and just copied the function from vf_eq.c. The
>> updated patch should contain all pixel formats with planar 8bit luma
>> component.
>
> Instead of listing the formats, check for pix_fmts
> that are 8bit and planar.

That might be an even better idea. Working on v3 of the patch I noticed 
that YA8 (which was on the list of v2) is not planar, but NV12 and NV21 
(which were missing in v2) have a planar Y.

Have attached version 3 of the patch which:
- filters the list of all pixel formats dynamically
- supports thr_b and thr_w parameters having the same value
- updates the warning if thr_b == thr_w to print the parameter value 
instead of the internal 8-bit value
- has reduced number of parentheses in CRC code

Regards,
Tobias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-avfilter-add-readvitc-filter.patch
Type: text/x-patch
Size: 13736 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160408/446d9a38/attachment.bin>


More information about the ffmpeg-devel mailing list