On Thu, Oct 04, 2012 at 06:43:16PM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > > --- > > libavcodec/Makefile | 7 +---- > > libavcodec/arm/mathops.h | 24 ++++++++++++++++++ > > libavcodec/inverse.c | 1 - > > libavutil/inverse.c => libavcodec/mathops.c | 14 ++++++++-- > > libavcodec/mathops.h | 29 ++++++++++++++++++++++ > > libavcodec/motion_est.c | 2 +- > > libavcodec/mpegvideo.c | 2 +- > > libavcodec/mpegvideo_enc.c | 1 + > > libavcodec/ra144.c | 2 +- > > libavcodec/roqaudioenc.c | 2 +- > > libavutil/Makefile | 1 - > > libavutil/arm/intmath.h | 24 ------------------ > > libavutil/intmath.h | 35 > > --------------------------- > > libavutil/mathematics.c | 11 -------- > > 14 files changed, 70 insertions(+), 85 deletions(-) > > delete mode 100644 libavcodec/inverse.c > > rename libavutil/inverse.c => libavcodec/mathops.c (79%) > > I'd prefer to see this split in one patch per table/macro.
ff_sqrt depends on ff_sqrt_tab and FASTDIV, which depends on ff_inverse. So I don't see how this could be split. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
