Hi,
On Mon, Aug 17, 2015 at 5:45 PM, wm4 <nfx...@googlemail.com> wrote:
> On Mon, 17 Aug 2015 15:41:03 -0400
> "Ronald S. Bultje" <rsbul...@gmail.com> wrote:
>
> > ---
> > libavcodec/avcodec.h | 2 --
> > libavcodec/avdct.c | 2 ++
> > libavcodec/options_table.h | 2 --
> > 3 files changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> > index 6b824d5..6f9b026 100644
> > --- a/libavcodec/avcodec.h
> > +++ b/libavcodec/avcodec.h
> > @@ -2947,9 +2947,7 @@ typedef struct AVCodecContext {
> > int dct_algo;
> > #define FF_DCT_AUTO 0
> > #define FF_DCT_FASTINT 1
> > -#if FF_API_UNUSED_MEMBERS
> > #define FF_DCT_INT 2
> > -#endif /* FF_API_UNUSED_MEMBERS */
> > #define FF_DCT_MMX 3
> > #define FF_DCT_ALTIVEC 5
> > #define FF_DCT_FAAN 6
> > diff --git a/libavcodec/avdct.c b/libavcodec/avdct.c
> > index f92c691..3b622ba 100644
> > --- a/libavcodec/avdct.c
> > +++ b/libavcodec/avdct.c
> > @@ -58,7 +58,9 @@ static const AVOption avdct_options[] = {
> > #if FF_API_ARCH_ALPHA
> > {"simplealpha", "experimental / for debugging", 0, AV_OPT_TYPE_CONST,
> {.i64 = FF_IDCT_SIMPLEALPHA }, INT_MIN, INT_MAX, V|E|D, "idct"},
> > #endif
> > +#if FF_API_UNUSED_MEMBERS
> > {"ipp", "experimental / for debugging", 0, AV_OPT_TYPE_CONST, {.i64 =
> FF_IDCT_IPP }, INT_MIN, INT_MAX, V|E|D, "idct"},
> > +#endif
> > {"xvid", "experimental / for debugging", 0, AV_OPT_TYPE_CONST, {.i64 =
> FF_IDCT_XVID }, INT_MIN, INT_MAX, V|E|D, "idct"},
> > {"xvidmmx", "experimental / for debugging", 0, AV_OPT_TYPE_CONST, {.i64
> = FF_IDCT_XVID }, INT_MIN, INT_MAX, V|E|D, "idct"},
> > {"faani", "floating point AAN IDCT (experimental / for debugging)", 0,
> AV_OPT_TYPE_CONST, {.i64 = FF_IDCT_FAAN }, INT_MIN, INT_MAX, V|D|E, "idct"},
> > diff --git a/libavcodec/options_table.h b/libavcodec/options_table.h
> > index fd9c045..8dabb65 100644
> > --- a/libavcodec/options_table.h
> > +++ b/libavcodec/options_table.h
> > @@ -203,9 +203,7 @@ static const AVOption avcodec_options[] = {
> > {"dct", "DCT algorithm", OFFSET(dct_algo), AV_OPT_TYPE_INT, {.i64 =
> DEFAULT }, 0, INT_MAX, V|E, "dct"},
> > {"auto", "autoselect a good one (default)", 0, AV_OPT_TYPE_CONST, {.i64
> = FF_DCT_AUTO }, INT_MIN, INT_MAX, V|E, "dct"},
> > {"fastint", "fast integer", 0, AV_OPT_TYPE_CONST, {.i64 =
> FF_DCT_FASTINT }, INT_MIN, INT_MAX, V|E, "dct"},
> > -#if FF_API_UNUSED_MEMBERS
> > {"int", "accurate integer", 0, AV_OPT_TYPE_CONST, {.i64 = FF_DCT_INT },
> INT_MIN, INT_MAX, V|E, "dct"},
> > -#endif /* FF_API_UNUSED_MEMBERS */
> > {"mmx", NULL, 0, AV_OPT_TYPE_CONST, {.i64 = FF_DCT_MMX }, INT_MIN,
> INT_MAX, V|E, "dct"},
> > {"altivec", NULL, 0, AV_OPT_TYPE_CONST, {.i64 = FF_DCT_ALTIVEC },
> INT_MIN, INT_MAX, V|E, "dct"},
> > {"faan", "floating point AAN DCT", 0, AV_OPT_TYPE_CONST, {.i64 =
> FF_DCT_FAAN }, INT_MIN, INT_MAX, V|E, "dct"},
>
> I guess these weren't really unused, while "ipp" can go?
Right, "int" is used in various fate tests and I can't be bothered to
figure out whether that's useful or not. "ipp" is unused except for the
mention in avdct.c, so removing that makes everyone happy.
Ronald
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
