Remove a redundant negative length check in celt_decode.

Was previously testing only far on the negative side and using
the wrong scaling factor

their usage and to make any toolchain that doesn't like to #include
.c file happy.

Previously the autotools build litered whatever directory
'make' was invoked from with intermediate files. Adding
'subdir-objects' to AUTOMAKE_OPTIONS tells it to put them
in subdirectories according to the relative paths to the
source files, .i.e. silk/schur.o for $(srcdir)/silk/schur.c

The change to autogen.sh is necessary to handle the
(apparently new) case where an AUTOMAKE_OPTIONS line is
given without a minimum version number.

Thanks to Tim Terriberry for pointing out the issue.

This better distinguishes the reference implementation,
which is what this string is meant to do. Thanks to
Greg Maxwell for the suggestion.

This lets us verify this part of the API is working.

Remove trailing whitespace, convert tabs to 4 spaces, re-align
test_opus to use 4-space indents instead of 3, and re-wrap some
long lines.

This lets users print it out without context, which is a little
easier for clients.

Rather than compiling in the fixed version number from the configure
file, it's more helpful for testing and debugging to directly embed
the repository's revision. We use 'git describe --tags' for this, since
for tagged revisions it just returns the tag name, and in other cases
is more readable than a raw commit id.

If git isn't present or can't find a repository to query, we fall back
to the static version number in the configure script.

The new opus_get_version_string() call just returns a static string
defined by OPUS_VERSION, which is passed in from the build system
through config.h (or a custom compile line). Provide a fallback to
"unknown" if the build system fails to actually provide that definition.

This restores compilation with Makefile.draft.

In general, this means there will be builds out there with non-specific
version strings, since ports won't be forced to update the string. While
that's unfortunate, I think it's more valuable that the library be simple
to build.

silk_sources.mk no longer needs to be patched (s/if/ifdef/) for
the draft Makefile to work.

draft makefile. Increase precision of the static modes table.

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

whitespace.

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

non-ascii characters from the source.

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

Although the current encoder uses does not use them, the decoder
might have tried to use offsets beyond the end of these tables.
Inserting values here ensures that the stream can be decoded in a
well-defined way.
The particular values of these tables were chosen to match a
Gaussian distribution with sigma=3.93.

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

This removes the configurability of the OPUS_TEST_RANGE_CODER_STATE
consistency check, in favour of always running it. The performance
penalty is low, and it's an important check on correctness for the
reference encoder to provide.

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

Fix a warning on gcc 4.6. 'internal_sampling_rate_Hz' was
replaced by 'bandwidth' in a10e8796 but the local variable
holding the setting wasn't removed.

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

Automake version 1.11 and above provide a build mode where
each build command is replaced by a single-line annotation.
Since autotools build lines for libraries typically each fill
an entire terminal screen, this mode makes it easier to keep
track of build progress, and warnings stand out more clearly.

For debugging the build process, the old behaviour can be
recovered by running 'make V=1' at bulid time, or by passing
--disable-silent-rules to configure.

Since Automake 1.11 is still very recent, we only invoke
the AM_SILENT_RULES macro if it is defined, allowing fallback
to the old behaviour under previous versions of automake.

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

A comment in opus_decode() said, "bit 6 is ignored" when in
fact bit 6 is the padding flag, parsed in the previous block.

The comment, and the previous one, are modified to reflect
the current code.

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

I've changed the grouping of the options to place the performance-
related ones after the more serious thread-safety options. This
is still somewhat confusing as with C99 on wants -DHAVE_LRINTF
but NOT -Drestrict=

I've also added leading whitespace to the comment lines to better
distinguish them from the commented-out code lines.