GitLab

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