changeset 9691:ae73b5517c84

useless-if-before-free: Close stdout carefully.
author Jim Meyering <meyering@redhat.com>
date Mon, 11 Feb 2008 08:34:27 +0100
parents 521de8c5a745
children b25b75624f04
files ChangeLog build-aux/useless-if-before-free
diffstat 2 files changed, 13 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- 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'.
--- 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 @@
 
 (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) = @_;