[FFmpeg-devel] [PATCH] Add pad filter

Michael Niedermayer michaelni
Sat Dec 12 19:40:50 CET 2009


On Sat, Dec 12, 2009 at 07:23:01PM +0100, Stefano Sabatini wrote:
> On date Saturday 2009-12-12 18:43:00 +0100, Michael Niedermayer encoded:
> > On Sat, Dec 12, 2009 at 03:14:40PM +0100, Stefano Sabatini wrote:
> > > On date Tuesday 2009-12-08 13:42:01 +0100, Stefano Sabatini encoded:
> > > > On date Tuesday 2009-12-08 03:01:04 +0100, Michael Niedermayer encoded:
> > [...]
> > > > > but i think there might be another problem that will need fixing,
> > > > > are data[x] aligned to 16 bytes? because some optimized filters will
> > > > > need them aligned and id guess pad & crop can produce non aligned cases.
> > > > > so possibly we need a AV_PERM_ALIGNED that ensures maximum alignment the
> > > > > cpu might need initiating a copy of the frame if its insfficiently aligned.
> > > 
> > > Ping+patch for the pad filter updated.
> > 
> > read above, there is a problem in the pad and crop filters.
> 
> You / anyone never talked about optimized filters which require
> aligned data, if we keep adding requirements each day lavfi won't
> never end-up integrated into the main repo, especially considering
> that I'm doing the coding work almost alone.

hmm, then lets try to get vf_pad in independant of that but we will
need to deal with this issue eventually. Some opt. filters might be able
to handle misagligned input but in some cases this might not be practical.
When we run into such a thing then we will need AV_PERM_ALIGNED


>  
> > Besides this, we need regression tests for the filters, i plan to make
> > this mandatory for every new filter now, too many bugs sliped through
> > already.
> > No reg test for a filter -> no approval.
> 
> That's resonable, OK, but this needs a more thorough discussion on how
> to design a more complete test system, I'll do that in a more
> pertinent thread.

good

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

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091212/8d4a6767/attachment.pgp>



More information about the ffmpeg-devel mailing list