| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
| |
Bump the API documentation page footer for the latest release.
I missed this before making the 1.3.3 release, changing it
now and including this version on the website is less confusing,
I think.
|
|
|
|
| |
These should match http://downloads.xiph.org/releases/ogg/SHA256SUMS
|
| |
|
|
|
|
|
|
| |
ogg_stream_flush_fill to compile and use libogg under Visual Studio
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
|
|
|
|
|
|
|
| |
On 64-bit Windows, sizeof(unsigned char *) is 8, sizeof(long) is 4 and
alignof(ogg_int64_t) is 8. This results in a 4-byte hole in ogg_packet
after "long e_o_s", which means ogg_packet structs cannot be reliably
compared by memcmp. Compare ogg_packet structs member-by-member instead.
This fixes `make check` for me on mingw-w64/GCC.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
|
|
|
| |
Move off the end-of-life precise base image.
|
|
|
|
| |
Copied from the vorbis implementation by Marcel Metz.
|
|
|
|
|
|
|
|
|
| |
The previous commit changed the generated source file location
without updating the installed-file list or include directories,
breaking installation and downstream integration tests.
Adding the include directory doesn't seem to be necessary on macOS,
but even the initial build step fails on Linux without it.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A false continued packet occurs if the previous page ended at the
end of a packet, with no sequence number gap and no continued data
on that page, while the current page has the continued packet flag
set.
Previously, if you drained all of the buffered packet data by
repeatedly calling ogg_stream_packetout() after submitting the
previous page but before submitting the current page, libogg would
discard the continued data at the beggining of the current page.
However, if you left the previous page's data in the buffer, then
libogg would happily return the continued data at the start of the
current page as a separate packet when you eventually did call
ogg_stream_packetout().
This patch makes libogg consistently discard this data.
|
|
|
|
|
|
|
| |
Generate config_types in the build directory instead of
the source directory.
Signed-off-by: Ralph Giles <giles@thaumas.net>
|
| |
|