[FFmpeg-devel] [PATCH 1/2] Remove the add bias hack for the C version of DSPContext.float_to_int16_*().

[Alex Converse]

On Thu, Jan 27, 2011 at 7:20 AM, Justin Ruggles
<justin.ruggles at gmail.com> wrote:
> ---
> ?libavcodec/aac.h ? ? ? ? ? | ? ?1 -
> ?libavcodec/aacdec.c ? ? ? ?| ? 42 +++++++++++++++------------------------
> ?libavcodec/aacsbr.c ? ? ? ?| ? 13 ++---------
> ?libavcodec/ac3dec.c ? ? ? ?| ? 13 +----------
> ?libavcodec/ac3dec.h ? ? ? ?| ? ?1 -
> ?libavcodec/binkaudio.c ? ? | ? ?5 ----
> ?libavcodec/dca.c ? ? ? ? ? | ? 47 +++++++++++++++++--------------------------
> ?libavcodec/dsputil.c ? ? ? | ? ?9 +-------
> ?libavcodec/dsputil.h ? ? ? | ? ?3 +-
> ?libavcodec/nellymoserdec.c | ? ?9 +-------
> ?libavcodec/vorbis_dec.c ? ?| ? 38 ++++++++---------------------------
> ?libavcodec/wmadec.c ? ? ? ?| ? 21 +-----------------
> ?12 files changed, 54 insertions(+), 148 deletions(-)
>

There are some further improvements you could make if you wanted to go
above and beyond like subbing in memcpy()s and checking if
dspfunctions fit new simplified loop calculations but this *looks* OK
as is.

What kind of testing did you do?

Regards,
Alex Converse