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) <to_find_memory_regions>: Likewise.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 71f84e9c43b6de1d3c07424b605f89ce1b943555..d3030c28f480a10ebb46becf89ca6b4a0817040c 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,26 @@
+2010-08-31  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       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) <to_find_memory_regions>: Likewise.
+
 2010-08-31  Sami Wagiaalla  <swagiaal@redhat.com>
 
        * psymtab.c (add_psymbol_to_bcache): Remove 'static' from

diff --git a/gdb/defs.h b/gdb/defs.h
index c7a5215e2faf6f499507277c483f7abc92ddf6cc..30b572194fb44c29c0f78fede9d0565b112e8552 100644 (file)
--- 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 f9977a8ed60beb7362c5b66e522e4b8f6a3bd9c4..de0c459e85e2e1a2129329d36882158e4d4b455c 100644 (file)
--- 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 d499864904eef673654652e3263cb595ec1d8cbe..9e2f44d6612af7f51229d0aff8cf68bae2f8f20f 100644 (file)
--- 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 a38a5a357c804fb01e1551688989749f5b7bfb07..595c6431dd56abb4347a72e69412265771452a96 100644 (file)
--- 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 df84813c9011d21cc45c1167b288786a624db62e..caeb6044bdf57290f5681806b9d394be4e81af44 100644 (file)
--- 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 791908b8f23cfe99fe3c475664721cb9c6c47395..e50594d125cefa963b77ef885bd2054715305328 100644 (file)
--- 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 de8e26dd597dda76136b98fa1631f4293a756248..8802461e7cfb2a39b23b3dbd11b5e28659b4e04a 100644 (file)
--- 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 48c4a2cf26546dfc2441853f1f33fc375fff4316..59841258554a0a39b800bd2f829de7c777c759c1 100644 (file)
--- 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 7cedf8fc20b347e3ca71d5f9bb12d0b1fd7da9ba..7290d90151ce175909af41b04f6c1cd70850ab5f 100644 (file)
--- 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 */