[FFmpeg-devel] [PATCH] read colr atom into metadata

[Reimar Döffinger]

On Sat, Jul 24, 2010 at 02:22:52PM -0700, Baptiste Coudurier wrote:
> On 7/23/10 11:18 AM, Reimar D?ffinger wrote:
> >Hello,
> >As subject says.
> >No particularly good reasoning yet, except that I _suspect_ this
> >will be necessary for ProRes.
> >At the very least it will make building the ImageDescription for
> >the binary decoder in MPlayer less messy...
> 
> I'm not against the patch, though it may be better to either export
> it through AVCodecContext->color_primaries, etc ..., or to export
> complete stsd atom in some extradata.
> 
> The patch may also cause side effects with h264 containing colr
> atoms and these are common. Needs testing with -vcodec copy
> <file.mov>, and playback.

We might need some better solution somewhen, but the ProRes decoder
actually seems to work fine without this atom, so I'd rather drop
this for now.