diff options
author | Joel Brobecker <brobecker@gnat.com> | 2013-05-09 06:58:16 +0000 |
---|---|---|
committer | Joel Brobecker <brobecker@gnat.com> | 2013-05-09 06:58:16 +0000 |
commit | 3b613351e514faed72ecf9edad9eaad3db8efa58 (patch) | |
tree | 48f027710140a3e9afac2580cef9bd3e268a6200 /binutils/ChangeLog | |
parent | 8220fd513bc5ab80a27510c760a5658cb03701fd (diff) | |
download | binutils-redhat-3b613351e514faed72ecf9edad9eaad3db8efa58.tar.gz |
Use fputc in place of putc to avoid -Wunused-value warning (AIX).
Currently, bfd does not compile with -Wunused-value because
the following code:
val = putc ('\n', f);
gets expanded into some code that triggers a warning:
warning: value computed is not used [-Wunused-value]
This is because putc is implemented as a macro...
>#define putc(__x, __p) (((!((__p)->_flag & 0xC000)) && \
> ((__p)->_flag = ((__p)->_flag & 0x3FFF) | 0x8000)),\
> (--(__p)->_cnt < 0 ? \
> __flsbuf((unsigned char) (__x), (__p)) : \
> (int) (*(__p)->_ptr++ = (unsigned char) (__x))))
It's the first part, before the coma operator, which triggers
the unused-value warning.
This patch fixes the issue by simply avoiding the macro and using
fputc instead.
bfd/ChangeLog:
* bfd.c (_bfd_default_error_handler): Replace use of putc
by fputc. Add comment explaining why.
Diffstat (limited to 'binutils/ChangeLog')
0 files changed, 0 insertions, 0 deletions