diff options
| author | Alan Modra <amodra@gmail.com> | 2015-01-11 08:30:33 +1030 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2015-01-12 18:07:52 +1030 |
| commit | b38ead219b31fed4014cd5b1078da968744b879d (patch) | |
| tree | aa980814e17268e12648817156ffbbd9017fe100 | |
| parent | 005e54bb792bc74f1168def16f35078be588871a (diff) | |
| download | gdb-b38ead219b31fed4014cd5b1078da968744b879d.zip gdb-b38ead219b31fed4014cd5b1078da968744b879d.tar.gz gdb-b38ead219b31fed4014cd5b1078da968744b879d.tar.bz2 | |
Assorted compiler warning fixes
The C standard doesn't guarantee a function pointer can be cast to
void* and vice versa.
binutils/
* prdbg.c (print_debugging_info): Don't use void* for function
pointer param.
* budbg.h (print_debugging_info): Update prototype.
gas/
* read.c (s_altmacro, s_reloc): Make definition static.
| -rw-r--r-- | binutils/ChangeLog | 6 | ||||
| -rw-r--r-- | binutils/budbg.h | 3 | ||||
| -rw-r--r-- | binutils/prdbg.c | 5 | ||||
| -rw-r--r-- | gas/ChangeLog | 4 | ||||
| -rw-r--r-- | gas/read.c | 4 |
5 files changed, 17 insertions, 5 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index e6fa3c1..9458a84 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2015-01-12 Alan Modra <amodra@gmail.com> + + * prdbg.c (print_debugging_info): Don't use void* for function + pointer param. + * budbg.h (print_debugging_info): Update prototype. + 2015-01-08 Nick Clifton <nickc@redhat.com> PR binutils/17512 diff --git a/binutils/budbg.h b/binutils/budbg.h index fc1ee2b..18c022d 100644 --- a/binutils/budbg.h +++ b/binutils/budbg.h @@ -29,7 +29,8 @@ extern void *read_debugging_info (bfd *, asymbol **, long, bfd_boolean); /* Routine used to print generic debugging information. */ extern bfd_boolean print_debugging_info - (FILE *, void *, bfd *, asymbol **, void *, bfd_boolean); + (FILE *, void *, bfd *, asymbol **, + char * (*) (struct bfd *, const char *, int), bfd_boolean); /* Routines used to read and write stabs information. */ diff --git a/binutils/prdbg.c b/binutils/prdbg.c index 4ef481c..6e2b217 100644 --- a/binutils/prdbg.c +++ b/binutils/prdbg.c @@ -291,7 +291,8 @@ static const struct debug_write_fns tg_fns = bfd_boolean print_debugging_info (FILE *f, void *dhandle, bfd *abfd, asymbol **syms, - void *demangler, bfd_boolean as_tags) + char * (*demangler) (struct bfd *, const char *, int), + bfd_boolean as_tags) { struct pr_handle info; @@ -302,7 +303,7 @@ print_debugging_info (FILE *f, void *dhandle, bfd *abfd, asymbol **syms, info.filename = NULL; info.abfd = abfd; info.syms = syms; - info.demangler = (char * (*)(struct bfd *, const char *, int)) demangler; + info.demangler = demangler; if (as_tags) { diff --git a/gas/ChangeLog b/gas/ChangeLog index 97422c1..a0deb6a 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,7 @@ +2015-01-12 Alan Modra <amodra@gmail.com> + + * read.c (s_altmacro, s_reloc): Make definition static. + 2015-01-10 Andrew Burgess <andrew.burgess@embecosm.com> * config/tc-avr.c (md_apply_fix): Update the contents of VALP for @@ -1579,7 +1579,7 @@ s_align_ptwo (int arg) /* Switch in and out of alternate macro mode. */ -void +static void s_altmacro (int on) { demand_empty_rest_of_line (); @@ -3984,7 +3984,7 @@ s_rva (int size) /* .reloc offset, reloc_name, symbol+addend. */ -void +static void s_reloc (int ignore ATTRIBUTE_UNUSED) { char *stop = NULL; |