Both POSIX, the Linux documentation and UNIX Network Programming volume 1
by Stephens state that NULL is accepted.

This avoid the following compiler message:

../../examples/player_example.c: In function ‘main’:
../../examples/player_example.c:839:22: warning: passing argument 2 to ‘restrict’-qualified parameter aliases with argument 4 [-Werror=restrict]
  839 |           n=select(n,&empty,&writefs,&empty,&timeout);
      |                      ^~~~~~          ~~~~~~
../../examples/player_example.c:843:18: warning: passing argument 2 to ‘restrict’-qualified parameter aliases with argument 4 [-Werror=restrict]
  843 |         select(n,&empty,&writefs,&empty,NULL);
      |                  ^~~~~~          ~~~~~~

Apparently this still has users, so test it, rather than
removing it. Verify we can build and clean up without error.

Add a build description for gitlab's continuous integration build
and test runners. This gives us ongoing converage on commmits
and merge requests in the upstream repository.

Based on the gcc:9 image we currently use for most projects.
It already includes make and autotools, but we need to install
dependencies for the example programs and documentation.

  - debian buster has fig2dev in its own package.
  - latex-extras is needed for wrapfig, which has been superceeded
    by lwarp-wrapfig.

NB defining a separate job at the test stage clears all the
generated artefacts, which means we lose our configure script
and makefiles. Therefore isn't more efficient to just do the
tests as extra steps in the build stage.

GStreamer's "theoradec" element always calls th_decode_ctl() for all telemetry
options, passing 0 if they are not enabled. This results in our creating a
Cairo buffer and performing YUV/RGB conversion, even if all telemetry is
disabled, which is very slow and can cause noticeable lag during playback.

Align indents to two spaces for cosmetic uniformity.

Jan Niklas Hasse authored 4 years ago and Ralph Giles committed 4 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Jan Niklas Hasse authored 5 years ago and Ralph Giles committed 5 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Sebastian Buchwald authored 6 years ago and Ralph Giles committed 5 years ago

From GCC documentation (for GCC 5 and above):
- ‘+’ means that this operand is both read and written by the instruction.
- Only read-only operands can use ‘%’.

Thus, ‘+’ and ‘%’ cannot be used at the same time.

Signed-off-by: Timothy B. Terriberry <tterribe@xiph.org>
Signed-off-by: Ralph Giles <giles@thaumas.net>

Conform to local style, use shorter variable names, and add
some comments so the intent of the flow is more clear.

Also propagate the check to the transcoder example which
duplicates the same code.

Guido Günther authored 7 years ago and Ralph Giles committed 5 years ago

Abort on error from `vorbis_analysis_headerout`.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Add these to .gitignore so they don't clutter up version control
status messages and grep output.

Update links and place the description further away in time;
this codec isn't widely used any more.

This is more readable on the gitlab/github landing pages.

This will bail on files exceeding 16384x16384 (useful for fuzzing).

These were left as a convenience for those on old systems.
Automake 1.11 is pretty widely available now.

This is more appropriate now that we've moved the primary repository
into git from subversion.

Replace our more complex autogen.sh script with a copy of the
one from the Opus repository, which just calls autoreconf. That
command does the same things more efficiently and is reliably
available now.

This prefixes generated object file names with their relative
path. The idea is to avoid conflicts. We don't have any, but
automake warns about the issue, so set the option to address that.

Cameron Cawley authored 7 years ago and Ralph Giles committed 7 years ago

Signed-off-by: Ralph Giles <giles@thaumas.net>

Seems to only effect OS X according to Travis CI

The package enumeration tests in configure weren't relative to the
 ${srcdir}, and the Makefile.am rules didn't handle a destination
 directory different from the source directory.

Global pointers are initialized to zero in C, but not in C++.

Add an explicit initializer to reduce confusion and avoid
bugs when the example code is copied into C++ applications.

Based on a patch by Alex Houghton <houghton1411@aol.com>.

We made sure to emit emms before potentially calling/returning to
 applicate code, but not before calling libc code.
It turns out musl libc's malloc/free implementation use floating
 point, so they don't work correctly unless we do.
Add some extra calls to make sure the FPU is in the proper state
 any place we might potentially call malloc or free.

Thanks to Rune (ztsdztsd on Trac) for the report.

Fixes #2287

If we have some dupe frames right near the end of the buffer, our
 metric window can be slightly larger than the buffer.
However, the clamping we use to determine how many frames' worth of
 metrics to add to the buffer had a mix of signed and unsigned
 types, so if that number when negative, instead of clamping
 against zero, it would ask for all remaining frames in the file.
That would cause an infinite loop when we tried to find the last
 keyframe in the circular buffer in oc_enc_select_qi() (which was
 not big enough to actually hold that many frames).

This patch changes the clamp to a form that works with all unsigned
 values.

Thanks to Brion Vibber for the report.

Fixes #2229

Forward-port of r19507 from svn.

In the case where we got a data packet before receiving all three
 headers, the old code would check to see if the second through
 seventh bytes matched the magic string "theora" (extremely
 unlikely, but possible), and if so, return TH_EBADHEADER, otherwise
 return TH_ENOTFORMAT.
That this was not consistent was a bit non-sensical.

5a5f5bb2 changed to returning TH_EBADHEADER if we got a data
 packet after receiving the first header, but left the old behavior
 for the first packet.
Change instead to explicitly return TH_ENOTFORMAT if the first
 header was missing (since we only check one bit of the packet to
 determine whether or not it's a data packet, odds are it's a packet
 for some other kind of data, like a Vorbis header).
We continue to return TH_EBADHEADER if we see a data packet after
 encountering a valid header, but before reading all three.

Also re-arrange the NULL checks to follow continue to allow the
 undocumented ability to pass in NULL for parameters which are not
 needed by the next header in the sequence.
E.g., it's perfectly all right to pass NULL for _setup when
 expecting to read the comment header next.
In this case we'll now return TH_EBADHEADER instead of TH_EFAULT if
 the packet was actually a data packet.

Our documentation says we will, so we should.

Thanks to Chris Peterson for the report.

Fixes #2279.

Signed-off-by: Ralph Giles <giles@thaumas.net>

Newer texlive distributions require \usepackage{float} for
the [H] figure layout directive. We use this to make sure
the header layout figures are at the top of their respective
sections for conventional reference like in an RFC, so requiring
the additional package is safer than falling back to standard
latex [h] or [!h].

Note that [H] from the float package is incompatible with
fallback specifiers.

The configure script will automatically check for the new package.

oc_mb_fill_cmapping11() was defined without the last two parameters
 of its fellow mapping functions, meaning it got called with extra
 undeclared parameters.

This could confuse the emscripten JavaScript cross-compiler's asm.js
 optimizations, as it's very picky about function signatures.

Fixes #2068.
Patch by Brion Vibber <brion@pobox.com>.

svn path=/trunk/theora/; revision=19261

Patch by valtri.
https://trac.xiph.org/ticket/2032

svn path=/trunk/theora/; revision=19181

Patch from valtri. https://trac.xiph.org/ticket/2032

svn path=/trunk/theora/; revision=19180

svn path=/trunk/theora/; revision=19178

Correct Cb plane bug in conversion from RGB->YCbCr; the upper-right pixel was ignored by mistake and the upper-left pixel used twice.


svn path=/trunk/theora/; revision=19154

This flag breaks clang, which does not support it, and is obsolete
 on gcc anyway.

svn path=/trunk/theora/; revision=19116

svn path=/trunk/theora/; revision=19094

svn path=/trunk/theora/; revision=19088