On Tue, Jan 19, 2016 at 09:16:26AM +0100, Carl Eugen Hoyos wrote: > On Tuesday 19 January 2016 02:09:06 am Michael Niedermayer wrote: > > On Tue, Jan 19, 2016 at 12:33:40AM +0100, Carl Eugen Hoyos wrote: > > > Hi! > > > > > > Attached patch fixes ticket #4479 here. > > > > > > Please comment, Carl Eugen > > > > if w/h is set at that point then their values may be unrelated to the > > actual w/h values thus the set_sar there is wrong even if w/h is set > > > > the patch is fine, iam not sure if the call shouldnt be (re)moved > > though moving it might be somewhat non trivial > > The attached patch also fixes ticket #4479 for me. > > Thank you, Carl Eugen
also ok, maybe add a <= 0 check for the 2 parameters [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB What does censorship reveal? It reveals fear. -- Julian Assange
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel