<div dir="ltr">I will correct it and send a new patch.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 12, 2014 at 6:04 PM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 3/12/2014 8:45 AM, Steve Borho wrote:<br>
>> > -            for (int c = 0; c < width; c++)<br>
>> > +            y += lumaWidth;<br>
>> > +            Y += lumaWidth;<br>
>> > +            v += chromaWidth;<br>
>> > +            V += chromaWidth;<br>
>> > +            u += chromaWidth;<br>
>> > +            U += chromaWidth;<br>
><br>
> blech. the destination buffer has loads of padding (stride is much<br>
> wider than width), writing past the right edge is perfectly safe but<br>
> you do need to be careful about reading beyond the input stride.<br>
<br>
</div>I'd just like to note that using one letter variables that differ only by case<br>
is *insane*, and renders the code unreadable.<br>
<span class="HOEnZb"><font color="#888888"><br>
- Derek<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">With Regards,<div><br></div><div>Murugan. V</div><div>+919659287478</div></div>
</div>