[Ffmpeg-devel] raw video supportin quicktimewrongly assumesrawaudio
Tomas Stenlund
tomas.stenlund
Sun Nov 5 08:24:26 CET 2006
----- Original Message -----
From: "Baptiste Coudurier" <baptiste.coudurier at smartjog.com>
To: "FFmpeg development discussions and patches" <ffmpeg-devel at mplayerhq.hu>
Sent: Saturday, November 04, 2006 12:53 PM
Subject: Re: [Ffmpeg-devel] raw video supportin quicktimewrongly
assumesrawaudio
> 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.
Okey, thanks. Will do ...
>
> --
> Baptiste COUDURIER GnuPG Key Id: 0x5C1ABAAA
> SMARTJOG S.A. http://www.smartjog.com
> Key fingerprint 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
> Phone: +33 1 49966312
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>
More information about the ffmpeg-devel
mailing list