Mercurial > hg > octave-lojdl > gnulib-hg
changeset 17266:b039ff4451d0
fprintftime: depend on stdio, not ignore-value
* lib/strftime.c [FPRINTFTIME]: Do not include ignore-value.h.
(cpy) [FPRINTFTIME]: Use plain fwrite, not ignore_value of fwrite,
since the stdio module arranges to silence that warning now.
* modules/fprintftime (Depends-on): Depend on stdio, not ignore-value.
| author | Paul Eggert <eggert@cs.ucla.edu> |
|---|---|
| date | Fri, 04 Jan 2013 01:55:18 +0000 |
| parents | b3f19d3d0811 |
| children | ee4cb5937493 |
| files | ChangeLog lib/strftime.c modules/fprintftime |
| diffstat | 3 files changed, 10 insertions(+), 11 deletions(-) [+] |
line wrap: on
line diff
--- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2013-01-04 Paul Eggert <eggert@cs.ucla.edu> + + fprintftime: depend on stdio, not ignore-value + * lib/strftime.c [FPRINTFTIME]: Do not include ignore-value.h. + (cpy) [FPRINTFTIME]: Use plain fwrite, not ignore_value of fwrite, + since the stdio module arranges to silence that warning now. + * modules/fprintftime (Depends-on): Depend on stdio, not ignore-value. + 2012-10-04 Simon Josefsson <simon@josefsson.org> stdint-tests: Fix expanded-before-required-warning.
--- a/lib/strftime.c +++ b/lib/strftime.c @@ -26,7 +26,6 @@ #else # include <config.h> # if FPRINTFTIME -# include "ignore-value.h" # include "fprintftime.h" # else # include "strftime.h" @@ -209,15 +208,7 @@ else if (to_uppcase) \ fwrite_uppcase (p, (s), _n); \ else \ - { \ - /* 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)); \ - } \ + fwrite (s, _n, 1, p); \ } \ while (0) \ )