[Ffmpeg-devel] raw video support in quicktimewrongly assumesrawaudio

[Baptiste Coudurier]

Tomas Stenlund wrote:
> 
>> [...]
>>> Baptiste Coudurier wrote:
>>>>>>
>>>>>> [...]
>>>>>>
>>>>>> Solution is to delay 'stsd' parsing until we are sure about the codec
>>>>>> type. Im actually working on it.
>>>>>>
>>>>> Reading in the entire moov structure or just that part ?
>>>>
>>>> Just 'stsd' part. Anyway it seems that not overwriting codec_type might
>>>> be better than nothing. It will not be perfect though.
>>>>
>>>
>>> Did that for now until we find a sample which needs the perfect solution
>>> which would be a pain to implement.
>>>
>>> Check out latest svn.
>>>
>>
>> Quick one. I will do ... much better than running my "dirty" version ...
>>
> Works perfectly!
> However to be sure that I actually do a good solution, is it accepted to
> rewrite the fourcc and add the depth as the last byte ? I saw that they
> have done that in raw.c for RGB555 and 565 ( I think mtv.c uses it).
> So i would in that case rewrite the fourcc in mov.c and add the
> MKTAG('r', 'a', 'w', 24) and map it to PIX_FMT_RGB24 in
> pixelFormatTags ?
> 
> A bit ugly ?
> 

Yes. raw.c must be modified to handle bits_per_sample when 'raw ' codec
tag is set.

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