Re: new snapshot available: coreutils-7.2.69-269b

2009-04-30 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

According to Jim Meyering on 4/30/2009 10:14 AM:
 So far there have been only two problems (thanks for the testing
 and reports, Elbert Pol and James Youngman!) and both are fixed
 in this new snapshot.
 
 I'll release coreutils-7.3 tomorrow if this no-known-problems
 condition persists for a few hours longer.

How about this quickie cleanup patch?  stdopen is unused (was it ever
used? only lib/ChangeLog-2007 mentions it), and it seems that our current
reliance on gnulib's stdio--.h and friends does the same job.

- --
Don't work too hard, make some time for fun as well!

Eric Blake e...@byu.net
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.9 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkn5+/UACgkQ84KuGfSFAYA1CQCeLCZZ/ashgBCutJVSU3i3DjBU
tigAoJI58uhBKXwAcLEEf9hkXaGhYnSQ
=+CpQ
-END PGP SIGNATURE-
From f69e20799be0829f9b710b593d33e10720e59096 Mon Sep 17 00:00:00 2001
From: Eric Blake e...@byu.net
Date: Thu, 30 Apr 2009 13:18:49 -0600
Subject: [PATCH] stdopen: remove unused code

* m4/stdopen.m4: Delete now-unused file.
* lib/stdopen.h: Likewise.
* lib/stdopen.c: Likewise.
---
 lib/stdopen.c |   74 -
 lib/stdopen.h |   16 
 m4/stdopen.m4 |   13 --
 3 files changed, 0 insertions(+), 103 deletions(-)
 delete mode 100644 lib/stdopen.c
 delete mode 100644 lib/stdopen.h
 delete mode 100644 m4/stdopen.m4

diff --git a/lib/stdopen.c b/lib/stdopen.c
deleted file mode 100644
index e9b5544..000
--- a/lib/stdopen.c
+++ /dev/null
@@ -1,74 +0,0 @@
-/* stdopen.c - ensure that the three standard file descriptors are in use
-
-   Copyright (C) 2005, 2006 Free Software Foundation, Inc.
-
-   This program is free software: you can redistribute it and/or modify
-   it under the terms of the GNU General Public License as published by
-   the Free Software Foundation, either version 3 of the License, or
-   (at your option) any later version.
-
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU General Public License for more details.
-
-   You should have received a copy of the GNU General Public License
-   along with this program.  If not, see http://www.gnu.org/licenses/.  */
-
-/* Written by Paul Eggert and Jim Meyering.  */
-
-#include config.h
-
-#include stdopen.h
-
-#include sys/types.h
-#include sys/stat.h
-#include fcntl.h
-#include unistd.h
-#include errno.h
-
-/* Try to ensure that all of the standard file numbers (0, 1, 2)
-   are in use.  Without this, each application would have to guard
-   every call to open, dup, fopen, etc. with tests to ensure they
-   don't use one of the special file numbers when opening a file.
-   Return false if at least one of the file descriptors is initially
-   closed and an attempt to reopen it fails.  Otherwise, return true.  */
-bool
-stdopen (void)
-{
-  int fd;
-  bool ok = true;
-
-  for (fd = 0; fd = 2; fd++)
-{
-  if (fcntl (fd, F_GETFD)  0)
-   {
- if (errno != EBADF)
-   ok = false;
- else
-   {
- static const int contrary_mode[]
-   = { O_WRONLY, O_RDONLY, O_RDONLY };
- int mode = contrary_mode[fd];
- int new_fd;
- /* Open /dev/null with the contrary mode so that the typical
-read (stdin) or write (stdout, stderr) operation will fail.
-With descriptor 0, we can do even better on systems that
-have /dev/full, by opening that write-only instead of
-/dev/null.  The only drawback is that a write-provoked
-failure comes with a misleading errno value, ENOSPC.  */
- if (mode == O_RDONLY
- || (new_fd = open (/dev/full, mode) != fd))
-   new_fd = open (/dev/null, mode);
- if (new_fd != fd)
-   {
- if (0 = new_fd)
-   close (new_fd);
- ok = false;
-   }
-   }
-   }
-}
-
-  return ok;
-}
diff --git a/lib/stdopen.h b/lib/stdopen.h
deleted file mode 100644
index d54e5f1..000
--- a/lib/stdopen.h
+++ /dev/null
@@ -1,16 +0,0 @@
-#ifndef STDOPEN_H
-# define STDOPEN_H 1
-
-# include stdbool.h
-
-# ifdef __cplusplus
-extern C {
-# endif
-
-bool stdopen (void);
-
-# ifdef __cplusplus
-}
-# endif
-
-#endif
diff --git a/m4/stdopen.m4 b/m4/stdopen.m4
deleted file mode 100644
index b18c22d..000
--- a/m4/stdopen.m4
+++ /dev/null
@@ -1,13 +0,0 @@
-#serial 1
-dnl Copyright (C) 2005 Free Software Foundation, Inc.
-dnl This file is free software; the Free Software Foundation
-dnl gives unlimited permission to copy and/or distribute it,
-dnl with or without modifications, 

Re: new snapshot available: coreutils-7.2.69-269b

2009-04-30 Thread Jim Meyering
Eric Blake wrote:
 According to Jim Meyering on 4/30/2009 10:14 AM:
 So far there have been only two problems (thanks for the testing
 and reports, Elbert Pol and James Youngman!) and both are fixed
 in this new snapshot.

 I'll release coreutils-7.3 tomorrow if this no-known-problems
 condition persists for a few hours longer.

 How about this quickie cleanup patch?  stdopen is unused (was it ever
 used? only lib/ChangeLog-2007 mentions it), and it seems that our current
 reliance on gnulib's stdio--.h and friends does the same job.

Yes, I know it's unused.
I've refrained from removing it because it
demonstrates a sometimes-useful alternate paradigm.
But removing it is the way to go.  This isn't the place for it.

Please go ahead and remove it.


___
Bug-coreutils mailing list
Bug-coreutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-coreutils