[FFmpeg-cvslog] avcodec/nellymoserenc: fix sign error
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Fri Nov 14 19:52:45 CET 2014
On Thu, Nov 13, 2014 at 09:13:23PM +0100, Michael Niedermayer wrote:
> ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Thu Nov 13 20:40:37 2014 +0100| [5c805d69a49a1f32a7a8a1b16fb3d631d85ca56d] | committer: Michael Niedermayer
>
> avcodec/nellymoserenc: fix sign error
>
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
>
> > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=5c805d69a49a1f32a7a8a1b16fb3d631d85ca56d
> ---
>
> libavcodec/nellymoserenc.c | 2 +-
> tests/fate/audio.mak | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/libavcodec/nellymoserenc.c b/libavcodec/nellymoserenc.c
> index 9bbdd52..78f1092 100644
> --- a/libavcodec/nellymoserenc.c
> +++ b/libavcodec/nellymoserenc.c
> @@ -175,7 +175,7 @@ static av_cold int encode_init(AVCodecContext *avctx)
> /* Generate overlap window */
> ff_init_ff_sine_windows(7);
> for (i = 0; i < POW_TABLE_SIZE; i++)
> - pow_table[i] = -pow(2, -i / 2048.0 - 3.0 + POW_TABLE_OFFSET);
> + pow_table[i] = pow(2, -i / 2048.0 - 3.0 + POW_TABLE_OFFSET);
Hm, I think you should then also change this line:
static float pow_table[POW_TABLE_SIZE]; ///< -pow(2, -i / 2048.0 - 3.0);
(note that I have no clue which would be right and why, though the
fate test seems to agree it is better)
More information about the ffmpeg-cvslog
mailing list