Re: I: coreutils-20041123: src/touch.c regression

[Paul Eggert]

Jim Meyering <address@hidden> writes:

> Your switch statement should probably handle EBADF, too,
> to avoid an unnecessary utimes call after such a failure.

EBADF shouldn't happen as it violates the futimens documentation,
which says "FD must be either negative -- in which case it is ignored
-- or a file descriptor that is open on FILE.", and the code in
question is reachable only if FD is nonnegative.  I omitted a test for
EBADF because I figured that there's no point optimizing cases that
shouldn't happen.