diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-09-28 19:17:56 +0200 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2022-10-04 18:01:01 +0200 |
| commit | e47f3e366bc0760dbd44b5d555c1bb601a1d5011 (patch) | |
| tree | 6996a91843dad42cbd1998e8082f9c5872f04b57 /libavcodec | |
| parent | 1e1ddbb70dd13c00c2e7378c17b3435325b856cf (diff) | |
| download | ffmpeg-e47f3e366bc0760dbd44b5d555c1bb601a1d5011.tar.gz | |
avcodec/g723_1enc: Remove unnecessary av_clipl_int32()
partial_corr is an int16_t and so the av_clipl_int32()
never clips and can be removed. This also avoids
undefined left-shifts of negative numbers.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/g723_1enc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/libavcodec/g723_1enc.c b/libavcodec/g723_1enc.c index f3baf7b4ec..a22985ca44 100644 --- a/libavcodec/g723_1enc.c +++ b/libavcodec/g723_1enc.c @@ -200,8 +200,7 @@ static void levinson_durbin(int16_t *lpc, int16_t *autocorr, int16_t error) partial_corr = temp / (error << 1); - lpc[i] = av_clipl_int32((int64_t) (partial_corr << 14) + - (1 << 15)) >> 16; + lpc[i] = (partial_corr + (1 << 1)) >> 2; /* Update the prediction error */ temp = MULL2(temp, partial_corr); |