Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-09-24 Thread Tomas Härdin
On Tue, 2014-09-16 at 09:03 +0100, tim nicholson wrote: > On 10/09/14 21:45, Tomas Härdin wrote: > > On Tue, 2014-08-19 at 22:30 +0200, Michael Niedermayer wrote: > >> On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: > >>> Hi! > >>> > >>> Attached patch removes a request for sample

Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-09-16 Thread tim nicholson
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/09/14 21:45, Tomas Härdin wrote: > On Tue, 2014-08-19 at 22:30 +0200, Michael Niedermayer wrote: >> On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: >>> Hi! >>> >>> Attached patch removes a request for samples of which we already

Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-09-10 Thread Tomas Härdin
On Tue, 2014-08-19 at 22:30 +0200, Michael Niedermayer wrote: > On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch removes a request for samples of which we already > > have several that all work fine. > > field_dominance can have 256 different values

Re: [FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-08-19 Thread Michael Niedermayer
On Tue, Aug 19, 2014 at 01:30:24AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch removes a request for samples of which we already > have several that all work fine. field_dominance can have 256 different values, do we have samples for all ? do they even exist ? is the st->codec->field_

[FFmpeg-devel] [PATCH]Do not ask for mxf samples with unknown field dominance

2014-08-18 Thread Carl Eugen Hoyos
Hi! Attached patch removes a request for samples of which we already have several that all work fine. Please comment, Carl Eugendiff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index 7a4633f..aabae69 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -1603,9 +1603,6 @@ sta