[Ffmpeg-devel] raw video support in quicktime wrongly assumes raw audio

[Baptiste Coudurier]

Hi

Tomas Stenlund wrote:
> [...]
> 
> So it assumes, wrongly, that this is an audio codec.
> 

Right.

> When I looked at the sourcecode for mov.c I found out that the 'raw ' support
> for video was commented out and that the mov_read_stsd always tried the
> audio fourcc before the video regardless of what the hdlr header said. So
> the 'raw ' exists for both video and audio codec.

Yes. Point is that 'hdlr' atom can be located after 'stsd' atom,
therefore codec type is not known while parsing 'stsd'.

Solution is to delay 'stsd' parsing until we are sure about the codec
type. Im actually working on it.

> I'm completely new to the ffmpeg architecture and how you want to solve
> things but it was pretty simple to add this support so it worked for me. I added
> support for 'raw ' in mov.c and used the hdlr information if available in
> the stsd reader and mapped 'raw ' to RGB24 format in the raw.c.

Not sure. Check avctx->bits_per_sample value got from 'stsd' I think.

[...]

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