Christoph Reiter authored 2 years ago and Tristan Matthews committed 2 years ago

aarch64 code is referencing stdint types but the file is missing
the include.

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

With apologies to anyone still using cvs for speex development.

(cherry picked from speex commit fe47ce7a423155b58a74582e488a506b562dae6f)

LRFLEW authored 4 years ago and Tristan Matthews committed 4 years ago

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Karl Tomlinson authored 4 years ago and Ralph Giles committed 4 years ago

This was added in
0dd7bfeb
to address truncation reported at
http://lists.xiph.org/pipermail/speex-dev/2009-June/007277.html

Truncation of the most-signficant bit was occuring on conversion to
spx_word16_t in MULT16_32_Q15().  Changes to MULT16_32_Q15() mean this will no
longer occur.

The associated adjustment to the bit-shift before saturation was accidentally
removed in
0e5d424f
which caused the output to have half the intended amplitude when the
interpolating resampler was used.

Reported by Andreas Pehrson.

Karl Tomlinson authored 4 years ago and Ralph Giles committed 4 years ago

The 32-bit arg and return value have one bit more than required to represent
+/-32767 with 15 fractional bits.

Keeping the most significant bit allows saturation for 16-bit conversion to
sometimes be delayed until after these operations.

Roman Shterenzon authored 6 years ago and Tristan Matthews committed 6 years ago

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Reported by Mark Harris

 - Do not define macros, functions, or variables with file scope using
   names beginning with an underscore (these names are reserved for the
   implementation; see C89 section 7.1.3 or any later version) or that
   shadow other global declarations
 - Avoid declarations after statements (speex_assert) for C89 compat
 - Silence unused parameter warning in resampler_basic_zero
 - No need for the stack_alloc.h macros within #ifdef VAR_ARRAYS; use
   the standard C syntax
 - When OUTSIDE_SPEEX, define EXPORT if not already defined
 - Update URL to https

Frank Barchard authored 8 years ago and Tristan Matthews committed 6 years ago

port optimized inner_product_single and WORD2INT(x) for fixed
and floating point from 32 bit armv7 NEON to aarch64 NEON.

Avindra Goolcharan authored 7 years ago and Tristan Matthews committed 7 years ago

Minor fix to delete a single unused variable

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Dario Lombardo authored 8 years ago and Tristan Matthews committed 8 years ago

Add checks to avoid den_rate and num_rate to be set to 0.

Change-Id: Ia4880521e7ab73d0fdc44377f4badadb09365471
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Signed-off-by: Dario Lombardo <lomato@gmail.com>
Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Reported on Opus mailing list, see:
http://lists.xiph.org/pipermail/opus/2016-July/003591.html

speex_alloc already sets these arrays to 0.
Reported by Jean-Yves Avenard <jyavenard@mozilla.com>

Jean-Yves Avenard authored 8 years ago and Tristan Matthews committed 8 years ago

From bugzilla.mozilla.org bug #1266260

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Jean-Yves Avenard authored 8 years ago and Tristan Matthews committed 8 years ago

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Wim Taymans authored 9 years ago and Tristan Matthews committed 8 years ago

Add a test that resamples a sine wave with varying samplerates to create
a sweep.

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Wim Taymans authored 9 years ago and Tristan Matthews committed 9 years ago

Make sure we don't overflow when calculating the phase for the new
sample rate.

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Wim Taymans authored 9 years ago and Tristan Matthews committed 9 years ago

Use Euclids algorithm to calculate the greatest common divisor to
simplify the resample ratio fraction instead of the slow iterative
method.

Signed-off-by: Tristan Matthews <tmatth@videolan.org>

Reported-by: Fabian Henze <flyser42@gmx.de>

Signed-off-by: Tristan Matthews <le.businessman@gmail.com>

Signed-off-by: Tristan Matthews <le.businessman@gmail.com>

Signed-off-by: Tristan Matthews <le.businessman@gmail.com>

Patch adapted from https://trac.xiph.org/ticket/1298



Signed-off-by: Tristan Matthews <le.businessman@gmail.com>

Signed-off-by: Tristan Matthews <le.businessman@gmail.com>