20 Sep
2018
20 Sep
'18
4:17 p.m.
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 check the return status can and those that don't can just have the naked unlink.
Signed-off-by: John Ferlan <jferlan@redhat.com> ---
As a result of Michal's review comment on my recent storage driver error path series...
MinGW is happy, Clang is happy, what more can one wish for?! :) Reviewed-by: Erik Skultety <eskultet@redhat.com>