[Ffmpeg-cvslog] r8212 - trunk/libavformat/movenc.c

[Baptiste Coudurier]

Hi

Alex Beregszaszi wrote:
> Hi,
> 
>> -    put_be16(pb, track->mode == MODE_MOV ? track->enc->channels : 2); /* Number of channels */
>> -    /* FIXME 8 bit for 'raw ' in mov */
>> -    put_be16(pb, 16); /* Reserved */
>> +    if (track->mode == MODE_MOV) {
>> +        put_be16(pb, track->enc->channels);
>> +        put_be16(pb, 16); /* FIXME 8 bit for 'raw ' in mov */
>> +        put_be16(pb, track->audio_vbr ? -2 : 0); /* compression ID */
>> +    } else { /* reserved for mp4/3gp */
>> +        put_be16(pb, 2);
> 
> I think this should be changed to track->enc->channels and the next
> field to track->enc->bits_per_sample.

NO !!! these fields are reserved for mp4/3gp.

[...]

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312