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

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

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

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

I accidentally broke it in r18218.

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

Patch by Ron Lee.

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

Newer versions of png.h don't include it for us.
This is only needed when compiling with OC_DUMP_IMAGES (i.e., for
 debugging).

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

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

There was never any reason for these to not be const, and it
 shouldn't break backwards API or ABI compatibility.

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

This was broken when the dequant_tables indexing changed in commit
 r16102, but it only affected post-processing quality, so we never
 noticed.
With gcc 4.8.0, this can now trigger a segfault during decoder
 initialization.

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

Save activity from first pass. Gives a small SSIM improvement (e.g. 0.04dB for 360p elephants dream at 750kbit/s)

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

Thanks to Holden for the report.

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

ref_frame_data was being allocated with the aligned allocator, but
 freed with the normal _ogg_free() function on failure, which
 doesn't work.
This would only cause a problem if there was just enough memory to
 satisfy the reference frame allocation (just over 4.5 or 9 bytes
 per pixel) but not enough for the fragment buffer offets (1/16 or
 1/8th byte per pixel).

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

This allows it to be used from dump_video without having to disable
 striped decoding there.

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

Thanks to darkstarbyte for pointing out the typo.

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

1) Allow specification of the section alignment and make the
    default alignment 4 bytes (like the RVCT manual says it should
    be).
   This ensures that linking our ARM asm into a Thumb executable
    won't get the section placed on a 2-byte boundary.
2) Add a linker hint to indicate we don't need an executable stack.
   Otherwise it assumes we do.
3) Stop building and linking to an (empty) armopts object file.
   This is needed to make 2) work, since we include this file, so
    we can't add the linker hint at the end of it, and linking in
    just one object without that hint makes the whole library ask
    for an executable stack.

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

This is a more proper attribution for r17993.

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

Because we just removed target by removing AC_CANONICAL_TARGET,
and host is correct anyway.


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

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

It wrongly claimed the returned buffer was owned by libtheora, when
 in reality libtheora was creating a temporary buffer and not
 storing the pointer to the returned packet anywhere.
Since there's no encoder parameter passed to this function, it
 doesn't even have a place to store it.
This situation is sub-optimal, but can be solved by migrating to
 the 1.0 API, which does require an encoder instance to encode a
 comment, and thus does not have this problem.

Fixes #726.

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

It's the only example that needs it.
Patch by 0brad0.
Fixes #1587.

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

Fixes #1751.

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

David Schleef tells me this is only for compilers.
Fixes one part of #1800.

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

Patch from David Schleef.

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

Patch by mnemo.
Fixes #1804.

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

This extends the fix from r17276 dealing with the removal of the DC
 skip-prevention flag in r17174.
If we allow SKIPing even when the cost is right near the maximum
 representable SSD, then the cost of other blocks can push it over
 the edge, causing us to randomly skip a block halfway through the
 macroblock.

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

THEORA_DISABLE_FLOAT C preprocessor symbol.

This originally disabled slow floating point code in the library,
but the critical path code has all been removed. It also used
to disable function of th_granule_time() which returns a double,
but it has not done so since prior to the 1.0 release, and there
have been no complaints.

It might be helpful to restore this option if it eliminated    
the th_granule_time() API entry entirely, so that platforms
without native floating point don't have to link to an emulation
library, but in the absence of a request we prefer shorter code.

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

Make a note that passing 0 for the bitrate may not be guaranteed to
 return an error in future versions.

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