<div dir="ltr">Thanks Jean-Marc for the follow up.<div>+Gustaf (author of the patch), who would have more to say!<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Sep 23, 2019 at 4:41 PM Jean-Marc Valin <<a href="mailto:jmvalin@jmvalin.ca">jmvalin@jmvalin.ca</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Yves,<br>
<br>
Sorry for the delay (I blame the Google spam filter). Have you actually<br>
seen problems that this patch fixes? If so, do you have a way to<br>
reproduce the problem?<br>
<br>
While technically your patch would be safe, the simple fact that it<br>
would trigger would mean a bug elsewhere. Note that HB_gain is<br>
essentially 1 - exp(-celt_rate/1024.) where celt_rate is supposed to<br>
always be non-negative because compute_silk_rate_for_hybrid() should<br>
never give SILK more bits than there are in total. That means HB_gain<br>
should be bounded between 0 and 1 and should definitely never be NaN.<br>
<br>
So either the patch does nothing (in which case we don't need it), or it<br>
does something and there's something much scarier going on than NaNs. In<br>
the latter case, I think more investigation is required.<br>
<br>
Cheers,<br>
<br>
        Jean-Marc<br>
<br>
<br>
<br>
On 9/12/19 1:00 PM, Yves Gerey wrote:<br>
> Hi, I'm still Yves.<br>
> <br>
> This other fix<br>
> <<a href="https://chromium-review.googlesource.com/c/chromium/src/+/1061499/" rel="noreferrer" target="_blank">https://chromium-review.googlesource.com/c/chromium/src/+/1061499/</a>> never<br>
> made it to opus repository.<br>
> <br>
> It seems safe and sane enough to be applied to opus master branch.<br>
> What do you think?<br>
> <br>
> -- <br>
> Warm regards,<br>
> Yves<br>
> <br>
> _______________________________________________<br>
> opus mailing list<br>
> <a href="mailto:opus@xiph.org" target="_blank">opus@xiph.org</a><br>
> <a href="http://lists.xiph.org/mailman/listinfo/opus" rel="noreferrer" target="_blank">http://lists.xiph.org/mailman/listinfo/opus</a><br>
> <br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Warm regards,<div>Yves</div></div></div>