[FFmpeg-devel] [PATCH] read colr atom into metadata
Sat Jul 24 23:52:50 CEST 2010
On Sat, Jul 24, 2010 at 02:22:52PM -0700, Baptiste Coudurier wrote:
> On 7/23/10 11:18 AM, Reimar D?ffinger wrote:
> >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.
More information about the ffmpeg-devel