fixes addressing Spencer Dawkins <spencer@mcsr-labs.org> comments

svn path=/trunk/vorbis/; revision=14055

Many fixes suggested by Alfred <ah@tr-sys.de>

svn path=/trunk/vorbis/; revision=14054

test/write_read.c : Move delaration before first statement.

svn path=/trunk/vorbis/; revision=13685

Fix compile warnings in recently re-enabled test code.

svn path=/trunk/vorbis/; revision=13684

Hook up the libvorbis self test code so it is run by automake's 'check'
target. This only enables the sharedbook.c self test, as the codebook.c
self test has bit rotted.

Unfortunately the doc stuff scrolls the results off the screen, but
given the trouble we've had with the doc build in the past, I'm
loath to place it before the code.

svn path=/trunk/vorbis/; revision=13666

Fix to bug #1229

svn path=/trunk/vorbis/; revision=13657

Removing erikd's 'test harness' from Makefile.am as it breaks the build.
Not reverting r13652 entirely-- yet.

svn path=/trunk/vorbis/; revision=13656

Add the bare bones of a test suite.

svn path=/trunk/vorbis/; revision=13652

One more patch to bug #300

svn path=/trunk/vorbis/; revision=13651

Bug #300 was not completely killed; a setting for uncoupled modes in
setup_22.h got missed.

svn path=/trunk/vorbis/; revision=13644

Erikd: take care that added comments with detailed explanations are
actually correct.  The default callbacks in vorbisfile.h are not an
attempt to preserve ABI, they're due to peculiarities of Windows
linking strategies.

svn path=/trunk/vorbis/; revision=13643

lib/lookup_data.h : Add ifdef around FROMdB_LOOKUP to prevent compiler warnings.

svn path=/trunk/vorbis/; revision=13631

include/vorbis/vorbisfile.h : Add comments about a desired change that cannot be made until we're willing to break ABI compatibility.

svn path=/trunk/vorbis/; revision=13630

Detect availability of C99 inline keyword in configure and use it in lib/scales.h. Also add workaround for MSVS which uses __inline.

svn path=/trunk/vorbis/; revision=13629

doc/Makefile.am : Fix broken build.

svn path=/trunk/vorbis/; revision=13590

Leftovers...

svn path=/trunk/vorbis/; revision=13584

Remove warnings about unused or uninitialized variables.

svn path=/trunk/vorbis/; revision=13578

Remove unused parameter from function _ve_amp.

svn path=/trunk/vorbis/; revision=13577

Make _v_writestring const correct.

svn path=/trunk/vorbis/; revision=13576

Move struct and func definition to codec_internal.h.

svn path=/trunk/vorbis/; revision=13575

draft 07

svn path=/trunk/vorbis/; revision=13574

Make vorbis comment functions const correct.

svn path=/trunk/vorbis/; revision=13571

Simplify configuration-uri handling

svn path=/trunk/vorbis/; revision=13558

add the start of a real configuration string, as suggested by Colin

svn path=/trunk/vorbis/; revision=13557

Move back gzip, bzip2 and vorbis reference to the normative section

svn path=/trunk/vorbis/; revision=13556

add reference to rfc4588 as suggested by Colin

svn path=/trunk/vorbis/; revision=13555

Reverting previous change.

Ivo: do not change copyright policy without asking people.

svn path=/trunk/vorbis/; revision=13526

Collective Copyright statement.

svn path=/trunk/vorbis/; revision=13525

Make VS2003 do what VS2005 already does

svn path=/trunk/vorbis/; revision=13380

Visual Studio 2005 projects file updates: Missing link dependencies "libogg.lib libvorbis.lib" added, no more debugging info for release builds, solution file added

svn path=/trunk/vorbis/; revision=13377

Bump version number post-release.

svn path=/trunk/vorbis/; revision=13363

Update library versioning and changelog for the 1.2.0 release.

svn path=/trunk/vorbis/; revision=13312

Catch one bulletproofing oversight; only an app bug would trip it

svn path=/trunk/vorbis/; revision=13294

Update documentation, version numbers, copyright dates

svn path=/trunk/vorbis/; revision=13293

Add ov_fopen() (like ov_open, but does the fopen within libvorbisfile)
and extend ov_open_callbacks() mechanism as means of encouraging use
over ov_open().

