Hi, On Fri, Apr 29, 2011 at 2:08 PM, Diego Biurrun <[email protected]> wrote: > int/unsigned is the natural memory access type for CPUs, using sized types > for temporary variables, counters and similar just increases code size and > can possibly cause a slowdown. > --- > libavcodec/alpha/simple_idct_alpha.c | 6 +- > libavcodec/jfdctfst.c | 18 ++-- > libavcodec/jfdctint.c | 18 ++-- > libavcodec/vorbis.c | 9 +- > libavcodec/vorbis.h | 2 +- > libavcodec/vorbisdec.c | 227 > ++++++++++++++++------------------ > 6 files changed, 134 insertions(+), 146 deletions(-)
Don't touch the alpha code unless you have access to one. Didn't you just commit such changes to a vorbis decoder already? I'm confused about them re-appearing here. Regardless, speed tests are missing to confirm no significant slowdown. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
