[MPlayer-dev-eng] Re: [PATCH] demux_ogg.c broken bitrate detection on some files fixed

Nico Sabbi nicola_sabbi at fastwebnet.it
Tue Sep 5 22:24:13 CEST 2006


Andrew Savchenko wrote:

>Hi!
>
>  
>
>>This one should work fine.
>>The problem was in nombr, minbr and maxbr variables data type: they
>>are unsigned int, but ogg stream allows them to be set in "-1" which
>>means unspecifed.
>>
>>So I've fix their type and add a simply check wich set them to 0 if
>>they are -1, this allows to set properly bitrate in
>>sh->wf->nAvgBytesPerSec.
>>    
>>
>
>Is this pacth ignored?
>Are there any opinions to apply it?
>
>I understand, it takes some time to review new patch, moreover it is 
>minor bugfix. But almost three days has left... Please, reply.
>  
>
>------------------------------------------------------------------------
>

harmless => applied, thanks



More information about the MPlayer-dev-eng mailing list