<div dir="ltr">I will update the commit message and send a new patch.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 22, 2014 at 5:57 PM, Tim Walker <span dir="ltr"><<a href="mailto:tdskywalker@gmail.com" target="_blank">tdskywalker@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 22 Apr 2014, at 08:46, <a href="mailto:murugan@multicorewareinc.com">murugan@multicorewareinc.com</a> wrote:<br>

<br>
> # HG changeset patch<br>
> # User Murugan Vairavel <<a href="mailto:murugan@multicorewareinc.com">murugan@multicorewareinc.com</a>><br>
> # Date 1398149117 -19800<br>
> #      Tue Apr 22 12:15:17 2014 +0530<br>
> # Node ID bc8321f5968c829c6f2914b23d9eaa5106485163<br>
> # Parent  84315557c97ff2b10cf01910d6b131e28fce1781<br>
> testbench: fix crash on downshift and upshift module<br>
><br>
> Force testbench to pass height and width >= 16, As the encoder prevents encode<br>
> if height and width of the picture less than 16.<br>
<br>
</div>I'm probably being terribly picks, but I would format the commit message differently:<br>
<br>
---<br>
testbench: make sure width and height are >= 16.<br>
<br>
Fixes a crash caused by the encoder refusing to encode if the width or height are less than 16.<br>
---<br>
<br>
Patch itself looks good.<br>
<br>
Tim<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><div dir="ltr">With Regards,<div><br></div><div>Murugan. V</div><div>+919659287478</div></div>
</div>