<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On 18 Dec. 2017 12:20 am, "Berthold Stoeger" <<a href="mailto:bstoeger@mail.tuwien.ac.at">bstoeger@mail.tuwien.ac.at</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rick,<br>
<br>
In current master, gcc produces an uninitialized variable warning in profile.c,<br>
which seems correct to me.<br>
<br>
In l.1044 last_ceiling is compared to first_ceiling, but last_ceilng is set<br>
first in l.1056. I guess decoMode() cannot change in this function, so gcc's<br>
analysis seems correct.<br>
<br>
There is another warning concerning entry in l.807. To make the warning go<br>
away, you could add a<br>
  if (pi->nr <= 0)<br>
    return;<br>
block before the loop. Even if pi->nr can never be <= 0, this seems like a<br>
good idea for robustness' sake.</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Hi Berthold,</div><div dir="auto">Timezone differences mean I missed this discussion. My experience in programming is limited and entirely informal. So long as the result is as expected, I will happily defer to the judgement of you, Dirk and Robert.</div><div dir="auto"><br></div><div dir="auto">Cheers,</div><div dir="auto">Rick</div></div>