[FFmpeg-devel] [PATCH] avformat: Add simple ACLR atom reading to set the color range

Michael Niedermayer michaelni at gmx.at
Tue Feb 17 16:33:40 CET 2015

On Tue, Feb 17, 2015 at 01:52:19PM +0100, wm4 wrote:
> On Tue, 17 Feb 2015 12:40:52 +0000
> Kevin Wheatley <kevin.j.wheatley at gmail.com> wrote:
> > On Tue, Feb 17, 2015 at 12:25 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > > if the codec id doesnt match the expected, mov_read_extradata will
> > > return 0 even without any truncation.
> > > In this case the error message would be incorrect
> > 
> > So should I code the test against codec id against the files I have or
> > the ones that might be valid, but I don't have samples for?
> I think this really should be done in the decoder instead.

i agree if it was used in only one codec but its used in at least
AVUI and DNXHD judging from the code and i have found a MJPEG one
locally that uses this atom too

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150217/92b86eb1/attachment.asc>

More information about the ffmpeg-devel mailing list