[FFmpeg-devel] [PATCH] avformat/mxfenc: Add color siting element
nichot20 at yahoo.com
Thu May 21 16:06:11 CEST 2015
-----BEGIN PGP SIGNED MESSAGE-----
On 21/05/15 11:59, Michael Niedermayer wrote:
> On Thu, May 21, 2015 at 09:00:56AM +0200, Tomas Härdin wrote:
>> Why is this "guessing" code in mxfenc? This should be something that'
>> taken care of before calling any muxer (like in
>> avformat_write_header()), so everyone can benefit from it and so ther
>> less risk of duplication when it's needed elsewhere.
>> I've seen stunts like this pulled in more places than mxfenc, where
>> muxers will do dubious things like parse certain kinds of essence. I
>> not a fan.
> nor am i, ill try to move it into common code
+1 to both your comments its why I have hesitated on some of the other
mxf metadata thats been requested.
Key Fingerprint 38CF DB09 3ED0 F607 8B67 6CED 0C0B FC44 8B0B FC83
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.22 (GNU/Linux)
-----END PGP SIGNATURE-----
More information about the ffmpeg-devel