[FFmpeg-devel] [PATCH] lavfi: Port fspp to FFmpeg

[arwa arif]

>
>
> > --- a/libavfilter/allfilters.c
> > +++ b/libavfilter/allfilters.c
> > @@ -141,6 +141,7 @@ void avfilter_register_all(void)
> >      REGISTER_FILTER(FRAMEPACK,      framepack,      vf);
> >      REGISTER_FILTER(FRAMESTEP,      framestep,      vf);
> >      REGISTER_FILTER(FREI0R,         frei0r,         vf);
> > +    REGISTER_FILTER(FSPP,           fspp,           vf);
> >      REGISTER_FILTER(GEQ,            geq,            vf);
> >      REGISTER_FILTER(GRADFUN,        gradfun,        vf);
> >      REGISTER_FILTER(HALDCLUT,       haldclut,       vf);
> > diff --git a/libavfilter/libmpcodecs/vf_fspp.c
> b/libavfilter/libmpcodecs/vf_fspp.c
> > index d457859..3a80dc2 100644
> > --- a/libavfilter/libmpcodecs/vf_fspp.c
> > +++ b/libavfilter/libmpcodecs/vf_fspp.c
> > @@ -710,8 +710,8 @@ const vf_info_t ff_vf_info_fspp = {
> >  #if HAVE_MMX_INLINE
> >
> >  DECLARE_ASM_CONST(8, uint64_t, MM_FIX_0_382683433)=FIX64(0.382683433,
> 14);
> > -DECLARE_ALIGNED(8, uint64_t, ff_MM_FIX_0_541196100)=FIX64(0.541196100,
> 14);
> > -DECLARE_ALIGNED(8, uint64_t, ff_MM_FIX_0_707106781)=FIX64(0.707106781,
> 14);
> > +extern uint64_t ff_MM_FIX_0_707106781;
> > +extern uint64_t ff_MM_FIX_0_541196100;
>
> still looks weird
>

Carl Eugen Hoyos has commented on it in previous mails.

> +
> > +typedef int32_t int_simd16_t;
> > +static const int16_t FIX_0_382683433   = FIX(0.382683433, 14);
> > +static const int16_t FIX_0_541196100   = FIX(0.541196100, 14);
> > +static const int16_t FIX_0_707106781   = FIX(0.707106781, 14);
> > +static const int16_t FIX_1_306562965   = FIX(1.306562965, 14);
> > +static const int16_t FIX_1_414213562_A = FIX(1.414213562, 14);
> > +static const int16_t FIX_1_847759065   = FIX(1.847759065, 13);
> > +static const int16_t FIX_2_613125930   = FIX(-2.613125930, 13);
> > +static const int16_t FIX_1_414213562   = FIX(1.414213562, 13);
> > +static const int16_t FIX_1_082392200   = FIX(1.082392200, 13);
> > +
>
> > +typedef struct fsppContext {
>
> typedef struct {
>
> should be enough


The mul_thrmat function uses a pointer to this struct, so can't remove
that.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavfi-port-mp-uspp-to-a-native-libavfilter-filter.patch
Type: text/x-patch
Size: 91353 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141218/7f6f0a37/attachment.bin>