On Thu, Nov 26, 2015 at 10:39 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > Hi, > > On Thu, Nov 26, 2015 at 10:23 AM, Ganesh Ajjanagadde > <gajjanaga...@gmail.com> wrote: >> >> #if CONFIG_HARDCODED_TABLES >> +#include "libavutil/tablegen.h" >> #if USE_FIXED >> #define cbrt_tableinit_fixed() >> #include "libavcodec/cbrt_fixed_tables.h" >> @@ -43,19 +45,19 @@ >> #include "libavcodec/cbrt_tables.h" >> #endif >> #else >> +#include "libavutil/libm.h" >> static uint32_t cbrt_tab[1 << 13]; > > > In cbrt_tablegen_template.c: > > #include <stdlib.h> > #define CONFIG_HARDCODED_TABLES 0 > #include "cbrt_tablegen.h" > #include "tableprint.h" > > So same comment as for the other patch. (Rest looks good, I think.)
Also noted that lrint needs to go into the avutil/tablegen.h. > > Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
