diff options
author | Mark Harris <mark.hsj@gmail.com> | 2017-02-20 19:51:40 -0800 |
---|---|---|
committer | Mark Harris <mark.hsj@gmail.com> | 2017-02-26 19:10:45 -0800 |
commit | d6d70371e85ec83307f6df0e067d353daa8e6f33 (patch) | |
tree | 9e118ef0267e2e0734cc9db4c5bbd6016aa0e089 /celt/vq.c | |
parent | 8056706f48880bbf0bb7ee842f1067b85f613353 (diff) | |
download | libopus-d6d70371e85ec83307f6df0e067d353daa8e6f33.tar.gz |
Fix compiler warnings
- celt/modes.c:430:14: warning: cast from 'const unsigned char *' to
'opus_int16 *' increases required alignment from 1 to 2 [-Wcast-align]
- 'C[0][1]' may be used uninitialized [-Wmaybe-uninitialized]
- Unused variable/parameter
- Value stored is never read
- MSVC warnings about "possible loss of data" due to type conversions
- MSVC warning C4146: unary minus operator applied to unsigned type
- silk/NLSF_del_dec_quant.c:137:20: warning: array subscript is above
array bounds [-Warray-bounds] (gcc -O3 false positive)
- src/mlp_train.h:39:20: warning: function declaration isn't a prototype
[-Wstrict-prototypes]
- Remove SMALL_FOOTPRINT code from SSE 4.1 FIR implementation, matching
the C implementation.
The clang -Wcast-align warnings with SSE intrinsics are a known
clang issue: https://llvm.org/bugs/show_bug.cgi?id=20670
Diffstat (limited to 'celt/vq.c')
-rw-r--r-- | celt/vq.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -214,7 +214,7 @@ opus_val16 op_pvq_search_c(celt_norm *X, int *iy, int K, int N, int arch) rcp = EXTRACT16(MULT16_32_Q16(K, celt_rcp(sum))); #else /* Using K+e with e < 1 guarantees we cannot get more than K pulses. */ - rcp = EXTRACT16(MULT16_32_Q16(K+0.8, celt_rcp(sum))); + rcp = EXTRACT16(MULT16_32_Q16(K+0.8f, celt_rcp(sum))); #endif j=0; do { #ifdef FIXED_POINT |