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
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
