[FFmpeg-devel] [PATCH]Read aspect ratio from mxf
tomas.hardin at codemill.se
Fri Nov 21 10:18:17 CET 2014
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 force the sar to 4:3
> > > if h264 10bit 1440x1080 video has sar 3:4.
> > > + av_dict_set(&st->metadata, "display_aspect_ratio_num",
> > > NULL, 0); + av_dict_set(&st->metadata,
> > > "display_aspect_ratio_den", NULL, 0); + }
> > 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.
> Thank you, Carl Eugen
Looks good to me. I recall doing something similar way back that I never
sent to this list, in order to deal with similar issues.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 836 bytes
Desc: This is a digitally signed message part
More information about the ffmpeg-devel