Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-21 Thread Tomas Härdin
On Sun, 2014-11-16 at 02:03 +0100, Carl Eugen Hoyos wrote: > On Saturday 15 November 2014 11:57:00 pm Michael Niedermayer wrote: > > On Sat, Nov 15, 2014 at 02:50:38AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4107 for me. > > > An alternative would be to for

Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-18 Thread Carl Eugen Hoyos
Carl Eugen Hoyos ag.or.at> writes: > > I suggest you add a documented as private/internal > > display_aspect_ratio to AVStream instead of metadata > > also av_reduce can be replaced by av_mul_q which is > probably simpler > > New patch attached. The patch was merged. Thank you, Carl Eugen __

Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-15 Thread Michael Niedermayer
On Sun, Nov 16, 2014 at 02:03:04AM +0100, Carl Eugen Hoyos wrote: > On Saturday 15 November 2014 11:57:00 pm Michael Niedermayer wrote: > > On Sat, Nov 15, 2014 at 02:50:38AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4107 for me. > > > An alternative would be

Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-15 Thread Carl Eugen Hoyos
On Saturday 15 November 2014 11:57:00 pm Michael Niedermayer wrote: > On Sat, Nov 15, 2014 at 02:50:38AM +0100, Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #4107 for me. > > An alternative would be to force the sar to 4:3 > > if h264 10bit 1440x1080 video has sar 3:4. > > +

Re: [FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-15 Thread Michael Niedermayer
On Sat, Nov 15, 2014 at 02:50:38AM +0100, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4107 for me. > An alternative would be to force the sar to 4:3 > if h264 10bit 1440x1080 video has sar 3:4. > > Please review, Carl Eugen > mxfdec.c |4 > utils.c | 15 +

[FFmpeg-devel] [PATCH]Read aspect ratio from mxf

2014-11-14 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4107 for me. An alternative would be to force the sar to 4:3 if h264 10bit 1440x1080 video has sar 3:4. Please review, Carl Eugen diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index b533e2a..3ac5e68 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/m