Use of ov_open() now strongly discouraged under Windows;
ov_open_callbacks(fd,vf,NULL,0,OV_CALLBACKS_NOCLOSE) as a direct
replacement or ov_fopen() as a convenient substitute.

svn path=/trunk/vorbis/; revision=13289

We have a new encoder version string.

svn path=/trunk/vorbis/; revision=13288

New built copies of the decoder spec.

svn path=/trunk/vorbis/; revision=13287

Add vorbis_synthesis_idheader to the .def file for win32.

svn path=/trunk/vorbis/; revision=13284

Also update the configured version.

svn path=/trunk/vorbis/; revision=13225

Update the changelog. Also bump the version number to a new minor. There
are still no encoder improvements over 1.1, but the handling of
multiplexed streams is a new feature and warrents the jump.

svn path=/trunk/vorbis/; revision=13224

Update the doc link to the rtp draft as well.

svn path=/trunk/vorbis/; revision=13221

Well.  *that* deadlock bug has been lurking in seeking for a long time.
A bigger chunksize + just the right annodex file finally reproduced it
here.

svn path=/trunk/vorbis/; revision=13217

New vorbisfile multiplexing support passes built-in playback and seeking
tests.

svn path=/trunk/vorbis/; revision=13215

Multiplex support fixes; basic functions test OK on single stream files
now, time to test chains and multiplexing.

svn path=/trunk/vorbis/; revision=13214

Multiplexed stream support for vorbisfile *IN PROGRESS*
Don't use it, I know for certain it's broken.

Just a commit to make switching machines easier :-)

svn path=/trunk/vorbis/; revision=13213

Add return checks to all instances of seek_func and _seek_helper.
Should close bug #853.

svn path=/trunk/vorbis/; revision=13211

Although bug 552 was complaining about behavior that was illegal
anyway, added an additional proofing check to vorbisfile to aid a
developer figuring out what he's doing wrong.

svn path=/trunk/vorbis/; revision=13210

Fix for bug 577; used a different approach than the submitted patch
(rather than fudging an out of bound result after computation, do it
the right way to begin with)

Add test for fix to seeking_example.

svn path=/trunk/vorbis/; revision=13198

Draft filename changed, build system must be updated to match.

svn path=/trunk/vorbis/; revision=13197

06 commit to ietf

svn path=/trunk/vorbis/; revision=13190

...if we're going to verify against minimum legal blocksize, we really
ought to verify against maximum too.

svn path=/trunk/vorbis/; revision=13179

More fixes to handle the null-entry codebook case.  It appears the
failure to handle this case affects more of the code than thought;
many of the old low-bitrate crash cases trace back to this problem.

This also affects tremor.  These changes are not yet thoroughly
tested.

svn path=/trunk/vorbis/; revision=13172

Apply patch as supplied in ticket 851

svn path=/trunk/vorbis/; revision=13171

The automated book training scripts managed to create a few residue
books with no codewords (in that mode with that partitioning scheme,
no values happened to fall into that partition).  This is not really
an error.

