On Mon, Mar 30, 2015 at 03:01:54PM +0200, tomas.har...@codemill.se wrote: > On 2015-03-30 14:21, Michael Niedermayer wrote: > >On Mon, Mar 30, 2015 at 12:31:04PM +0200, tomas.har...@codemill.se > >wrote: > >>On 2015-03-30 12:19, tim nicholson wrote: > >>>Some software mis decodes IMX material in mxf if this metadata field is > >>>missing. See the discussion on the ffmpeg ML:- > >>> > >>>"[FFmpeg-user] How to set 3 specific metadata flags > >>>(ITU601/displayoffset) in FFmpegs IMX50 MXF-OP1a encoding?" > >>> > >>>This patch add this field to mxf wrapped material. > >> > >>Looks fine, but I know from experience that all this stuff gets > >>tricky quickly. Like there should be mechanisms in lavf for dealing > >>with presentation rectangles and all that jazz, including different > >>aspect ratios at different layers. > > > >should i apply this patch or wait ? > > Apply it - dealing with this properly (like for properly remuxing > MOV <--> MXF) would require quite a bit of design that shouldn't > hold up this patch.
applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in which you can choose your dictator
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
