Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-10 Thread Michael Niedermayer
On Wed, Sep 10, 2014 at 10:47:12PM +0200, Tomas Härdin wrote: > On Tue, 2014-09-09 at 12:37 +0200, Michael Niedermayer wrote: > > On Mon, Sep 08, 2014 at 12:17:14PM +, Gaullier Nicolas wrote: > > > I did not found an easy way to set up initialization values to properly > > > handle defaults bu

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-10 Thread Tomas Härdin
On Tue, 2014-09-09 at 12:37 +0200, Michael Niedermayer wrote: > On Mon, Sep 08, 2014 at 12:17:14PM +, Gaullier Nicolas wrote: > > I did not found an easy way to set up initialization values to properly > > handle defaults but I am not a highly skilled developer, and maybe someone > > will fin

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-09 Thread Michael Niedermayer
On Mon, Sep 08, 2014 at 12:17:14PM +, Gaullier Nicolas wrote: > I did not found an easy way to set up initialization values to properly > handle defaults but I am not a highly skilled developer, and maybe someone > will find how to implement this more elegantly. > They are also many other pro

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-08 Thread James Darnley
On 2014-09-08 14:17, Gaullier Nicolas wrote: > [FFmpeg-devel] MXF : default fied dominance is TFF That should be *field* dominance. If you're going to submit another patch please correct the commit message. signature.asc Description: OpenPGP digital s

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-08 Thread Gaullier Nicolas
e Carl Eugen Hoyos Envoyé : lundi 8 septembre 2014 12:15 À : ffmpeg-devel@ffmpeg.org Objet : Re: [FFmpeg-devel] MXF : default fied dominance is TFF Gaullier Nicolas arkena.com> writes: > case 0x3212: > descriptor->field_dominance = avio_r8(pb); > +descripto

Re: [FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-08 Thread Carl Eugen Hoyos
Gaullier Nicolas arkena.com> writes: > case 0x3212: > descriptor->field_dominance = avio_r8(pb); > +descriptor->field_dominance_present = 1; Is it possible to instead initialize descriptor->field_dominance to MXF_TFF? > case SeparateFields: > +

[FFmpeg-devel] MXF : default fied dominance is TFF

2014-09-08 Thread Gaullier Nicolas
The field dominance is an optional property in the MXF specifications. It shall be assumed to be 1 (top field first) if not present. Nicolas diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 7a4633f..7517285 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -152,6