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
signature.asc
Description: This is a digitally signed message part
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
