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

[Baptiste Coudurier]

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.

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org