Re: [PATCH] fclose: guarantee behavior on seekable stdin

[Bruno Haible]

Eric Blake wrote:
> * modules/fclose (Depends-on): Add fflush.

Now that 'fclose' depends on 'fflush', an m4_ifdef is not necessary any more:


2011-05-07  Bruno Haible  <address@hidden>

        fclose: Simplify autoconf macro.
        * m4/fclose.m4 (gl_FUNC_FCLOSE): Assume gl_FUNC_FFLUSH_STDIN is
        defined.

--- m4/fclose.m4.orig   Sat May  7 13:04:33 2011
+++ m4/fclose.m4        Sat May  7 13:03:51 2011
@@ -1,4 +1,4 @@
-# fclose.m4 serial 3
+# fclose.m4 serial 4
 dnl Copyright (C) 2008-2011 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -6,12 +6,10 @@
 
 AC_DEFUN([gl_FUNC_FCLOSE],
 [
-  m4_ifdef([gl_FUNC_FFLUSH_STDIN], [
-    gl_FUNC_FFLUSH_STDIN
-    if test $gl_cv_func_fflush_stdin = no; then
-      gl_REPLACE_FCLOSE
-    fi
-  ])
+  gl_FUNC_FFLUSH_STDIN
+  if test $gl_cv_func_fflush_stdin = no; then
+    gl_REPLACE_FCLOSE
+  fi
 ])
 
 AC_DEFUN([gl_REPLACE_FCLOSE],
-- 
In memoriam The victims of the Skelani massacre 
<http://books.google.de/books?id=nltdtAo38K0C&pg=PA41&lpg=PA41&source=bl&ots=DX2VTN1nga&sig=d_BXUZM0I81hZtfrSBA3392p-KY&hl=de&ei=JHy6TcGTCYmLswbpsrDzBQ&sa=X&oi=book_result&ct=result&resnum=1&ved=0CBkQ6AEwAA#v=onepage&q&f=false>