[src/trunk]: src/bin/sh The previous commit was obviously made by a broken mi...

[kre <kre%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/f20dee86ad49
branches:  trunk
changeset: 449265:f20dee86ad49
user:      kre <kre%NetBSD.org@localhost>
date:      Fri Mar 01 06:15:01 2019 +0000

description:
The previous commit was obviously made by a broken mindless automoton
with an IQ that underflows when one attempts to enter it as an
unnormalised 160 bit long long double...

Whoever would believe that (~0 & anything) was a meaningful thing
to write?   And three times in one #define.   That could not possibly
have been me, could it?

Simplify, simplify, simplify.           NFC.

diffstat:

 bin/sh/redir.c |  7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

diffs (28 lines):

diff -r 16bb218351e3 -r f20dee86ad49 bin/sh/redir.c
--- a/bin/sh/redir.c    Fri Mar 01 06:14:52 2019 +0000
+++ b/bin/sh/redir.c    Fri Mar 01 06:15:01 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: redir.c,v 1.65 2019/03/01 05:23:35 kre Exp $   */
+/*     $NetBSD: redir.c,v 1.66 2019/03/01 06:15:01 kre Exp $   */
 
 /*-
  * Copyright (c) 1991, 1993
@@ -37,7 +37,7 @@
 #if 0
 static char sccsid[] = "@(#)redir.c    8.2 (Berkeley) 5/4/95";
 #else
-__RCSID("$NetBSD: redir.c,v 1.65 2019/03/01 05:23:35 kre Exp $");
+__RCSID("$NetBSD: redir.c,v 1.66 2019/03/01 06:15:01 kre Exp $");
 #endif
 #endif /* not lint */
 
@@ -764,8 +764,7 @@
     O_ALT_IO|O_DIRECT|O_NOSIGPIPE)
 
 #ifndef        O_CLOEXEC
-# define O_CLOEXEC     ((~0 & ~ALLFLAGS) ^      \
-                               ((~0 & ~ALLFLAGS) & ((~0 & ~ALLFLAGS) - 1)))
+# define O_CLOEXEC     ((~ALLFLAGS) ^ ((~ALLFLAGS) & ((~ALLFLAGS) - 1)))
 #endif
 
        /* for any system we support, close on exec is always defined */