[FFmpeg-devel] [PATCH 1/2] Remove unneeded add bias from 3 functions.

[Måns Rullgård]

Justin Ruggles <justin.ruggles at gmail.com> writes:

> On 01/31/2011 03:38 PM, M?ns Rullg?rd wrote:
>
>> Justin Ruggles <justin.ruggles at gmail.com> writes:
>> 
>>> DSPContext.vector_fmul_window()
>>> DCADSPContext.lfe_fir()
>>> SynthFilterContext.synth_filter_float()
>>> ---
>>>  libavcodec/aacdec.c                |   20 ++++++++++----------
>>>  libavcodec/ac3dec.c                |    4 ++--
>>>  libavcodec/arm/dcadsp_init_arm.c   |    2 +-
>>>  libavcodec/arm/dcadsp_neon.S       |    5 ++---
>>>  libavcodec/arm/dsputil_init_neon.c |    3 +--
>>>  libavcodec/arm/dsputil_neon.S      |   15 +++++----------
>>>  libavcodec/arm/fft_init_arm.c      |    2 +-
>>>  libavcodec/arm/synth_filter_neon.S |    8 +++-----
>>>  libavcodec/atrac1.c                |    2 +-
>>>  libavcodec/dca.c                   |    4 ++--
>>>  libavcodec/dcadsp.c                |    6 +++---
>>>  libavcodec/dcadsp.h                |    2 +-
>>>  libavcodec/dsputil.c               |   10 ++++++----
>>>  libavcodec/dsputil.h               |    5 +----
>>>  libavcodec/ppc/float_altivec.c     |   14 ++++----------
>>>  libavcodec/synth_filter.c          |    6 +++---
>>>  libavcodec/synth_filter.h          |    2 +-
>>>  libavcodec/twinvq.c                |    1 -
>>>  libavcodec/vorbis_dec.c            |    6 +++---
>>>  libavcodec/wmaprodec.c             |    2 +-
>>>  libavcodec/x86/dsputil_mmx.c       |   20 ++++++++------------
>>>  21 files changed, 59 insertions(+), 80 deletions(-)
>> 
>> Is this the same as what I sent apart from the x86 bits?
>
> It should be.  The patch didn't apply cleanly, so I had to do 2 files
> manually.

I think I had another patch of yours under it.

> If you could double-check the ARM part to make sure it builds and
> passes fate that would be great.

diff says you got it right, and fate agrees.  Pushed.

-- 
M?ns Rullg?rd
mans at mansr.com