[FFmpeg-devel] [PATCH 1/5] avutil: add pixelutils API

Clément Bœsch u at pkh.me
Sat Aug 2 23:15:02 CEST 2014


On Sat, Aug 02, 2014 at 08:35:27PM +0200, Michael Niedermayer wrote:
[...]
> > +typedef struct AVPixelUtils {
> > +    /**
> > +     * Sum of absolute differences block functions.
> > +     * src1 and src2 addresses need to be aligned to the block size.
> > +     */
> > +    av_pixelutils_sad_fn sad[AV_PIXEL_BLOCKSZ_NB];
> > +
> > +    /**
> > +     * Sum of absolute differences block functions, (half) unaligned version.
> > +     * src1 address needs to be aligned to the block size.
> > +     * src2 has no alignment requirement; the 'u' refers to this second source.
> > +     */
> > +    av_pixelutils_sad_fn sad_u[AV_PIXEL_BLOCKSZ_NB];
> 
> the ABI depends on AV_PIXEL_BLOCKSZ_NB with this design as the arrays
> in the struct depend on its value so no enum values could be added

Right. I simplified the whole API with calling just one function. It
simplifies a lot the usage in filters and it's also ABI stable. New
patchset incoming.

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140802/5bca145c/attachment.asc>


More information about the ffmpeg-devel mailing list