> On Jul 6, 2017, at 9:39 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> > wrote: > > On 7/7/2017 2:20 AM, Dave Rice wrote: >> Currently this patch writes the clap atom under this condition >> "track->par->codec_id == AV_CODEC_ID_RAWVIDEO & track->mode == MODE_MOV”; >> however, AV_CODEC_ID_RAWVIDEO isn’t quite what the spec asks for. Any advice >> on how to write this condition for “uncompressed Y´CbCr data formats”; for >> instance v210 is considered uncompress Y´CbCr in QuickTime but is not a >> AV_CODEC_ID_RAWVIDEO. I haven’t found a concise way to add a condition for >> uncompressed YUV. > > First, it should be '&&', I assume.
Thanks. > I don't think there's any concise way of checking such conditions... you'll > probably have to do something like a list of "codes" (v410, v210, raw) and > some sort of sanity check for track->par->format to confirm is is YCbCr. > Maybe someone knows better than myself. I see that although it’s mandatory for uncompressed YUV, there’s no constraint to not use it otherwise, so I think I’ll proceed with just a check for MODE_MOV. I just checked ffmbc’s movenc.c and it also only checks for MODE_MOV. Dave Rice _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel