ogg dev - Apr 2010 - Make check failure

Hi all,

I've just grabbed a copy of SVN head for libogg and 'make check'
is failing:

    ./test_framing
    testing single page encoding... 0, (0),  granule:0 ok.
    testing basic page encoding... 0, (0),  granule:0 1, (1),  granule:6151 ok.
    testing basic nil packets... 0, (0),  granule:0 1, (1),  granule:10247 ok.
    testing initial-packet lacing > 4k... 0, (0),  granule:0 1, (1), 
granule:2055 ok.
    testing single packet page span... 0, (0),  granule:0 1, (1), 2, (2), 
granule:3079 ok.
    testing page spill expansion... 0, (0),  granule:0 1, header content
mismatch at pos 5:
     (0)4f:4f (1)67:67 (2)67:67 (3)53:53 (4)00:00 (5)00:04 (6)07:07 (7)10:14
(8)00:00 (9)00:00
     (10)00:00 (11)00:00 (12)00:00 (13)00:00 (14)01:01 (15)02:02 (16)03:03
(17)04:04 (18)01:01
     (19)00:00 (20)00:00 (21)00:00 (22)ce:bc (23)8f:d8 (24)17:df (25)1a:12
(26)17:18 (27)ff:ff
     (28)ff:ff (29)ff:ff (30)ff:ff (31)ff:ff (32)ff:ff (33)ff:ff (34)ff:ff
(35)ff:ff (36)ff:ff
     (37)ff:ff (38)ff:ff (39)ff:ff (40)ff:ff (41)ff:ff (42)ff:ff (43)ff:ff
(44)0a:0a (45)ff:ff
     (46)04:04 (47)ff:ff (48)00:00 (49)00:00


I'm building this on an x86_64 Debian Testing system with gcc
version 4.4.3


Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/

Erik de Castro Lopo wrote:
> I've just grabbed a copy of SVN head for libogg and 'make
check'
> is failing:
The breakage happened in this commit:

    r17098 | gmaxwell | 2010-03-29 16:35:11 +1100 (Mon, 29 Mar 2010) | 1 line

    Add a 'ogg_stream_pageout_fill' function to allow smart applications
with
    delay sensitive flushing to produce big pages. Increase the default minimum
    fill amount to 8 based on latency measurements with actual files. These
    changes may be controversial but since we've recently had a release I
thought
    there would be no harm in getting them into the repository for discussion.

Firstly, is gmaxwell on this list?

Secondly should be be trying controvertial changes in HEAD? Surely thats
what separate branches are for, so that the trunk never breaks like this.

Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/