I will correct it and send a new patch.
On Wed, Mar 12, 2014 at 6:04 PM, Derek Buitenhuis < [email protected]> wrote: > On 3/12/2014 8:45 AM, Steve Borho wrote: > >> > - for (int c = 0; c < width; c++) > >> > + y += lumaWidth; > >> > + Y += lumaWidth; > >> > + v += chromaWidth; > >> > + V += chromaWidth; > >> > + u += chromaWidth; > >> > + U += chromaWidth; > > > > blech. the destination buffer has loads of padding (stride is much > > wider than width), writing past the right edge is perfectly safe but > > you do need to be careful about reading beyond the input stride. > > I'd just like to note that using one letter variables that differ only by > case > is *insane*, and renders the code unreadable. > > - Derek > _______________________________________________ > x265-devel mailing list > [email protected] > https://mailman.videolan.org/listinfo/x265-devel > -- With Regards, Murugan. V +919659287478
_______________________________________________ x265-devel mailing list [email protected] https://mailman.videolan.org/listinfo/x265-devel
