I think this patch is enough to meet your requirement.<br><br><div class="gmail_quote">2011/4/23 Takashi Mochizuki <span dir="ltr"><<a href="mailto:mochi@da2.so-net.ne.jp">mochi@da2.so-net.ne.jp</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I think these snippet has a problem with x264_encoder_reconfig().<br>
<div class="im"><br>
+        else if(  h->param.rc.i_aq_mode && h->param.analyse.b_psnr )<br>
+        {<br>
</div>+            x264_log( h, X264_LOG_WARNING, "--psnr used with AQ on: results will be invalid!\n" );<br>
+            s = "psnr";<br>
+        }<br>
<br>
If libx264.a is used with --tune psnr --psnr equivalent parameters,<br>
libx264 claims "--psnr used with AQ on..." when x264_encoder_reconfig() is called.<br>
Because current implementation in x264_validate_parameters() could force i_aq_mode to 1.<br>
<br>
So it should be:<br>
+        else if(  h->param.rc.i_aq_mode && h->param.rc.f_aq_strength != 0 && h->param.analyse.b_psnr )<br>
<div class="im"><br>
<br>
>> encoder/encoder.c - x264_validate_parameters()<br>
>>  775     h->param.rc.i_aq_mode = x264_clip3( h->param.rc.i_aq_mode, 0, 2 );<br>
>>  776     h->param.rc.f_aq_strength = x264_clip3f( h->param.rc.f_aq_strength, 0, 3 );<br>
>>  777     if( h->param.rc.f_aq_strength == 0 )<br>
>>  778         h->param.rc.i_aq_mode = 0;<br>
>>  779     /* MB-tree requires AQ to be on, even if the strength is zero. */<br>
>>  780     if( !h->param.rc.i_aq_mode && h->param.rc.b_mb_tree )<br>
>>  781     {<br>
>>  782         h->param.rc.i_aq_mode = 1;<br>
>>  783         h->param.rc.f_aq_strength = 0;<br>
>>  784     }<br>
>><br>
>> Thus, when I use --tune psnr with mb_tree, it forces aq_mode on, and I will get<br>
<br>
<br>
</div><font color="#888888">Takashi Mochizuki<br>
</font><div><div></div><div class="h5">_______________________________________________<br>
x264-devel mailing list<br>
<a href="mailto:x264-devel@videolan.org">x264-devel@videolan.org</a><br>
<a href="http://mailman.videolan.org/listinfo/x264-devel" target="_blank">http://mailman.videolan.org/listinfo/x264-devel</a><br>
</div></div></blockquote></div><br>