<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 16, 2013 at 1:06 AM, Deepthi Nandakumar <span dir="ltr"><<a href="mailto:deepthi@multicorewareinc.com" target="_blank">deepthi@multicorewareinc.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"># HG changeset patch<br># User Deepthi Nandakumar <<a href="mailto:deepthi@multicorewareinc.com" target="_blank">deepthi@multicorewareinc.com</a>><br>
# Date 1379311495 -19800<br># Node ID 22c4e67c7246cd77968a86e967377e4a18b47b31<br>
# Parent  881444f5910b2b0e0f286a6ca47fcc743515cbb2<br>RDLevel: remove CLI option no-rdoqts<br><br>If RDOQ and TS are enabled, RDOQTS is enabled.<br></div></blockquote><div><br></div><div style>For some reason, Deepthi, your patches are line wrapped.  Can you push this?</div>
<div style><br></div><div style>Also, I wonder if we should be changing the way the coding tools are listed in the log to match the new CLI format</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><br>diff -r 881444f5910b -r 22c4e67c7246 source/encoder/encoder.cpp<br>--- a/source/encoder/encoder.cpp    Mon Sep 16 09:41:34 2013 +0530<br>
+++ b/source/encoder/encoder.cpp    Mon Sep 16 11:34:55 2013 +0530<br>@@ -219,6 +219,11 @@<br>         _param->rc.rateControlMode = X265_RC_ABR;<br>     }<br> <br>+    if(_param->bEnableRDOQ && _param->bEnableTransformSkip)<br>

+    {<br>+        _param->bEnableRDOQTS = 1;<br>+    }<br>+<br>     /* Set flags according to RDLevel specified - check_params has verified that RDLevel is within range */<br>     switch(_param->bRDLevel)<br>     {<br>

diff -r 881444f5910b -r 22c4e67c7246 source/x265opts.h<br>--- a/source/x265opts.h    Mon Sep 16 09:41:34 2013 +0530<br>+++ b/source/x265opts.h    Mon Sep 16 11:34:55 2013 +0530<br>@@ -80,8 +80,6 @@<br> OPT("cbqpoffs",        param->cbQpOffset,             required_argument, 0, "Chroma Cb QP Offset")<br>

 OPT("crqpoffs",        param->crQpOffset,             required_argument, 0, "Chroma Cr QP Offset")<br> OPT("rd",              param->bRDLevel,               required_argument, 0, "Level of RD in mode decision 0:least....2:full RDO")<br>

-OPT("no-rdoqts",       param->bEnableRDOQTS,                no_argument, 0, "Disable RDO quantization with transform skip")<br>-OPT("rdoqts",          param->bEnableRDOQTS,                no_argument, 0, "Enable RDO quantization with transform skip")<br>

 OPT("no-signhide",     param->bEnableSignHiding,            no_argument, 0, "Disable hide sign bit of one coeff per TU (rdo)")<br> OPT("signhide",        param->bEnableSignHiding,            no_argument, 0, "Hide sign bit of one coeff per TU (rdo)")<br>

 <br><br></div>
<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>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Steve Borho
</div></div>