[FFmpeg-devel] [PATCH] Add 'Presentation Y offset' metadata
michaelni at gmx.at
Mon Mar 30 15:12:17 CEST 2015
On Mon, Mar 30, 2015 at 03:01:54PM +0200, tomas.hardin at codemill.se wrote:
> On 2015-03-30 14:21, Michael Niedermayer wrote:
> >On Mon, Mar 30, 2015 at 12:31:04PM +0200, tomas.hardin at codemill.se
> >>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.
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel