<div dir="ltr">Thanks for pointing that out, that was indeed unintentional. Pushed the fix. <br><div><div id="__tbSetup"></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Sep 16, 2013 at 9:11 PM, Derek Buitenhuis <span dir="ltr"><<a href="mailto:derek.buitenhuis@gmail.com" target="_blank">derek.buitenhuis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, Sep 16, 2013 at 1:30 PM, Deepthi Nandakumar<br>
<<a href="mailto:deepthi@multicorewareinc.com">deepthi@multicorewareinc.com</a>> wrote:<br>
> # HG changeset patch<br>
> # User Deepthi Nandakumar <<a href="mailto:deepthi@multicorewareinc.com">deepthi@multicorewareinc.com</a>><br>
> # Date 1379334518 -19800<br>
> # Node ID 46b065f7d676e7ff26c46a40f1790bdae290d7fa<br>
> # Parent  881444f5910b2b0e0f286a6ca47fcc743515cbb2<br>
> CLI options: Eliminate rdoqts option; cleanup<br>
><br>
> 1. Eliminate rdoqts CLI option: enabled when rdoq and ts are both enabled.<br>
> 2. Rearrange default initialisations in x265_param_ t structure<br>
><br>
> diff -r 881444f5910b -r 46b065f7d676 source/common/common.cpp<br>
> --- a/source/common/common.cpp    Mon Sep 16 09:41:34 2013 +0530<br>
> +++ b/source/common/common.cpp    Mon Sep 16 17:58:38 2013 +0530<br>
> @@ -115,45 +115,58 @@<br>
>      va_end(arg);<br>
>  }<br>
><br>
> -extern "C"<br>
>  void x265_param_default(x265_param_t *param)<br>
<br>
</div>This looks incorrect. The function needs to be exported and to be able<br>
to be called<br>
from C.<br>
<div class="im"><br>
<br>
> diff -r 881444f5910b -r 46b065f7d676 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 17:58:38 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 = 0;<br>
> +    }<br>
<br>
</div>Please add a note in the commit message about this.<br>
<br>
Rest is OK.<br>
<br>
- Derek<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></div>