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

[Stefano Sabatini]

On date Friday 2011-04-08 15:19:58 +0200, Michael Niedermayer encoded:
> 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

Please don't commit it yet, I prefer to merge the nits/typos fixes in
the first patch rather than pollute history.
-- 
FFmpeg = Fundamental and Frenzy Multipurpose Philosofic Exciting Gadget