[FFmpeg-devel] [PATCH]Read aspect ratio from mxf
Michael Niedermayer
michaelni at gmx.at
Sun Nov 16 04:34:46 CET 2014
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>
More information about the ffmpeg-devel
mailing list