Age | Commit message (Collapse) | Author |
|
|
|
Add newer source directories to the destdir file tree so we
can include all sources referenced from opus_sources.mk.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
[[ ]], the compound command is not supported by all
shell interpreter. [ ], the buildin command is more
common.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
CMAKE_SOURCE_DIR corresponds to the top project's source directory.
CMAKE_BINARY_DIR corresponds to the top project's binary directory.
The usage of these variables doesn't cause any problems when Opus is built as a standalone project.
This is not the case when Opus is added as submodule: the variables are set by the project that calls "add_subdirectory()".
The fix consists in using PROJECT_SOURCE_DIR and PROJECT_BINARY_DIR, which always refer to the current project.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
This lighter-weight intrinsics header is available starting
with Microsoft Visual Studio 2017, so the previous change
to allow this header failed when building with Visual
Studio 2015.
Restores the appveyor continuous integration build.
|
|
remove usage of stdio.h in production code
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
in tests.
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
Add doc/trivial_example.c to the autotools build so we get
some minimal verification that the code compiles.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Silence a gcc warning by checking the return value of the fread()
call instead of the feof() guard. This prevents an infinite loop
in the case of a read error. Otherwise, when end-of-file is reached
fread() will certainly return a smaller number of elements read
than requested, so both cases are handled now.
Add a comment to clarify that we're dropping a partial frame on
purpose to keep the code simple.
Also add more braces around conditional bodies for less error-prone
style.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
The simple codec round-trip example file in the doc directory
opens an input and output pcm file. It was working fine on
POSIX systems, but not on Windows, which treats text files
differently.
This is confusing in a example, so it's better to add an
explicit binary flag to the fopen() calls. This does nothing
on unix-like systems, but should make the example work for
developers on Windows.
Thanks to Wavesonics who reported this on irc.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Hector Martin <marcan@marcan.st>
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
|
|
Describe builds for the gitlab continuous integration
service runners. This does a trial build under both
autotools and cmake, so we get some coverage for changes
on that hosting platform.
After the same script in the vorbis and ogg projects.
Signed-off-by: Mark Harris <mark.hsj@gmail.com>
|
|
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
The xml2rfc DTD requires a <date> tag within reference > front.
Where the reference was to a published article, I tried to add
the correct publication date. For others, like references to
wikipedia, I just added an empty tag, since at this point
there's no way to recover a retrieval date.
With this change the draft is considered valid by xml2rfc 2.43.0.
However, it fails to include the externally generated text
for opus_source.base64.
|
|
Corrects an xml2rfc warning. Apparently text anchors can't
contain whitespace.
|
|
Corrects a warning on xml2rfc.
|
|
Switch to the newer style arguments to build both text and html
versions of the ietf drafts, in line with what build_oggdraft.sh
does.
Current xml2rfc (v2.43.0) no longer supports the old syntax, so
the script was just creating the text/plain version twice.
|
|
Use the more secure urls by default.
|
|
It think the [[ ]] conditional expression brackets are a bashism.
They don't work on Debian's /bin/sh (which is dash). Use the more
portable `test` built-in instead.
|
|
Add newer source directories to the destdir file tree so we
can include all sources referenced from opus_sources.mk.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
All the drafts are currently published, so this is not
particularly important. However, the code bitrots if not
tested regularly, so this will remind us to maintain what
we have in the tree.
xml2rfc is installed with pip.
Using a python base image makes for a smaller download than
installing python on the gcc image used for the other build
jobs.
|
|
|
|
Do a trial build under autotools and cmake. After the same
script in the vorbis project.
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
- Fix typo in OPUS_USE_NEON description.
- Set OPUS_PRESUME_NEON for iOS platforms as all armv7 and higher
iOS devices support NEON.
- Fix detection of aarch64 for OPUS_CPU_ARM and adding sources from
celt_sources_arm. (previously would miss armcpu.c and arm_celt_map.c)
- Change "armv7-a" to "arm" in MATCHES checks against
CMAKE_SYSTEM_PROCESSOR as systems like the RPi3 report as
"armv7l".
- Rename OPUS_MAY_SUPPORT_NEON to OPUS_MAY_HAVE_NEON as this name is
used everywhere else in the CMake build system. Without this,
runtime capability detection is broken on aarch64.
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
non-windows when target use runtime check (GH #154). For windows we only use /arch flag when target is presumed to support SSE to avoid AVX function pollution (GH #132).
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
dynamic library
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Install include directory must be `include/opus`, not `include`.
Old path is still here for compatibility.
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
When using OPUS_CUSTOM, `CELTDecoder->end` can be larger than 21.
Assert against 25 instead in OPUS_CUSTOM builds.
Signed-off-by: Jean-Marc Valin <jmvalin@jmvalin.ca>
|
|
|
|
|