This is what got submitted as version 01

This is the less-surprising behavior and will hopefully
result in fewer corner case bugs (e.g. losing gain
after seeking). This commit also updates the documentation.

The last commit changed some of the multistream tests to use a
dual-mono setup instead of a single-stereo setup. Some of the
accompanying explicit valgrind checks were still setup to
look for stereo streams.

This CTL was requested by Nicolas George for FFmpeg.

Thanks to Ron Lee for pointing it out.

Adapted from http://wiki.xiph.org/OggOpus

Ron authored 12 years ago and Gregory Maxwell committed 12 years ago

And explicitly specify what we want included in EXTRA_DIST of doc/Makefile.am,
otherwise make dist picks up whatever random junk you might have laying around
in that dir at the time you run it.

Tweak the top level delegation targets, because this means distclean will be
run in the doc dir by default now without our help.

Ron authored 12 years ago and Gregory Maxwell committed 12 years ago

We already report whether or not the docs will be built in the summary box
at the end of the configure run.

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

Also get the subname rigtht.

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.

root authored 12 years ago and Jean-Marc Valin committed 12 years ago

modified the Visual Studio 2010 projects and solution file to include a x64 configuration. modified the output folders such that the architecture is included in the path (so you get Release64, Debug32, etc.) so you can have all build types in one tree.

This was a problem for static builds

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

It's less surprising for people using the -uninstalled
pkg-config file if we advertise the static library rather
than the shared build. That way the executables work without
having to hack the same location into LD_LIBRARY_PATH, and
the resulting binaries are easier to distribute.

For this to actually work with opus-tools, the -fPIE had
to be turned off, as it was in 1d128b4d5c6f. Support
for static PIE builds would be restored if we compiled
libopus.a with -fPIC.

Also remove the private header locations. These are no
longer required to declare any of the public api.

Also enforces an upper bound of 510 kb/s even for frames that are
smaller than 20 ms. This reduces waste for high bitrate VBR.

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

With these changes, opus-tools can actually build using it.

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++.

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

On MacOS, stdlib.h ends up including sys/signal.h, generating
warnings about the local variables called 'signal' shadowing
the global symbol signal(3).

This was originally done in 86476906 but it missed some use
of 'signal' in prototypes in headers where it didn't cause
warnings. Later the prototypes were moved around and the
warnings came back.

This also cleans up some cases in where stdlib.h was used
but shouldn't be required.

Both are still built and installed by default though.

Instead of just its build stamp :)

As best we recall, this was used to version the
library filename during rapid development. It's
no longer needed now that the bitstream is frozen
and isn't hooked up to anything in the build system.

The config.h version defines were declared in the
middle of the math feature testing. This commit moves
it to the top of configure.ac where the other version
number code resides.