However, the residue encoding code did not account for the possibility
of looking up a codebook best match and finding no entries.  Because
the unused book only came up very rarely (four hours of training audio
didn't hit it once), it took a while to notice and only comes up on
platforms with subtly different FP than the machine used to do
training.

This bug affects very low bitrate stereo 8, 11, 16 and 22kHz modes;
added code to check for 'no match in codebook' (correct behavior is to
encode nothing) and for good measure, removed the four dummy codebooks
as there's no point in wasting a few bytes in the header to pack a
nothing.

Closes bug 1142.

svn path=/trunk/vorbis/; revision=13170

Add additional check when attempting to encode values through
codebooks with no entries.  Ideally that wouldn't happen, but it looks
like the automated codebook training ended up with a few 'null' books.

svn path=/trunk/vorbis/; revision=13169

Attempted fix for bug #300; the last comment identified a legitimate static bounds overrun that would have explained the behavior.

svn path=/trunk/vorbis/; revision=13168

Don't forget to test all changes in a code cleanup-- residue type 2 encoding was broken by last night's round of fixes.

Tested fixed.

svn path=/trunk/vorbis/; revision=13167

Draft update

svn path=/trunk/vorbis/; revision=13163

Numerous small changes to resolve possible and certain issues found by
Coverity.

svn path=/trunk/vorbis/; revision=13162

Correct a cleanup dispatch error when a bitstream is rejected due to
illegal map/floor/residue type.  The cleanup subdispatch was done by
type, unfortunately, the invalid type just read in.  Fixed.

also bump lib revision.

svn path=/trunk/vorbis/; revision=13160

Spec addition to clarify corner case; residue and floor decoded sections
may overflow the base vector's actual storage; this *must* be handled by
decoders.

svn path=/trunk/vorbis/; revision=13159

Catch one more omission in the vector length guarding

svn path=/trunk/vorbis/; revision=13155

I had forgotten that the res2 top level interprets limits differently
than res0 and res1; guarding code (and garbled playback) fixed

svn path=/trunk/vorbis/; revision=13154

If you're going to guard against max blocksize, make sure you choose the right blocksize.

svn path=/trunk/vorbis/; revision=13153

Additional bounds checking to guard against the case where a residue
or floor is decoding into a vector larger than the current blocksize.
This is not illegal, but no encoder has ever tried it and as such, it
had not been noticed that the code did not handle the case correctly.

svn path=/trunk/vorbis/; revision=13151

Update change log with the news so far.

svn path=/trunk/vorbis/; revision=13150

Correct a bug where the guard code is allowing illegally small blocksizes.

svn path=/trunk/vorbis/; revision=13149

Fix the SDP example

svn path=/trunk/vorbis/; revision=13132

Update autogen.sh to handle automake 1.10.

svn path=/trunk/vorbis/; revision=13069

Update the makefile to reference the updated rtp drafts.

svn path=/trunk/vorbis/; revision=13068

Draft updated

svn path=/trunk/vorbis/; revision=12996

Latest batch of fixes

svn path=/trunk/vorbis/; revision=12995

Make nut v understandable

svn path=/trunk/vorbis/; revision=12946

Fixed the Packed Headers Detail Figure

svn path=/trunk/vorbis/; revision=12940

First rephrase of the new header packing procedure, more pending

svn path=/trunk/vorbis/; revision=12939

Update the makefile for the new RTP draft revision number.

svn path=/trunk/vorbis/; revision=12934

Yet another change in the headers packing, in practice nothing changes if you use a dummy comment header ( 2| 30| 28| ident| comment| setup| ) but should be flexible enough to be used for other codecs or evolutions of vorbis

svn path=/trunk/vorbis/; revision=12929

Headers packing changes as requested by rillian

svn path=/trunk/vorbis/; revision=12923

Typo fixes from Silvia Pfeiffer

svn path=/trunk/vorbis/; revision=12922

Update the text document

svn path=/trunk/vorbis/; revision=12908

To be committed soon

svn path=/trunk/vorbis/; revision=12907

Colin Perkins' latest comments

svn path=/trunk/vorbis/; revision=12906

The structure is not freed, just its contents.

svn path=/trunk/vorbis/; revision=12897

Updated document, will be sumbitted to ietf soon

svn path=/trunk/vorbis/; revision=12889

Offer/Answer clarification

svn path=/trunk/vorbis/; revision=12888

Iana update completed

svn path=/trunk/vorbis/; revision=12887

iana update?

svn path=/trunk/vorbis/; revision=12886

idnits fixes

svn path=/trunk/vorbis/; revision=12883

uncommitted date/name change...

svn path=/trunk/vorbis/; revision=12832

New draft

svn path=/trunk/vorbis/; revision=12831

move to base64 instead of base16

svn path=/trunk/vorbis/; revision=12813

not sure how I missed this wrong includedir

svn path=/trunk/vorbis/; revision=12652

Add missing vqsplit.h and localcodebook.h headers to the distribution.

svn path=/trunk/vorbis/; revision=12581

Updates, pending review

svn path=/trunk/vorbis/; revision=12364

Updated/added Xcode build files.

svn path=/trunk/vorbis/; revision=12345

Quoth Monty on IRC.

svn path=/trunk/vorbis/; revision=12295

cleanup part 2

svn path=/trunk/vorbis/; revision=12135

cleanup part 1

svn path=/trunk/vorbis/; revision=12134

wtay's comment: inline is not in-band and vice versa

svn path=/trunk/vorbis/; revision=12133

Colin Perkins' suggestion: note rate: and channels: in the iana section

svn path=/trunk/vorbis/; revision=12056