<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial">This patch made logic bug, the <span style="line-height: 23.8px; white-space: pre-wrap;">m_reconRowFlag and </span><span style="line-height: 23.8px; white-space: pre-wrap;">numRowFinished </span><span style="line-height: 23.8px; white-space: pre-wrap;">use to enable Sao filter when all row finished.</span><br><pre><br>At 2016-09-27 19:17:16,ashok@multicorewareinc.com wrote:
># HG changeset patch
># User Ashok Kumar Mishra<ashok@multicorewareinc.com>
># Date 1474974965 -19800
>#      Tue Sep 27 16:46:05 2016 +0530
># Node ID 5fa48115cfaa9022a72c84337b46df366c063ad0
># Parent  c0d91c2b40484664c3420abfffa10fa9cb707598
>frameFilter: check for reconRowFlag
>
>diff -r c0d91c2b4048 -r 5fa48115cfaa source/encoder/framefilter.cpp
>--- a/source/encoder/framefilter.cpp   Tue Sep 27 14:37:25 2016 +0530
>+++ b/source/encoder/framefilter.cpp   Tue Sep 27 16:46:05 2016 +0530
>@@ -503,10 +503,13 @@
>         processPostRow(row);
> 
>     // NOTE: slices parallelism will be execute out-of-order
>-    int numRowFinished;
>-    for(numRowFinished = 0; numRowFinished < m_numRows; numRowFinished++)
>-        if (!m_frame->m_reconRowFlag[numRowFinished].get())
>-            break;
>+    int numRowFinished = 0;
>+    if (m_frame->m_reconRowFlag)
>+    {
>+        for (numRowFinished = 0; numRowFinished < m_numRows; numRowFinished++)
>+            if (!m_frame->m_reconRowFlag[numRowFinished].get())
>+                break;
>+    }
> 
>     if (numRowFinished == m_numRows)
>     {
>_______________________________________________
>x265-devel mailing list
>x265-devel@videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
</pre></div>