<div dir="ltr">







<p class="gmail-p1">Hi.<br></p>
<p class="gmail-p1">I found that DTX is always unavailable when DISABLE_FLAOT_API is not defined in OPUS_AUTO mode.</p>
<p class="gmail-p1">As you know analysis_info.valid is alway true except NaN case and is_silence is alway true except digital zero signal case.</p>
<p class="gmail-p1">In general, following condition will be alway true except exceptional case.</p>
<p class="gmail-p1">(analysis_info.valid || is_silence)</p>
<p class="gmail-p1">But in a code, there is a NOT expression in front of above condition, so st->silk_mode.useDTX will be always disable</p><p class="gmail-p1">Here is the code.</p>
<p class="gmail-p1">#ifndef DISABLE_FLOAT_API</p>
<p class="gmail-p1"><span class="gmail-Apple-converted-space">       </span>st->silk_mode.useDTX = st->use_dtx && !(analysis_info.valid || is_silence);</p>
<p class="gmail-p1">#else</p>
<p class="gmail-p1"><span class="gmail-Apple-converted-space">       </span>st->silk_mode.useDTX = st->use_dtx;</p>
<p class="gmail-p1">#endif</p>
<p class="gmail-p2">Is it a bug or are there any reason for this?<br></p>
<p class="gmail-p1">







</p><p class="gmail-p1">In my opinion, the NOT expression should to be removed.</p><p class="gmail-p1">I attached a fixed file.</p>
<p class="gmail-p1">Please review.</p>
<p class="gmail-p2"><br></p>
<p class="gmail-p1">Thanks,</p>
<p class="gmail-p1">InGyu Kang</p></div>