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 <[email protected]> > --- > > 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 <[email protected]> -- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
