[FFmpeg-devel] [PATCH] add fieldorder video filter

Mark Himsley mark at mdsh.com
Fri Apr 8 16:37:17 CEST 2011


On 08/04/11 13:14, Stefano Sabatini wrote:
> On date Friday 2011-04-08 12:10:32 +0100, Mark Himsley encoded:
>> On 31/03/11 08:47, Mark Himsley wrote:
>>> Converting to and from interlaced PAL DV files, with their
>>> bottom-field-first interlace field order, can be a pain. Converting tff
>>> files to DV results in tff DV files, which are hard to work with in
>>> editing software.
>>>
>>> The attached filter can:
>>>
>>> Convert field order by either moving all of the lines in the picture up
>>> by 1 line (bff to tff conversion) or down by 1 line (tff to bff
>>> conversion). The remaining line, the bottom line in bff to tff
>>> transforms or the top line in tff to bff transforms, is filled by
>>> copying the closest line in that field.
>>>
>>> Previous to this filter I have used a filter chain like this to do bff
>>> to tff conversion.
>>>
>>> format=yuv422p,crop=720:575:0:1,pad=720:576:0:0:black
>>>
>>> but that chain does not fill the remaining line.
>>
>> New version attached.
>>
>> removes bitstream formats from the accepted list of formats
>> hopefully addresses Stefano's usability nit's
>> makes tff the default
>> implements Stefano's suggestion for accepting string values in input
>> implements using av_image_get_linesize() to find the number of bytes
>> of video in a line
>> updated docs
>
> Looks OK but for a few nits and typos I'll address when committing the
> patch.

Thanks. Sorry for the nits and typos. I am interested in what I've got 
wrong.

> I note that most/all non-paletted bitstream formats should be
> already supported by the implemented logic, but this can be added with
> another patch if you have no interest in supporting them.

I am interested in supporting the bitstream formats, I'm just a bit 
ignorant about them at the moment.

-- 
Mark


More information about the ffmpeg-devel mailing list