[FFmpeg-devel] [PATCH] Add 'Presentation Y offset' metadata

[Michael Niedermayer]

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
> >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
-------------- 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/20150330/329cb004/attachment.asc>