From http://vfrmaniac.fushizen.eu/contents/opus_in_isobmff.html

Based on http://vfrmaniac.fushizen.eu/contents/opus_in_isobmff.html

Considers the case where one stream would eat up all the remaining bytes,
including a 2-byte size that would then not leave enough for the even a PLC
packet in the next streams.

Limit now at 2*streams-1 and anything below that returns OPUS_BUFFER_TOO_SMALL
rather than OPUS_BAD_ARG

These are normative whether capitalized or not. Capitalize the ones
which make sense as normative requirements, and reword the rest.

We want to general guidelines for stream-level encapsulation to
apply to multiplexed as well as degenerate streams. 'Files' can
therefore specific to the mine-type discussion for unmultiplexed
Ogg physical streams.

These are the changes proposed in
 <http://www.ietf.org/mail-archive/web/codec/current/msg03058.html>,
 with some minor additions (see follow-up on the list).

cherry-picked from speexdsp 86779a06f6500d041573d6252d4971d3bfcb4b18

Promote the new tag definitions to a separate subsection,
balance tags, improve wording, and document our motivation
for the R128 tags.

Thanks to Calvin Walton for pointing this out.

Patch from Greg Maxwell in response to implementor feedback.
https://www.ietf.org/mail-archive/web/codec/current/msg03053.html

Conflicts:
	doc/draft-ietf-payload-rtp-opus.xml

Timothy B. Terriberry authored 10 years ago and Jean-Marc Valin committed 10 years ago

Here are my proposals to resolve a few issues with the current
 draft.
See the corresponding message to the list for details.

Timothy B. Terriberry authored 10 years ago and Jean-Marc Valin committed 10 years ago

This is the result of an editing pass for clarity and consistency.

genversion.bat would fail if an argument was passed with quotations.
This is because the quotations were not being stripped before using
them as variables, causing the _temp files not being created.

Adding ~ to the arguments causes windows to strip the quotations,
enabling it to be used with path names containing spaces and the
MSVS pre-build event.

Signed-off-by: Ralph Giles <giles@mozilla.com>

Marcello Caramma (mcaramma) authored 10 years ago and Jean-Marc Valin committed 10 years ago

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

Addresses comments from several months ago

OPUS_AUTO should now work fine, while using OPUS_BITRATE_MAX would still
be a bad idea for controlling CBR rate for multistream. The encoder should
now throw an error early on if max_data_bytes is just too small to do anything.

Koen Vos authored 10 years ago and Jean-Marc Valin committed 10 years ago

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

Koen Vos authored 10 years ago and Jean-Marc Valin committed 10 years ago

- Bug fix: the CNG synthesis filter output consisted of only the prediction
  values, now added the input to that.
- CNG excitation level now takes into account the level from the PLC (since
  both PLC and CNG can be active at the same time).
- CNG is now called before PLC_glue() to smoothen the transition out of CNG.

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

Rhishikesh Agashe authored 10 years ago and Jean-Marc Valin committed 10 years ago

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

When an overflow happened in silk_sum_sqr_shift(), the sample pair causing
the overflow would be counted twice. This would lead to the C0 in
silk_burg_modified() being too large and cause problems on loud, tonal
signals. Thanks to Marcello Caramma for tracking down the problem.

Use abs(sweep_bps) instead of fabs(). "sweep_bps" is an integer,
and on some compilers this line generates a warning.

Patch from Tina le Grand.
Reviewed by Jean-Marc Valin

The patch in 76e831d9 got us most of the way there, but
 out-of-tree builds required a second Makefile.am rule, which was
 missing @ARM2GNU_PARAMS@.
Also, the arm2gnu.pl was terminating argument processing on any
 argument beginning with --, rather than an argument that was just
 -- by itself (as is the normal convention in GNU programs).
That meant it never saw the --apple flag even when it was passed.

Thanks to Jonathan Lennox for the report and for testing.

Martin Storsjö authored 11 years ago and Timothy B. Terriberry committed 11 years ago

This allows building the arm assembly for iOS.

This checks for the __APPLE__ preprocessor built-in define to
determine whether this extra handling should be enabled.

Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>

Martin Storsjö authored 11 years ago and Timothy B. Terriberry committed 11 years ago

This avoids having to use the public symbol name when jumping here,
on platforms where the public symbols have an underscore prefix.

Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>

Bruno Randolf authored 11 years ago and Jean-Marc Valin committed 11 years ago

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