Re: [libvirt] [PATCH] Remove ignore_value or void from unlink calls

2018-09-20 Thread Erik Skultety
On Thu, Sep 20, 2018 at 08:12:06AM -0400, John Ferlan wrote: > There seems to be no need to add the ignore_value wrapper or > caste with (void) to the unlink() calls, so let's just remove > them. I assume at one point in time Coverity complained. So, > let's just be consistent - those that care to

[libvirt] [PATCH] Remove ignore_value or void from unlink calls

2018-09-20 Thread John Ferlan
There seems to be no need to add the ignore_value wrapper or caste with (void) to the unlink() calls, so let's just remove them. I assume at one point in time Coverity complained. So, let's just be consistent - those that care to check the return status can and those that don't can just have the na