On Sat, Mar 15, 2014 at 11:56:24AM +0100, Anton Khirnov wrote: > On Fri, 14 Mar 2014 05:42:19 -0700, Diego Biurrun <[email protected]> wrote: > > --- > > libavcodec/arm/dsputil_init_armv6.c | 11 +++--- > > libavcodec/bfin/dsputil_init.c | 23 +++++++------ > > libavcodec/dsputil.c | 67 > > +++++++++++++++++-------------------- > > libavcodec/dsputil.h | 3 +- > > libavcodec/motion_est.c | 4 ++- > > libavcodec/ppc/dsputil_altivec.c | 21 ++++++------ > > libavcodec/x86/dsputilenc.asm | 7 ++-- > > libavcodec/x86/dsputilenc_mmx.c | 30 ++++++++--------- > > libavcodec/x86/motion_est.c | 20 ++++++----- > > 9 files changed, 95 insertions(+), 91 deletions(-) > > That parameter seems unused in the vast majority of cases, I wonder why is it > even passed.
Not sure what to make of this comment. It is used in some cases. I could create a separate function pointer type w/o that parameter, but this seems to quickly move out of scope .. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
