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

[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 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

>  avformat.h |    7 +++++++
>  mxfdec.c   |    2 ++
>  utils.c    |    5 +++++
>  3 files changed, 14 insertions(+)
> d1e24c2beb16c29ab9a79f9b745048b3b9245e15  patchmxfdar2.diff
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h

LGTM

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141116/dcc86ade/attachment.asc>