On Tue, Feb 24, 2015 at 11:41:02AM +0100, Carl Eugen Hoyos wrote: > On Monday 23 February 2015 02:02:42 pm Clément Bœsch wrote: > > On Mon, Feb 23, 2015 at 01:59:45PM +0100, Michael Niedermayer wrote: > > > On Mon, Feb 23, 2015 at 03:27:54AM +0100, Carl Eugen Hoyos wrote: > > > > > + int width = av_pix_fmt_desc_get(frame->format)->flags & > > > > AV_PIX_FMT_FLAG_PLANAR ? + frame->width : > > > > + frame->width * s->bpp; > > > > - for (j = 0; j < frame->width * s->bpp; j++) { > > > > + for (j = 0; j < width; j++) { > > > > > > should be ok, alternatively bpp could be renamed and set to 1 for > > > planar > > > > And moved out of the loop > > New patch attached.
ok if tested [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB He who knows, does not speak. He who speaks, does not know. -- Lao Tsu
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel