mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-12-23 12:43:46 +02:00
swresample/rematrix: Use clipping s16 rematrixing if overflows are possible
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 2f76157eb0
)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
parent
500cf2e159
commit
67c7f8ca14
@ -32,6 +32,9 @@
|
||||
|
||||
#define TEMPLATE_REMATRIX_S16
|
||||
#include "rematrix_template.c"
|
||||
#define TEMPLATE_CLIP
|
||||
#include "rematrix_template.c"
|
||||
#undef TEMPLATE_CLIP
|
||||
#undef TEMPLATE_REMATRIX_S16
|
||||
|
||||
#define TEMPLATE_REMATRIX_S32
|
||||
@ -362,23 +365,33 @@ av_cold int swri_rematrix_init(SwrContext *s){
|
||||
return r;
|
||||
}
|
||||
if (s->midbuf.fmt == AV_SAMPLE_FMT_S16P){
|
||||
int maxsum = 0;
|
||||
s->native_matrix = av_calloc(nb_in * nb_out, sizeof(int));
|
||||
s->native_one = av_mallocz(sizeof(int));
|
||||
if (!s->native_matrix || !s->native_one)
|
||||
return AVERROR(ENOMEM);
|
||||
for (i = 0; i < nb_out; i++) {
|
||||
double rem = 0;
|
||||
int sum = 0;
|
||||
|
||||
for (j = 0; j < nb_in; j++) {
|
||||
double target = s->matrix[i][j] * 32768 + rem;
|
||||
((int*)s->native_matrix)[i * nb_in + j] = lrintf(target);
|
||||
rem += target - ((int*)s->native_matrix)[i * nb_in + j];
|
||||
sum += FFABS(((int*)s->native_matrix)[i * nb_in + j]);
|
||||
}
|
||||
maxsum = FFMAX(maxsum, sum);
|
||||
}
|
||||
*((int*)s->native_one) = 32768;
|
||||
s->mix_1_1_f = (mix_1_1_func_type*)copy_s16;
|
||||
s->mix_2_1_f = (mix_2_1_func_type*)sum2_s16;
|
||||
s->mix_any_f = (mix_any_func_type*)get_mix_any_func_s16(s);
|
||||
if (maxsum <= 32768) {
|
||||
s->mix_1_1_f = (mix_1_1_func_type*)copy_s16;
|
||||
s->mix_2_1_f = (mix_2_1_func_type*)sum2_s16;
|
||||
s->mix_any_f = (mix_any_func_type*)get_mix_any_func_s16(s);
|
||||
} else {
|
||||
s->mix_1_1_f = (mix_1_1_func_type*)copy_clip_s16;
|
||||
s->mix_2_1_f = (mix_2_1_func_type*)sum2_clip_s16;
|
||||
s->mix_any_f = (mix_any_func_type*)get_mix_any_func_clip_s16(s);
|
||||
}
|
||||
}else if(s->midbuf.fmt == AV_SAMPLE_FMT_FLTP){
|
||||
s->native_matrix = av_calloc(nb_in * nb_out, sizeof(float));
|
||||
s->native_one = av_mallocz(sizeof(float));
|
||||
|
@ -31,11 +31,16 @@
|
||||
# define INTER double
|
||||
# define RENAME(x) x ## _double
|
||||
#elif defined(TEMPLATE_REMATRIX_S16)
|
||||
# define R(x) (((x) + 16384)>>15)
|
||||
# define SAMPLE int16_t
|
||||
# define COEFF int
|
||||
# define INTER int
|
||||
# ifdef TEMPLATE_CLIP
|
||||
# define R(x) av_clip_int16(((x) + 16384)>>15)
|
||||
# define RENAME(x) x ## _clip_s16
|
||||
# else
|
||||
# define R(x) (((x) + 16384)>>15)
|
||||
# define RENAME(x) x ## _s16
|
||||
# endif
|
||||
#elif defined(TEMPLATE_REMATRIX_S32)
|
||||
# define R(x) (((x) + 16384)>>15)
|
||||
# define SAMPLE int32_t
|
||||
|
Loading…
Reference in New Issue
Block a user