| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
Verify that the CMake build works from a release tarball.
This will give coverage for problems with part of the
CMake build description not being included in the
source package. We still consider the `make dist`
output from the GNU autotools build to be canonical.
|
|
|
|
|
|
| |
NB CMakelists was in the repo at the time of the 1.3.3 release,
but didn't make it into the 1.3.3 dist target, so it can't be
said to be available for that release.
|
|
|
|
|
|
| |
cmake in build dir is recommand by officially.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
| |
|
| |
|
|
|
|
|
|
|
| |
Use the correct library filename and only package the dll
if the build produced one.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
|
|
|
| |
The 2015 version of the project files can be opened by newer
versions of Visual Studio, and we want to deprecate the project
files in favour of the CMake build, so it doesn't make sense
to support three different versions.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Use <sys/types.h> not <inttypes.h> when building for Apple platforms,
as it works with modular dynamic-framework builds.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
| |
Copy the struct definition from ogg.h which has better comment
alignment.
|
|
|
|
|
|
|
| |
The pageno is declared `long` in the header, not `int`, although
it's a 32-bit value in the page header.
Thanks to Spencer Russell for reporting the issue.
|
|
|
|
|
|
|
|
| |
oy.data memory is alloc by ogg_sync_buffer(), but does not call free()
before main() exit. After fixing it, I test test_framing by valgrind.
And no more memory leak in test_framing and test_bitwise right now.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
|
|
|
|
|
| |
According to VS2017 docs, the initial LIBRARY line must include
a dll name. MSVC handles the missing name, but MinGW does not.
Removing the line entirely is also accepted by both toolchains,
but is contrary to documentation.
Based on a patch by evpobr in https://github.com/xiph/ogg/pull/42
No ABI change according to comments there.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The macOS system image already provides a version of xz through
homebrew, so we don't need to explicitly install it through
`brew bundle`.
Skipping the `brew update` step reduces the runtime of test jobs
from 6 minutes to 1, so we get test feedback more promptly.
The xz package is really only needed for 'make dist' targets,
so listing it in a Brewfile doesn't help users get started
building the library. Better not to clutter the top-level
directory further.
|
|
|
|
|
|
|
| |
The DEB package generator isn't available in the cmake version
provided by travis-ci.org, so only skip invoking it there.
Unbreaks cmake build tests after adding macOS coverage.
|
|
|
|
|
|
|
|
| |
Test on Linux and macOS with both gcc and clang.
Ubuntu 14.04 (trusty) is now the default travis environment.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
|
|
|
|
| |
* Use build_script step in AppVeyor configuration. This is a
preparation for building with multiple build systems.
* Add BUILD_SYSTEM env variable to AppVeyor build matrix. This
allows to switch between different build systems when building
multiple test matrix configurations.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
| |
Update cut-and-paste description of the reset functions to reference
resetting instead of clearing.
|
|
|
|
|
|
|
| |
We call ogg_stream_init() in main() of framing.c, but no
ogg_stream_clear() in corresponding. It will cause memory leak.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|\
| |
| |
| | |
The git.xiph.org and github branches had diverged.
|
| |
| |
| | |
@var@ is replaced by configure when autoconf or cmake when use cmake
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
Just cleanup.
Output path is relative to build directory anyway according
to documentation.
Related to #9.
|
| | |
|
|/ |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes a cppcheck warning. Possibly we should just convert
the array type to `long` since that's what oggpack_look()
returns, using negative values to report error.
However, none of the compared values are out of range
for either type so it doesn't really matter.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
| |
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
| |
|
|
|
|
| |
Reported by Mark Harris
|
|
|
|
|
|
| |
Suggested-By Mark Harris
Fixes #2297
|
| |
|
| |
|
|
|
|
|
|
| |
Found via `codespell -q 3`
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
|
| |
|
|
|
|
| |
Fixes xiph/ogg#35
|
|
|
|
| |
Signed-off-by: Tristan Matthews <tmatth@videolan.org>
|
| |
|
|
|
|
|
|
|
|
|
| |
Sets the correct include path when the build directory isn't the
the top source directory. Also, uses the .la file for linking,
which makes it possible to use shared libraries (and properly
build when libogg is used from a library).
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
|
|
|
|
| |
Subversion, like cvs, had a mechanism for replacing inline template
text on checkout for representing things like 'last modified date'.
Git does not support this, so remove the template strings from
file header comments.
Also less aggressive language is os_types.h.
|
|
|
|
| |
Useful for fuzzing
|