From de3622c2af51046d476a9daf6b8f77df0ae20c9f Mon Sep 17 00:00:00 2001
From: Bruno Haible <bruno@clisp.org>
Date: Sun, 17 Apr 2011 23:36:27 +0200
Subject: [PATCH] pipe2: Simplify code.

* lib/pipe2.c (pipe2): Reduce code duplication.
---
 ChangeLog   |  5 +++++
 lib/pipe2.c | 22 ++++++++--------------
 2 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 1f09ff845..c5e07aa07 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2011-04-17  Bruno Haible  <bruno@clisp.org>
 
+	pipe2: Simplify code.
+	* lib/pipe2.c (pipe2): Reduce code duplication.
+
+2011-04-17  Bruno Haible  <bruno@clisp.org>
+
 	nonblocking: Add comment.
 	* lib/fcntl.in.h (O_NONBLOCK): Add comment.
 
diff --git a/lib/pipe2.c b/lib/pipe2.c
index 6ffb101f3..18098c4b4 100644
--- a/lib/pipe2.c
+++ b/lib/pipe2.c
@@ -66,23 +66,17 @@ pipe2 (int fd[2], int flags)
 #if (defined _WIN32 || defined __WIN32__) && ! defined __CYGWIN__
 /* Native Woe32 API.  */
 
+  if (_pipe (fd, 4096, flags & ~O_NONBLOCK) < 0)
+    return -1;
+
   if (flags & O_NONBLOCK)
     {
-      int result = _pipe (fd, 4096, flags & ~O_NONBLOCK);
-      if (result != 0)
-        return result;
       if (set_nonblocking_flag (fd[0], true) != 0
           || set_nonblocking_flag (fd[1], true) != 0)
-        {
-          int saved_errno = errno;
-          close (fd[0]);
-          close (fd[1]);
-          result = -1;
-          errno = saved_errno;
-        }
-      return result;
+        goto fail;
     }
-  return _pipe (fd, 4096, flags);
+
+  return 0;
 
 #else
 /* Unix API.  */
@@ -131,6 +125,8 @@ pipe2 (int fd[2], int flags)
 
   return 0;
 
+#endif
+
  fail:
   {
     int saved_errno = errno;
@@ -139,6 +135,4 @@ pipe2 (int fd[2], int flags)
     errno = saved_errno;
     return -1;
   }
-
-#endif
 }
-- 
2.11.0