diff options
| author | Alan Modra <amodra@gmail.com> | 2019-09-06 12:07:05 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2019-09-06 12:22:59 +0930 |
| commit | b16c44debcf8a446e94e161e328169f5c8d5d811 (patch) | |
| tree | f402c08e8f7fc00a404055e36c416580377f1b3a /binutils/ChangeLog | |
| parent | 6ade8373c9e689dece1057366b5adf9f8e86fa22 (diff) | |
| download | binutils-gdb-b16c44debcf8a446e94e161e328169f5c8d5d811.tar.gz | |
bfd_get_filename
This macro says:
/* Cast from const char * to char * so that caller can assign to
a char * without a warning. */
I reckon that isn't such a good idea since it can result in char*
variables where const char* is appropriate. Not very many places
need the char* cast.
bfd/
* aout-target.h (object_p): Formatting.
* bfd-in.h (bfd_get_filename): Don't cast to char*.
* corefile.c (generic_core_file_matches_executable_p): Constify
variables and remove cast.
* bfd-in2.h: Regenerate.
binutils/
* nm.c (print_object_filename_bsd, print_object_filename_sysv),
(print_object_filename_posix, print_archive_filename_bsd),
(print_archive_filename_sysv, print_archive_filename_posix),
(print_archive_member_bsd, print_archive_member_sysv),
(print_archive_member_posix): Constify parameter.
(struct output_fns <print_object_filename, print_archive_filename>),
(<print_archive_member>): Likewise.
* objcopy.c (copy_archive): Add cast for make_tempdir.
ld/
* emultempl/elf32.em (gld${EMULATION_NAME}_search_needed): Constify
variable.
* emultempl/pe.em (gld_${EMULATION_NAME}_after_open): Likewise.
* emultempl/pep.em (gld_${EMULATION_NAME}_after_open): Likewise.
gdb/
* coffread.c (coff_symfile_read): Constify filename variable.
* dbxread.c (dbx_symfile_init, coffstab_build_psymtabs),
(elfstab_build_psymtabs, stabsect_build_psymtabs): Likewise.
* gdb_bfd.c (gdb_bfd_close_or_warn): Likewise.
* solib.c (reload_shared_libraries_1): Likewise.
* symfile.c (reread_symbols): Likewise.
* solib-aix.c (solib_aix_bfd_open): Add cast for xfree of filename.
* solib-darwin.c (darwin_bfd_open): Likewise.
* symfile-mem.c (symbol_file_add_from_memory): Likewise.
sim/cris/
* sim-if.c (sim_open): Constify filename variable.
Diffstat (limited to 'binutils/ChangeLog')
| -rw-r--r-- | binutils/ChangeLog | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 98e660d167d..198380eaeab 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,14 @@ +2019-09-06 Alan Modra <amodra@gmail.com> + + * nm.c (print_object_filename_bsd, print_object_filename_sysv), + (print_object_filename_posix, print_archive_filename_bsd), + (print_archive_filename_sysv, print_archive_filename_posix), + (print_archive_member_bsd, print_archive_member_sysv), + (print_archive_member_posix): Constify parameter. + (struct output_fns <print_object_filename, print_archive_filename>), + (<print_archive_member>): Likewise. + * objcopy.c (copy_archive): Add cast for make_tempdir. + 2019-08-29 Alan Modra <amodra@gmail.com> * dwarf.c (check_uvalue): Remove unnecessary pointer checks. |