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

Michael Niedermayer michaelni at gmx.at
Fri Apr 8 15:19:58 CEST 2011


On Fri, Apr 08, 2011 at 02:14:19PM +0200, 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

applied

regression tests would be nice to have too

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

While the State exists there can be no freedom; when there is freedom there
will be no State. -- Vladimir Lenin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110408/4d941975/attachment.asc>


More information about the ffmpeg-devel mailing list