<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div> </div><pre><br>At 2014-07-16 07:32:51,"Steve Borho" <steve@borho.org> wrote:
>On 07/15, Min Chen wrote:
>> # HG changeset patch
>> # User Min Chen <a href="mailto:chenm003@163.com>>> # Date 1405471890 25200>> # Node ID 78f7b217e5d53ab981bb0b5ac0f43e8c46260c9f>> # Parent  c923f4a9494619665bf49db7ae0e250e2f8c4ec7>> replace sse_sp(residual, ZERO) by ssd_s(residual)>> >> diff -r c923f4a94946 -r 78f7b217e5d5 source/Lib/TLibEncoder/TEncSearch.cpp>> --- a/source/Lib/TLibEncoder/TEncSearch.cpp        Mon Jul 14 17:27:04 2014 +0530>> +++ b/source/Lib/TLibEncoder/TEncSearch.cpp   Tue Jul 15 17:51:30 2014 -0700>> @@ -2374,9 +2374,8 @@>>      if ((cu->getSlice()->getPPS()->getTransquantBypassEnableFlag()))>>      {>>          bIsTQBypassEnable = true; // mark that the first iteration is to cost TQB mode.>> -        tqBypassMode = 2;>> -        if (m_param->bLossless)>> -            tqBypassMode = 1;>> +        if (!m_param->bLossless)>> +            tqBypassMode = 2;>>The patch looks good except this part, I'd like Ashok to review this>change. it looks unrelated to the rest of the patch anyway.>">chenm003@163.com>
>> # Date 1405471890 25200
>> # Node ID 78f7b217e5d53ab981bb0b5ac0f43e8c46260c9f
>> # Parent  c923f4a9494619665bf49db7ae0e250e2f8c4ec7
>> replace sse_sp(residual, ZERO) by ssd_s(residual)
>> 
>> diff -r c923f4a94946 -r 78f7b217e5d5 source/Lib/TLibEncoder/TEncSearch.cpp
>> --- a/source/Lib/TLibEncoder/TEncSearch.cpp  Mon Jul 14 17:27:04 2014 +0530
>> +++ b/source/Lib/TLibEncoder/TEncSearch.cpp  Tue Jul 15 17:51:30 2014 -0700
>> @@ -2374,9 +2374,8 @@
>>      if ((cu->getSlice()->getPPS()->getTransquantBypassEnableFlag()))
>>      {
>>          bIsTQBypassEnable = true; // mark that the first iteration is to cost TQB mode.
>> -        tqBypassMode = 2;
>> -        if (m_param->bLossless)
>> -            tqBypassMode = 1;
>> +        if (!m_param->bLossless)
>> +            tqBypassMode = 2;
>
>The patch looks good except this part, I'd like Ashok to review this
>change. it looks unrelated to the rest of the patch anyway.
>
</a>Its a part of another patch, please remove it.</pre><pre> </pre></div>