<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 7, 2013 at 4:07 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div>Since the default preset is medium, shouldnt x265_param_default apply medium settings only? So that the if (!strcmp(preset, "medium")) block here is empty?? <br>
</div>In that case, the preset (if specified) will change only the ones which are different from medium?<br></div></div></div></div></div></blockquote><div><br></div><div>that is where we need to get to; once we've settled on the contents of medium preset</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>
<br></div>Also, there are a few params here that<br></div>1. arent used and arent initialised properly (eg, constrainedIntra, weightedBipred)  lets remove them and add them back when the feature exists.<br></div></div></div>
</blockquote><div><br></div><div>no argument from me, though I suspect constrained intra is already functional from the HM code.  We just haven't made it a priority to test.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div><div></div>2. that are dependent on other param fields, like EnableRdo, EnabledRDOQ depends on RDLevel. This particular case is handled safely inside encoder::configure, but not sure about all such cases. <br>

<br></div>Just thought this is a good time for springcleaning. <br></div></blockquote><div><br></div><div>Yes, in my view EnabledRDOQ and EnableRdo should be removed, and the internal code should use RDLevel directly.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"></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>