<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 8, 2016 at 9:40 AM, Mario *LigH* Rohkrämer <span dir="ltr"><<a href="mailto:contact@ligh.de" target="_blank">contact@ligh.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I did not receive this patch via mailing list, but found it in the commits:<br></blockquote><div><br></div><div>Liujun had sent this patch to the mailing list.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="https://bitbucket.org/multicoreware/x265/commits/85671d5164b967c9d7921ed3cb5a2995efbd3c2c" rel="noreferrer" target="_blank">https://bitbucket.org/multicor<wbr>eware/x265/commits/85671d5164b<wbr>967c9d7921ed3cb5a2995efbd3c2c</a><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
# HG changeset patch<br>
# User liujun<br>
# Date 1472536596 -28800<br>
# Node ID 85671d5164b967c9d7921ed3cb5a29<wbr>95efbd3c2c<br>
# Parent 7878a576a112f61b9aa496397d995e<wbr>b918d4f0dc<br>
add --[no-]analysis-src-pics parameter to enable/disable use source frame pixels to do motion estimation.<br>
</blockquote>
<br>
There seems to be a copy/paste mistake, creating a double documentation for the new parameter:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
@@ -380,6 +382,8 @@<br>
" - 2 : Last pass, does not overwrite stats file\n"<br>
" - 3 : Nth pass, overwrites stats file\n");<br>
H0(" --stats Filename for stats file in multipass pass rate control. Default x265_2pass.log\n");<br>
+ H0(" --analyze-src-pics Motion estimation uses source frame planes. Default disable\n");<br>
+ H0(" --[no-]analyze-src-pics Motion estimation uses source frame planes. Default disable\n");<br>
H0(" --[no-]slow-firstpass Enable a slow first pass in a multipass rate control mode. Default %s\n", OPT(param->rc.bEnableSlowFirst<wbr>Pass));<br>
H0(" --[no-]strict-cbr Enable stricter conditions and tolerance for bitrate deviations in CBR mode. Default %s\n", OPT(param->rc.bStrictCbr));<br>
H0(" --analysis-mode <string|int> save - Dump analysis info into file, load - Load analysis buffers from the file. Default %d\n", param->analysisMode);<br>
</blockquote>
<br>
You may probably want to fix this double documentation with a future patch removing the first of both lines:<br></blockquote><div><br></div><div>Good catch - removed it now</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- H0(" --analyze-src-pics Motion estimation uses source frame planes. Default disable\n");<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
Fun and success!<br>
Mario *LigH* Rohkrämer<br>
mailto:<a href="mailto:contact@ligh.de" target="_blank">contact@ligh.de</a><br>
<br>
______________________________<wbr>_________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/l<wbr>istinfo/x265-devel</a><br>
</font></span></blockquote></div><br></div></div>