<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 19, 2013 at 7:02 AM, 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 9/19/2013 10:16 AM, Min Chen wrote:<br>
> +// Disabled because Frame Parallelism call processRowPost early, it is broken data.<br>
> +// OPT("sao-lcu-opt",     param->saoLcuBasedOptimization,required_argument, 0, "0: SAO picture-based optimization, 1: SAO LCU-based optimization ")<br>
<br>
</div>... Why not fix it instead of commenting stuff out?<br></blockquote></div><div class="gmail_extra"><br></div>I think I agree; we should keep the feature but don't allow an encoder to use it in combination with frame threads.  For lower CPU core counts, this will still be quite feasible.<br clear="all">
<div><br></div>-- <br>Steve Borho
</div></div>