<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 21, 2013 at 5:45 AM, <span dir="ltr"><<a href="mailto:deepthi@multicorewareinc.com" target="_blank">deepthi@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Deepthi Nandakumar <<a href="mailto:deepthi@multicorewareinc.com">deepthi@multicorewareinc.com</a>><br>
# Date 1382352322 -19800<br>
# Node ID f987c24c7bf2d3e6df0f96b2c75f3e71ab15a85e<br>
# Parent b2aa2aad2c6654797befe6098b7d14609b060299<br>
lowres: right and bottom margins are being extended twice?<br>
<br>
diff -r b2aa2aad2c66 -r f987c24c7bf2 source/common/lowres.cpp<br>
--- a/source/common/lowres.cpp Mon Oct 21 12:55:18 2013 +0530<br>
+++ b/source/common/lowres.cpp Mon Oct 21 16:15:22 2013 +0530<br>
@@ -158,25 +158,7 @@<br>
{<br>
::memcpy(src + y * srcStride, src, sizeof(Pel) * (extWidth));<br>
}<br>
-<br>
- /* extending right margin*/<br>
- if (2 * width > orig->getWidth())<br>
- {<br>
- for (y = 0; y < srcHeight; y++)<br>
- {<br>
- ::memset(src + srcWidth, src[srcWidth - 1], sizeof(Pel) * (X265_LOWRES_CU_SIZE - 1));<br>
- src += srcStride;<br>
- }<br>
- }<br>
-<br>
- /* extending bottom margin */<br>
- src = orig->getLumaAddr() + (srcHeight - 1) * srcStride;<br>
-<br>
- for (y = 1; y <= 2 * lines - srcHeight; y++)<br>
- {<br>
- ::memcpy(src + y * srcStride, src, sizeof(Pel) * (extWidth));<br>
- }<br>
-<br>
+<br></blockquote><div><br></div><div>good catch; not sure what happened here</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
/* downscale and generate 4 HPEL planes for lookahead */<br>
primitives.frame_init_lowres_core(orig->getLumaAddr(),<br>
lowresPlane[0], lowresPlane[1], lowresPlane[2], lowresPlane[3],<br>
_______________________________________________<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>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>