[FFmpeg-devel] [PATCH] oggpagesize option added

Ronald S. Bultje rsbultje
Thu Jan 27 15:30:05 CET 2011


Hi,

2011/1/27 Andres Gonzalez <acandido at hi-iberia.es>:
> On 26/01/11 21:54, Baptiste Coudurier wrote:
>> You still have to buffer the page at segments_count == 255.
>
> Ok! Here it is.
[..]
> @@ -208,8 +227,9 @@ static int ogg_buffer_data(AVFormatContext *s, AVStream *st,
>          if (i == total_segments)
>              page->granule = granule;
>
> -        if (page->segments_count == 255) {
> -            ogg_buffer_page(s, oggstream);
> +        if(page->segments_count == 255 ||
> +           (ogg->pref_size > 0 && page->size >= ogg->pref_size)) {
> +           ogg_buffer_page(s, oggstream);

You broke the indent of that last line, should be 4-spaces not
3-spaces. Still OK apart from that, whoever commits can fix that
himself...

Ronald



More information about the ffmpeg-devel mailing list