On Tue, 2015-04-28 at 12:27 +0200, Simon Josefsson wrote:

> This patch looks wrong to me -- it changes a void function into an int
> function, and changes return values from void to 0, which the caller
> ignores.  


I dont see a problem there. It is only used for avoiding errors compling
with CLANG.
If or if not a caller ignores a return value is not an issue.


> As far as I can tell, the patch is not needed at all since
> upstream has fixed this issue.  


Not in version 2.3.3
/Willem

Reply via email to