From b8edc417a7143d28618086f1d6d946201001a075 Mon Sep 17 00:00:00 2001 From: Jan Kratochvil Date: Tue, 31 Aug 2010 18:08:43 +0000 Subject: [PATCH] gdb/ Code cleanup. * defs.h (find_memory_region_ftype): New typedef. (exec_set_find_memory_regions): Use it. * exec.c (exec_set_find_memory_regions): Use find_memory_region_ftype. * fbsd-nat.c (fbsd_find_memory_regions): Likewise. * gcore.c (objfile_find_memory_regions): Likewise. * gnu-nat.c (gnu_find_memory_regions): Likewise. * linux-nat.c (linux_nat_find_memory_regions): Likewise. * procfs.c (iterate_over_mappings_cb_ftype): Remove. (iterate_over_mappings): Rename iterate_over_mappings_cb_ftype to find_memory_region_ftype. (insert_dbx_link_bpt_in_region): Likewise. (iterate_over_mappings): Likewise. Drop the comment part about the function prototype. (find_memory_regions_callback): Use find_memory_region_ftype. (proc_find_memory_regions): Likewise. (info_mappings_callback): Rename iterate_over_mappings_cb_ftype to find_memory_region_ftype. * target.c (dummy_find_memory_regions): Use find_memory_region_ftype. * target.h (struct target_ops) : Likewise. --- gdb/ChangeLog | 23 +++++++++++++++++++++++ gdb/defs.h | 11 ++++++----- gdb/exec.c | 6 +----- gdb/fbsd-nat.c | 4 +--- gdb/gcore.c | 4 +--- gdb/gnu-nat.c | 6 +----- gdb/linux-nat.c | 4 +--- gdb/procfs.c | 43 +++++++++---------------------------------- gdb/target.c | 2 +- gdb/target.h | 6 +----- 10 files changed, 45 insertions(+), 64 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 71f84e9..d3030c2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,26 @@ +2010-08-31 Jan Kratochvil + + Code cleanup. + * defs.h (find_memory_region_ftype): New typedef. + (exec_set_find_memory_regions): Use it. + * exec.c (exec_set_find_memory_regions): Use find_memory_region_ftype. + * fbsd-nat.c (fbsd_find_memory_regions): Likewise. + * gcore.c (objfile_find_memory_regions): Likewise. + * gnu-nat.c (gnu_find_memory_regions): Likewise. + * linux-nat.c (linux_nat_find_memory_regions): Likewise. + * procfs.c (iterate_over_mappings_cb_ftype): Remove. + (iterate_over_mappings): Rename iterate_over_mappings_cb_ftype to + find_memory_region_ftype. + (insert_dbx_link_bpt_in_region): Likewise. + (iterate_over_mappings): Likewise. Drop the comment part about the + function prototype. + (find_memory_regions_callback): Use find_memory_region_ftype. + (proc_find_memory_regions): Likewise. + (info_mappings_callback): Rename iterate_over_mappings_cb_ftype to + find_memory_region_ftype. + * target.c (dummy_find_memory_regions): Use find_memory_region_ftype. + * target.h (struct target_ops) : Likewise. + 2010-08-31 Sami Wagiaalla * psymtab.c (add_psymbol_to_bcache): Remove 'static' from diff --git a/gdb/defs.h b/gdb/defs.h index c7a5215..30b5721 100644 --- a/gdb/defs.h +++ b/gdb/defs.h @@ -634,12 +634,13 @@ extern void init_source_path (void); /* From exec.c */ +typedef int (*find_memory_region_ftype) (CORE_ADDR addr, unsigned long size, + int read, int write, int exec, + void *data); + /* Take over the 'find_mapped_memory' vector from exec.c. */ -extern void exec_set_find_memory_regions (int (*) (int (*) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *)); +extern void exec_set_find_memory_regions + (int (*func) (find_memory_region_ftype func, void *data)); /* Possible lvalue types. Like enum language, this should be in value.h, but needs to be here for the same reason. */ diff --git a/gdb/exec.c b/gdb/exec.c index f9977a8..de0c459 100644 --- a/gdb/exec.c +++ b/gdb/exec.c @@ -835,11 +835,7 @@ exec_has_memory (struct target_ops *ops) /* Find mapped memory. */ extern void -exec_set_find_memory_regions (int (*func) (int (*) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *)) +exec_set_find_memory_regions (int (*func) (find_memory_region_ftype, void *)) { exec_ops.to_find_memory_regions = func; } diff --git a/gdb/fbsd-nat.c b/gdb/fbsd-nat.c index d499864..9e2f44d 100644 --- a/gdb/fbsd-nat.c +++ b/gdb/fbsd-nat.c @@ -92,9 +92,7 @@ fbsd_read_mapping (FILE *mapfile, unsigned long *start, unsigned long *end, argument to FUNC. */ int -fbsd_find_memory_regions (int (*func) (CORE_ADDR, unsigned long, - int, int, int, void *), - void *obfd) +fbsd_find_memory_regions (find_memory_region_ftype func, void *obfd) { pid_t pid = ptid_get_pid (inferior_ptid); char *mapfilename; diff --git a/gdb/gcore.c b/gdb/gcore.c index a38a5a3..595c643 100644 --- a/gdb/gcore.c +++ b/gdb/gcore.c @@ -462,9 +462,7 @@ gcore_create_callback (CORE_ADDR vaddr, unsigned long size, } static int -objfile_find_memory_regions (int (*func) (CORE_ADDR, unsigned long, - int, int, int, void *), - void *obfd) +objfile_find_memory_regions (find_memory_region_ftype func, void *obfd) { /* Use objfile data to create memory sections. */ struct objfile *objfile; diff --git a/gdb/gnu-nat.c b/gdb/gnu-nat.c index df84813..caeb604 100644 --- a/gdb/gnu-nat.c +++ b/gdb/gnu-nat.c @@ -2487,11 +2487,7 @@ gnu_xfer_memory (CORE_ADDR memaddr, gdb_byte *myaddr, int len, int write, /* Call FUNC on each memory region in the task. */ static int -gnu_find_memory_regions (int (*func) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *data) +gnu_find_memory_regions (find_memory_region_ftype func, void *data) { error_t err; task_t task; diff --git a/gdb/linux-nat.c b/gdb/linux-nat.c index 791908b..e50594d 100644 --- a/gdb/linux-nat.c +++ b/gdb/linux-nat.c @@ -4104,9 +4104,7 @@ read_mapping (FILE *mapfile, regions in the inferior for a corefile. */ static int -linux_nat_find_memory_regions (int (*func) (CORE_ADDR, - unsigned long, - int, int, int, void *), void *obfd) +linux_nat_find_memory_regions (find_memory_region_ftype func, void *obfd) { int pid = PIDGET (inferior_ptid); char mapsfilename[MAXPATHLEN]; diff --git a/gdb/procfs.c b/gdb/procfs.c index de8e26d..8802461 100644 --- a/gdb/procfs.c +++ b/gdb/procfs.c @@ -438,17 +438,9 @@ static void free_syscalls (procinfo *pi); static int find_syscall (procinfo *pi, char *name); #endif /* DYNAMIC_SYSCALLS */ -/* A function type used as a callback back iterate_over_mappings. */ -typedef int (iterate_over_mappings_cb_ftype) - (CORE_ADDR vaddr, unsigned long size, int read, int write, int execute, - void *data); - static int iterate_over_mappings - (procinfo *pi, - iterate_over_mappings_cb_ftype *child_func, - void *data, - int (*func) (struct prmap *map, - iterate_over_mappings_cb_ftype *child_func, + (procinfo *pi, find_memory_region_ftype child_func, void *data, + int (*func) (struct prmap *map, find_memory_region_ftype child_func, void *data)); /* The head of the procinfo list: */ @@ -3784,7 +3776,7 @@ solib_mappings_callback (struct prmap *map, int (*func) (int, CORE_ADDR), static int insert_dbx_link_bpt_in_region (struct prmap *map, - iterate_over_mappings_cb_ftype *child_func, + find_memory_region_ftype child_func, void *data) { procinfo *pi = (procinfo *) data; @@ -5226,11 +5218,10 @@ procfs_use_watchpoints (struct target_ops *t) from the callback function, or zero. */ static int -iterate_over_mappings (procinfo *pi, - iterate_over_mappings_cb_ftype *child_func, +iterate_over_mappings (procinfo *pi, find_memory_region_ftype child_func, void *data, int (*func) (struct prmap *map, - iterate_over_mappings_cb_ftype *child_func, + find_memory_region_ftype child_func, void *data)) { char pathname[MAX_PROC_NAME_SIZE]; @@ -5282,23 +5273,12 @@ iterate_over_mappings (procinfo *pi, } /* Implements the to_find_memory_regions method. Calls an external - function for each memory region. The external function will have - the signature: - - int callback (CORE_ADDR vaddr, - unsigned long size, - int read, int write, int execute, - void *data); - + function for each memory region. Returns the integer value returned by the callback. */ static int find_memory_regions_callback (struct prmap *map, - int (*func) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *data) + find_memory_region_ftype func, void *data) { return (*func) ((CORE_ADDR) map->pr_vaddr, map->pr_size, @@ -5321,11 +5301,7 @@ find_memory_regions_callback (struct prmap *map, the callback. */ static int -proc_find_memory_regions (int (*func) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *data) +proc_find_memory_regions (find_memory_region_ftype func, void *data) { procinfo *pi = find_procinfo_or_die (PIDGET (inferior_ptid), 0); @@ -5364,8 +5340,7 @@ mappingflags (long flags) mappings'. */ static int -info_mappings_callback (struct prmap *map, - iterate_over_mappings_cb_ftype *ignore, +info_mappings_callback (struct prmap *map, find_memory_region_ftype ignore, void *unused) { unsigned int pr_off; diff --git a/gdb/target.c b/gdb/target.c index 48c4a2c..5984125 100644 --- a/gdb/target.c +++ b/gdb/target.c @@ -3005,7 +3005,7 @@ dummy_pid_to_str (struct target_ops *ops, ptid_t ptid) /* Error-catcher for target_find_memory_regions. */ static int -dummy_find_memory_regions (int (*ignore1) (), void *ignore2) +dummy_find_memory_regions (find_memory_region_ftype ignore1, void *ignore2) { error (_("Command not implemented for this target.")); return 0; diff --git a/gdb/target.h b/gdb/target.h index 7cedf8f..7290d90 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -504,11 +504,7 @@ struct target_ops int (*to_async_mask) (int); int (*to_supports_non_stop) (void); /* find_memory_regions support method for gcore */ - int (*to_find_memory_regions) (int (*) (CORE_ADDR, - unsigned long, - int, int, int, - void *), - void *); + int (*to_find_memory_regions) (find_memory_region_ftype func, void *data); /* make_corefile_notes support method for gcore */ char * (*to_make_corefile_notes) (bfd *, int *); /* get_bookmark support method for bookmarks */ -- 2.7.4