useless-if-before-free: Close stdout carefully.

diff --git a/ChangeLog b/ChangeLog
index 236260db8a91cb31df0bdc1f73ee909673fecd2a..b6209a4ab07eff08b4e513fb7de8f4999b763d03 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2008-02-11  Jim Meyering  <meyering@redhat.com>
+
+       * build-aux/useless-if-before-free: Close stdout carefully.
+
 2008-02-10  Bruno Haible  <bruno@clisp.org>
 
        New module 'git-merge-changelog'.

diff --git a/build-aux/useless-if-before-free b/build-aux/useless-if-before-free
index 1cf4cb03f643859ab6f18274185b91abb47c788a..a50db667b06369625b1a8b0b486ec3fae5ab2956 100755 (executable)
--- a/build-aux/useless-if-before-free
+++ b/build-aux/useless-if-before-free
@@ -2,7 +2,7 @@
 # Detect instances of "if (p) free (p);".
 # Likewise for "if (p != NULL) free (p);".  And with braces.
 
-my $VERSION = '2008-02-10 22:17'; # UTC
+my $VERSION = '2008-02-11 07:32'; # UTC
 # The definition above must lie within the first 8 lines in order
 # for the Emacs time-stamp write hook (at end) to update it.
 # If you change this file with Emacs, please let the write hook
@@ -31,6 +31,14 @@ use Getopt::Long;
 
 (my $ME = $0) =~ s|.*/||;
 
+# use File::Coda; # http://meyering.net/code/Coda/
+END {
+  defined fileno STDOUT or return;
+  close STDOUT and return;
+  warn "$ME: failed to close standard output: $!\n";
+  $? ||= 1;
+}
+
 sub usage ($)
 {
   my ($exit_code) = @_;