# HG changeset patch
# User Jim Meyering <meyering@redhat.com>
# Date 1256721849 -3600
# Node ID 5929f9acfa9431abf7ad02e1642df97a61f97fab
# Parent  e298b64a31cf98413bd222e02e60e4fc3c4c656d
fprintftime: avoid a warning about ignored fwrite return value

* lib/strftime.c [FPRINTFTIME]: Include "ignore-value.h".
(cpy) [FPRINTFTIME]: Ignore fwrite failure, even though technically,
that is unsafe.
* modules/fprintftime (Depends-on): Add ignore-value.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2009-10-28  Jim Meyering  <meyering@redhat.com>
 
+	fprintftime: avoid a warning about ignored fwrite return value
+	* lib/strftime.c [FPRINTFTIME]: Include "ignore-value.h".
+	(cpy) [FPRINTFTIME]: Ignore fwrite failure, even though technically,
+	that is unsafe.
+	* modules/fprintftime (Depends-on): Add ignore-value.
+
 	exclude: avoid an unwarranted warning
 	* lib/exclude.c (excluded_file_name): Initialize "rc" before switch.
 
diff --git a/lib/strftime.c b/lib/strftime.c
--- a/lib/strftime.c
+++ b/lib/strftime.c
@@ -31,6 +31,7 @@
 # else
 #  include "strftime.h"
 # endif
+# include "ignore-value.h"
 #endif
 
 #include <ctype.h>
@@ -203,7 +204,15 @@
 	 else if (to_uppcase)						      \
 	   fwrite_uppcase (p, (s), _n);					      \
 	 else								      \
-	   fwrite ((s), _n, 1, p))
+	   {								      \
+	     /* We are ignoring the value of fwrite here, in spite of the     \
+		fact that technically, that may not be valid: the fwrite      \
+		specification in POSIX 2008 defers to that of fputc, which    \
+		is intended to be consistent with the one from ISO C,         \
+		which permits failure due to ENOMEM *without* setting the     \
+		stream's error indicator.  */                                 \
+	     ignore_value (fwrite ((s), _n, 1, p)));			      \
+	   }
 #else
 # define cpy(n, s)							      \
     add ((n),								      \
diff --git a/modules/fprintftime b/modules/fprintftime
--- a/modules/fprintftime
+++ b/modules/fprintftime
@@ -7,6 +7,7 @@
 m4/fprintftime.m4
 
 Depends-on:
+ignore-value
 strftime
 
 configure.ac: