<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2015-07-30 9:32 GMT-03:00 Henning Rogge <span dir="ltr"><<a href="mailto:hrogge@gmail.com" target="_blank">hrogge@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just another thought about the code itself...<br>
<br>
any reason we could not simplify the following function to something like this?<br>
<br>
static unsigned int<br>
heap_perfect_log2 (unsigned int number) {<br>
    unsigned int log = 0, pow=1, original_number=number, i;<br>
    while (number >>= 1) ++log;<br>
    return original_number - (1 << log);<br>
}<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>No, the results are the same and the function looks cleaner. I will change this function in my patch.</div></div></div></div>