Kostya <[email protected]> writes: > On Mon, Jul 04, 2011 at 11:54:37AM +0100, Mans Rullgard wrote: >> The reference function can be inferred from the direction of the >> transform, no need for storing it with each entry. >> >> Signed-off-by: Mans Rullgard <[email protected]> >> --- >> libavcodec/dct-test.c | 56 >> ++++++++++++++++++++++++------------------------ >> 1 files changed, 28 insertions(+), 28 deletions(-) >> >> diff --git a/libavcodec/dct-test.c b/libavcodec/dct-test.c >> index cd68d48..689aef8 100644 >> --- a/libavcodec/dct-test.c >> +++ b/libavcodec/dct-test.c >> @@ -70,7 +70,6 @@ void ff_simple_idct_axp(DCTELEM *data); >> struct algo { >> const char *name; >> void (*func)(DCTELEM *block); >> - void (*ref) (DCTELEM *block); >> enum formattag { NO_PERM, MMX_PERM, MMX_SIMPLE_PERM, SCALE_PERM, >> SSE2_PERM, PARTTRANS_PERM } format; >> int mm_support; >> @@ -86,65 +85,65 @@ struct algo { >> static int cpu_flags; >> >> static const struct algo fdct_tab[] = { >> - {"REF-DBL", ff_ref_fdct, ff_ref_fdct, NO_PERM}, >> - {"FAAN", ff_faandct, ff_ref_fdct, FAAN_SCALE}, >> - {"IJG-AAN-INT", fdct_ifast, ff_ref_fdct, SCALE_PERM}, >> - {"IJG-LLM-INT", ff_jpeg_fdct_islow, ff_ref_fdct, NO_PERM}, >> + { "REF-DBL", ff_ref_fdct, NO_PERM }, >> + { "FAAN", ff_faandct, FAAN_SCALE }, >> + { "IJG-AAN-INT", fdct_ifast, SCALE_PERM }, >> + { "IJG-LLM-INT", ff_jpeg_fdct_islow, NO_PERM }, >> >> #if HAVE_MMX >> - {"MMX", ff_fdct_mmx, ff_ref_fdct, NO_PERM, >> AV_CPU_FLAG_MMX}, >> - {"MMX2", ff_fdct_mmx2, ff_ref_fdct, NO_PERM, >> AV_CPU_FLAG_MMX2}, >> - {"SSE2", ff_fdct_sse2, ff_ref_fdct, NO_PERM, >> AV_CPU_FLAG_SSE2}, >> + { "MMX", ff_fdct_mmx, NO_PERM, AV_CPU_FLAG_MMX >> }, >> + { "MMX2", ff_fdct_mmx2, NO_PERM, AV_CPU_FLAG_MMX2 >> }, >> + { "SSE2", ff_fdct_sse2, NO_PERM, AV_CPU_FLAG_SSE2 >> }, >> #endif >> >> #if HAVE_ALTIVEC >> - {"altivecfdct", fdct_altivec, ff_ref_fdct, NO_PERM, >> AV_CPU_FLAG_ALTIVEC}, >> + { "altivecfdct", fdct_altivec, NO_PERM, >> AV_CPU_FLAG_ALTIVEC }, >> #endif >> >> #if ARCH_BFIN >> - {"BFINfdct", ff_bfin_fdct, ff_ref_fdct, NO_PERM}, >> + { "BFINfdct", ff_bfin_fdct, NO_PERM }, >> #endif >> >> { 0 } >> }; >> >> static const struct algo idct_tab[] = { >> - {"FAANI", ff_faanidct, ff_ref_idct, NO_PERM}, >> - {"REF-DBL", ff_ref_idct, ff_ref_idct, NO_PERM}, >> - {"INT", j_rev_dct, ff_ref_idct, MMX_PERM}, >> - {"SIMPLE-C", ff_simple_idct, ff_ref_idct, NO_PERM}, >> + { "FAANI", ff_faanidct, NO_PERM }, >> + { "REF-DBL", ff_ref_idct, NO_PERM }, >> + { "INT", j_rev_dct, MMX_PERM }, >> + { "SIMPLE-C", ff_simple_idct, NO_PERM }, >> >> #if HAVE_MMX >> #if CONFIG_GPL >> - {"LIBMPEG2-MMX", ff_mmx_idct, ff_ref_idct, MMX_PERM, >> AV_CPU_FLAG_MMX, 1}, >> - {"LIBMPEG2-MMX2", ff_mmxext_idct, ff_ref_idct, MMX_PERM, >> AV_CPU_FLAG_MMX2, 1}, >> + { "LIBMPEG2-MMX", ff_mmx_idct, MMX_PERM, AV_CPU_FLAG_MMX, >> 1 }, >> + { "LIBMPEG2-MMX2", ff_mmxext_idct, MMX_PERM, AV_CPU_FLAG_MMX2, >> 1 }, >> #endif >> - {"SIMPLE-MMX", ff_simple_idct_mmx, ff_ref_idct, MMX_SIMPLE_PERM, >> AV_CPU_FLAG_MMX}, >> - {"XVID-MMX", ff_idct_xvid_mmx, ff_ref_idct, NO_PERM, >> AV_CPU_FLAG_MMX, 1}, >> - {"XVID-MMX2", ff_idct_xvid_mmx2, ff_ref_idct, NO_PERM, >> AV_CPU_FLAG_MMX2, 1}, >> - {"XVID-SSE2", ff_idct_xvid_sse2, ff_ref_idct, SSE2_PERM, >> AV_CPU_FLAG_SSE2, 1}, >> + { "SIMPLE-MMX", ff_simple_idct_mmx, MMX_SIMPLE_PERM, >> AV_CPU_FLAG_MMX }, >> + { "XVID-MMX", ff_idct_xvid_mmx, NO_PERM, AV_CPU_FLAG_MMX, >> 1 }, >> + { "XVID-MMX2", ff_idct_xvid_mmx2, NO_PERM, AV_CPU_FLAG_MMX2, >> 1 }, >> + { "XVID-SSE2", ff_idct_xvid_sse2, SSE2_PERM, AV_CPU_FLAG_SSE2, >> 1 }, >> #endif >> >> #if ARCH_BFIN >> - {"BFINidct", ff_bfin_idct, ff_ref_idct, NO_PERM}, >> + { "BFINidct", ff_bfin_idct, NO_PERM }, >> #endif >> >> #if ARCH_ARM >> - {"SIMPLE-ARM", ff_simple_idct_arm, ff_ref_idct, NO_PERM }, >> - {"INT-ARM", ff_j_rev_dct_arm, ff_ref_idct, MMX_PERM }, >> + { "SIMPLE-ARM", ff_simple_idct_arm, NO_PERM }, >> + { "INT-ARM", ff_j_rev_dct_arm, MMX_PERM }, >> #endif >> #if HAVE_ARMV5TE >> - {"SIMPLE-ARMV5TE", ff_simple_idct_armv5te, ff_ref_idct, NO_PERM }, >> + { "SIMPLE-ARMV5TE", ff_simple_idct_armv5te,NO_PERM }, > > Maybe you should've formatted it with an additional space before the third > column.
This was the most sane formatting I could think of allowing the lines to fit in 80 columns. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
