[FFmpeg-devel] Correct libvorbis.c behaviour with >2 channels

[Cyril Russo]

Le 27/05/2010 00:11, James Darnley a ?crit :
> On 26 May 2010 23:30, James Darnley<james.darnley at gmail.com>  wrote:
>    
>> Attached is a small patch (svn diff) to correct broken audio when
>> encoding 5.1 audio using libvorbis.  The code only did 1 and 2
>> channels correctly with all others giving the broken audio.  With the
>> patch, any number of channels should be handled correctly.  The
>> channels should be correctly arranged too.
>>
>>      
> Whoops...  Forgot a -1 in the lut index and a semicolon.
>    
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
I think you missed vorbis-data in the patch.
Also, the coding convention is
for(...) {
and not
for(...)
{

Best regards,
Cyril Russo