[x265] [PATCH] frameFilter: check for reconRowFlag

[chen]

This patch made logic bug, the m_reconRowFlag and numRowFinished use to enable Sao filter when all row finished.


At 2016-09-27 19:17:16,ashok at multicorewareinc.com wrote:
># HG changeset patch
># User Ashok Kumar Mishra<ashok at 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 at videolan.org
>https://mailman.videolan.org/listinfo/x265-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20160927/4a97195b/attachment.html>