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@xxxxxxxxxx> > --- > > 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@xxxxxxxxxx> -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
