[FFmpeg-devel] [PATCH] Fix bad comment in aac code, PING

Cyril Russo stage.nexvision
Thu May 27 11:57:38 CEST 2010


PING.
I'd like to post a patch to somehow help fixing issue1882 and this would 
be a welcome addition.


>> Index: libavcodec/aac.c
>> ===================================================================
>> --- libavcodec/aac.c    (revision 23312)
>> +++ libavcodec/aac.c    (working copy)
>> @@ -127,8 +127,8 @@
>>           }
>>       case 6:
>>           /* Some streams incorrectly code 5.1 audio as SCE[0] CPE[0] CPE[1] SCE[1]
>> -           instead of SCE[0] CPE[0] CPE[0] LFE[0]. If we seem to have
>> -           encountered such a stream, transfer the LFE[0] element to SCE[1] */
>> +           instead of SCE[0] CPE[0] CPE[1] LFE[0]. If we seem to have
>> +           encountered such a stream, transfer the SCE[1] element to LFE[0] */
>>           if (ac->tags_mapped == tags_per_config[ac->m4ac.chan_config] - 1&&  (type == TYPE_LFE || type == TYPE_SCE)) {
>>               ac->tags_mapped++;
>>               return ac->tag_che_map[type][elem_id] = ac->che[TYPE_LFE][0];
>>
> I think the text is trying to say that we are putting
> ac->che[TYPE_LFE][0] into ac->tag_che_map[TYPE_SCE][1].
>
> This is because the "LFE" will be read and looked up as SCE.1 in the bitstream.
>
Ok, it wasn't very clear.
I've attached fixed patch.

Cheers,
Cyril RUSSO


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: fixBadDoc.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100527/85f688d6/attachment.asc>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: Portion de message jointe
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100527/85f688d6/attachment.txt>



More information about the ffmpeg-devel mailing list