On Sun, 1 Mar 2015 21:59:02 +0000
Michael Forney <[email protected]> wrote:
Hey Michael,
Thanks for paying so close attention! You are right
about the return value!
> if (remove(path) < 0) {
> + if (!rm_fflag || errno != ENOENT)
> + rm_status = 1;
> if (!rm_fflag)
> weprintf("remove %s:", path);
> - rm_status = 1;
> }
I'd do it like this instead, makes it clearer that we're
dealing with one special case:
if (remove(path) < 0) {
if (!rm_fflag)
weprintf("remove %s:", path);
rm_status = !(rm_fflag && errno == ENOENT);
}
What do you think?
Cheers
FRIGN
--
FRIGN <[email protected]>
