gdb: use function_view for iterate_over_bp_locations' callback

Use a function_view instead of function pointer + data.  Actually,
nothing uses the data anyway, but that makes iterate_over_bp_locations
more like iterate_over_breakpoints.

gdb/ChangeLog:

	* breakpoint.c (iterate_over_bp_locations): Change callback to
	function view, remove data parameter.
	* breakpoint.h (iterate_over_bp_locations): Likewise.
	* record-full.c (record_full_sync_record_breakpoints): Remove
	data parameter.

Change-Id: I66cdc94a505f67bc640bcc66865fb535ee939a57

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 5d869dbfa7d4ed45e695a7a22a3d1b72e5964d5b..7e4b0bd95db8d60a037f092691bb10bd6d2913b0 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2021-04-22  Simon Marchi  <simon.marchi@polymtl.ca>
+
+       * breakpoint.c (iterate_over_bp_locations): Change callback to
+       function view, remove data parameter.
+       * breakpoint.h (iterate_over_bp_locations): Likewise.
+       * record-full.c (record_full_sync_record_breakpoints): Remove
+       data parameter.
+
 2021-04-22  Tom Tromey  <tom@tromey.com>
 
        * c-typeprint.c (c_type_print_base_struct_union): Use

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index c2d0ffba974d4783c11f03a5ca3a8eb1626c0f18..a20464a10623184d1eeea7e0afa88348f2efbe6c 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -2943,13 +2943,13 @@ insert_breakpoints (void)
 /* Invoke CALLBACK for each of bp_location.  */
 
 void
-iterate_over_bp_locations (walk_bp_location_callback callback)
+iterate_over_bp_locations (gdb::function_view<void (bp_location *)> callback)
 {
   struct bp_location *loc, **loc_tmp;
 
   ALL_BP_LOCATIONS (loc, loc_tmp)
     {
-      callback (loc, NULL);
+      callback (loc);
     }
 }
 

diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index ded498f55622626e4bbf75d1c507cd8cd31826b4..3447e25da205b93821308abae52b9898ace4a542 100644 (file)
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -1304,9 +1304,8 @@ extern void breakpoint_init_inferior (enum inf_context);
 
 extern void breakpoint_auto_delete (bpstat);
 
-typedef void (*walk_bp_location_callback) (struct bp_location *, void *);
-
-extern void iterate_over_bp_locations (walk_bp_location_callback);
+extern void iterate_over_bp_locations
+  (gdb::function_view<void (bp_location *)> callback);
 
 /* Return the chain of command lines to execute when this breakpoint
    is hit.  */

diff --git a/gdb/record-full.c b/gdb/record-full.c
index 23cbdcb58c40ab36cd4d2f77cdc60f6e8354ff1c..8a035b2663eab04926184fa744a07d952f1df147 100644 (file)
--- a/gdb/record-full.c
+++ b/gdb/record-full.c
@@ -1720,7 +1720,7 @@ struct record_full_breakpoint
 static std::vector<record_full_breakpoint> record_full_breakpoints;
 
 static void
-record_full_sync_record_breakpoints (struct bp_location *loc, void *data)
+record_full_sync_record_breakpoints (struct bp_location *loc)
 {
   if (loc->loc_type != bp_loc_software_breakpoint)
       return;