<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 11, 2013 at 9:52 AM, Min Chen <span dir="ltr"><<a href="mailto:chenm003@163.com" target="_blank">chenm003@163.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 Min Chen <<a href="mailto:chenm003@163.com">chenm003@163.com</a>><br>
# Date 1378911096 -28800<br>
# Node ID 8075ccfcde5697f8bbb3d5333d79406bcfd57960<br>
# Parent  91523e37c65e912368fc7c9c67ea9e482251a20a<br>
framefilter: Make code clearer<br>
We do not need the row delay when the loopfilter is disabled.<br></blockquote><div><br></div><div>To be even more pedantic, I usually put a blank line between the summary line and the longer description.  I've queued this with this tweak.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff -r 91523e37c65e -r 8075ccfcde56 source/encoder/frameencoder.cpp<br>
--- a/source/encoder/frameencoder.cpp   Wed Sep 11 22:50:25 2013 +0800<br>
+++ b/source/encoder/frameencoder.cpp   Wed Sep 11 22:51:36 2013 +0800<br>
@@ -947,10 +947,8 @@<br>
     // this row of CTUs has been encoded<br>
     if (row == m_numRows - 1)<br>
     {<br>
-        int lag = m_filterRowDelay;<br>
-        while (--lag)<br>
-            m_frameFilter.processRow(row - lag);<br>
-        m_frameFilter.processRow(row);<br>
+        for(int i = m_numRows - m_filterRowDelay; i < m_numRows; i++)<br>
+            m_frameFilter.processRow(i);<br>
         m_completionEvent.trigger();<br>
     }<br>
 }<br>
diff -r 91523e37c65e -r 8075ccfcde56 source/encoder/framefilter.cpp<br>
--- a/source/encoder/framefilter.cpp    Wed Sep 11 22:50:25 2013 +0800<br>
+++ b/source/encoder/framefilter.cpp    Wed Sep 11 22:51:36 2013 +0800<br>
@@ -104,10 +104,7 @@<br>
<br>
     if (!m_cfg->param.bEnableLoopFilter)<br>
     {<br>
-        if (row > 0)<br>
-            processRowPost(row - 1);<br>
-        if (row == m_numRows - 1)<br>
-            processRowPost(row);<br>
+        processRowPost(row);<br>
         return;<br>
     }<br>
<br>
<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>