<p dir="ltr"><br>
On Jun 10, 2015 22:02, "Dirk Hohndel" <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br>
><br>
> So you treat this function as void (called as void, no return at the end),<br>
> yet it's declared as int and returns report_error(...) in case of error.<br>
><br>
> Which one should it be?</p>
<p dir="ltr">The "int" return is literally because it makes the code cleaner with the whole</p>
<p dir="ltr">    return report_error(..);</p>
<p dir="ltr">thing, but nobody ever actually cares whether it succeeds or not, since we will end up using the cached copy regardless.</p>
<p dir="ltr">So it's a "'yes, it returns a status, but you don't necessarily have to use it" thing. </p>
<p dir="ltr">Not unusual. Lots of C functions return calls that are never used. How often have you used the return value of "memcpy()"? </p>
<p dir="ltr">     Linus<br>
</p>