[FFmpeg-devel] [PATCH] Add pad filter
Stefano Sabatini
stefano.sabatini-lala
Mon Nov 30 09:17:18 CET 2009
On date Monday 2009-11-30 01:28:41 +0100, Michael Niedermayer encoded:
> On Sun, Nov 29, 2009 at 02:13:51AM +0100, Stefano Sabatini wrote:
> > On date Friday 2009-11-27 02:42:59 +0100, Michael Niedermayer encoded:
> > > On Fri, Nov 27, 2009 at 01:09:28AM +0100, Stefano Sabatini wrote:
[...]
> > > > + y1 = y == 0 ? 0 : y + pad->y;
> > > > + h1 = h;
> > > > + if (y == 0)
> > > > + h1 += pad->y;
> > > > + if (y+h >= link->h)
> > > > + h1 += pad->h - pad->y - link->h;
> > > > +
> > > > + avfilter_draw_slice(link->dst->outputs[0], y1, h1);
> > >
> > > this draw slice behaves quite odd, i would have expectd that slices didnt grow
> >
> > Also this was wrong as it was misbehaving when in combination with
> > vflip, since the slices direction order wasn't respected.
> >
> > Now the filter sends slices with dimension not greater than those of
> > the input slice, and keeps in consideration the slices direction (which
> > unfortunately complicates the code).
>
> i think the code is too complex and messy now for what it does.
> Also it now partly duplicates slicify
What about to issue just a single slice for the top and bottom bars?
Then the user may manually add a slicify filter for decreasing the
size of the top/bottom bars if she cares, that should greatly simplify
the code.
Regards.
--
FFmpeg = Fundamentalist & Furious Merciful Portable EnGine
More information about the ffmpeg-devel
mailing list