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

Tomas Härdin 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.

/Tomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141121/632e5ff0/attachment.asc>


More information about the ffmpeg-devel mailing list