Mark Harris authored 9 years ago and Jean-Marc Valin committed 9 years ago

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

Conflicts:
	src/opus_multistream_encoder.c

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.

Philip Jägenstedt authored 12 years ago and Gregory Maxwell committed 12 years ago

I stumbled upon the typo in README.draft, so took the opportunity to
grep for common misspellings using List_of_common_misspellings.txt for
hunspell.

This could be confusing/misleading.

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

This patch also includes some small edits/additions to the main API
 documentation.

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

In places where an ignored return or a null-arg is a sure indication
of a bug add the GCC warning attributes. The null arg annotation
is not enable for Opus itself because it will cause the compiler
to optimize out some null checks. I don't trust our callers
quite that much.

Unlike the other headers, opus_multistream.h didn't declare
itself to be C code, causing confusing errors when the caller
tries to use the functions from C++.

This reverts commit 9f407afa.

Also added 3rd clause to "master" COPYING file

Ralph Giles authored 13 years ago and Jean-Marc Valin committed 13 years ago

Correct member comments copied from the non-multistream api,
and document the remaining undocumented members. Still very
rough.

Untested

Doxygen uses comments in the source code to generate documentation.
This commit just adds the descriptive headers and marks current
comments and appropriate for inclusion in the API docs. Many
symbols remain undocumented. See the warnings from the build output.

Only tested for the single-stream case!