The command line has changed, for the better, but we need to
update the way we call it. Also adds an install hint if it's
not found on the path.

NB I tried to detect and fallback to the old syntax for the
version 1 (TCL) implementation but this is hard because it
doesn't support any switches (like --version) and it no
longer parses our remote entities.

The max reduction now depends on intensity threshold too.

Some tests like Kamedo's HA test suggested that our tonal boost was a bit
too extreme.

NB they only check for len < 1, not for null data.

This should reduce computation, reduce stack usage, and be
 substantially easier to read, but should not change behavior.
It's not quite bit-exact because I collapsed the application of the
 decay and the fading into a single pass to save one multiply per
 sample, but the difference signal is silent.

It also changes the comments into complete sentences.

Gregory Maxwell authored 12 years ago and Jean-Marc Valin committed 12 years ago

Also remove a useless extern that crept into the tests from some
optimization driven testcase generation.

Calling PLC/FEC with a different size was not updating it

Library soname versioning is something we normally change
only right before each release. However, with 1.0.2 being
released from the 1.0.x branch with 3.0.3, it makes sense
to make the same update here. That way when we do a new
release from master we can just increment again and have
the version properly reflect ranges relative to the stable
branch.

This applies in addition to the earlier grant made by
Skype, which was purchased by Microsoft.

PLC and FEC now return exactly the number of samples specified for the
buffer rather than (usually) returning the size of the last packet.
Doc and tests are updated accordingly.

This avoids linker errors with VS2010 of the form
    failure during conversion to COFF: invalid file or corrupt
    [c:\users\giles\opus\source\opus_demo.vcxproj]

The problem does not occur if the project files are updated
to build under VS2012.

Nils Wallménius authored 12 years ago and Jean-Marc Valin committed 12 years ago

Gets rid of one stack buffer.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Adds more sizes to test_unit_mdct.c

Nils Wallménius authored 12 years ago and Jean-Marc Valin committed 12 years ago

Place output of imdct post-rotate and deshuffle in out buffer to avoid copying and simplify the code.

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

Nils Wallménius authored 12 years ago and Jean-Marc Valin committed 12 years ago

Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>

This is a heavily modified version of the demonstration program
Jüri Aedla posted to the mailing list. Verified to pass with
the current commit, but fail with the tree from two commits ago.

http://lists.xiph.org/pipermail/opus/2012-November/001834.html

This was reported by Juri Aedla and is limited to reading memory up
to about 60 kB beyond the compressed buffer. This can only be triggered
by a compressed packet more than about 16 MB long, so it's not a problem
for RTP. In theory, it *could* crash an Ogg decoder if the memory just after
the incoming packet is out-of-range.

We don't ever use the WORDS_BIGENDIAN cpp symbol in our code.

Was causing problems with the PLC

Previous code was so broken that it's`not worth listing the changes.
Aside from quality, one obvious improvement is a 3.75k reduction in stack size