<div dir="ltr"><div><div>Before I submit a patch with this change, I have a question.  I see that pow in the EqzConvertdB, PresetCallback, and PreampCallback functions is used instead of powf, but I didn't make these parts of the code.  Also, the double-precision version of real-valued literals has been used throughout the code long before I submitted my very first patch to the equalizer module a year ago.  Wouldn't it be wise to change these as well?  I could imagine that this would potentially lead to a snowball effect where I (or someone else) would have to go through the entire VLC code and change the double-precision versions of functions and literals used in assignments to single-precision variables and function arguments to their respective single-precision versions.<br>
</div><div><br></div>Regards,<br><br></div>Ron Wright<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Mar 30, 2013 at 5:14 AM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le vendredi 29 mars 2013 00:58:47, Ronald Wright a écrit :<br>
<div class="im">> It is easy to see that the computation of the alpha constant is slightly<br>
> inefficient, as the root value can be close to 1 for the lower frequencies,<br>
> which would result in significant roundoff error if this value is<br>
> subtracted from 1.  In this patch, the computation of the alpha, beta, and<br>
> gamma constants has been simplified and refactored.<br>
<br>
</div>TBH I don't have a pencil and a paper to double check the maths right now.<br>
Just wondering though, shouldn't sinf(), cosf() and .5f be used instead? (That<br>
could be a separate patch anyway.)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
</font></span></blockquote></div><br></div>