From 4495129abde9922d77215233837fb47738722de9 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Tue, 12 Sep 2017 21:25:07 -0600 Subject: [PATCH] Constify some commands in breakpoint.c This also makes delete_command static; but now I wonder if it is used in Insight and should not be touched. gdb/ChangeLog 2017-09-27 Tom Tromey * breakpoint.h (delete_command): Don't declare. * breakpoint.c (delete_command, enable_once_command) (enable_count_command, enable_delete_command, breakpoint_1) (maintenance_info_breakpoints, stopin_command, stopat_command) (delete_command, delete_trace_command, save_breakpoints) (save_breakpoints_command, save_tracepoints_command): Constify. --- gdb/ChangeLog | 9 +++++++++ gdb/breakpoint.c | 45 ++++++++++++++------------------------------- gdb/breakpoint.h | 4 ---- 3 files changed, 23 insertions(+), 35 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f9c97e5db4f..1a83a062ad5 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,12 @@ +2017-09-27 Tom Tromey + + * breakpoint.h (delete_command): Don't declare. + * breakpoint.c (delete_command, enable_once_command) + (enable_count_command, enable_delete_command, breakpoint_1) + (maintenance_info_breakpoints, stopin_command, stopat_command) + (delete_command, delete_trace_command, save_breakpoints) + (save_breakpoints_command, save_tracepoints_command): Constify. + 2017-09-27 Tom Tromey * macrocmd.c (macro_expand_command, macro_expand_once_command) diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 1073418e775..59cb3547cfa 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -94,12 +94,6 @@ enum exception_event_kind /* Prototypes for local functions. */ -static void enable_delete_command (char *, int); - -static void enable_once_command (char *, int); - -static void enable_count_command (char *, int); - static void disable_command (char *, int); static void enable_command (char *, int); @@ -184,9 +178,6 @@ static void info_breakpoints_command (char *, int); static void info_watchpoints_command (char *, int); -static int breakpoint_1 (char *, int, - int (*) (const struct breakpoint *)); - static int breakpoint_cond_eval (void *); static void cleanup_executing_breakpoints (void *); @@ -202,8 +193,6 @@ static enum print_stop_action print_bp_stop_message (bpstat bs); static int watchpoint_check (void *); -static void maintenance_info_breakpoints (char *, int); - static int hw_breakpoint_used_count (void); static int hw_watchpoint_use_count (struct breakpoint *); @@ -221,10 +210,6 @@ static void enable_breakpoint_disp (struct breakpoint *, enum bpdisp, static void stop_command (char *arg, int from_tty); -static void stopin_command (char *arg, int from_tty); - -static void stopat_command (char *arg, int from_tty); - static void tcatch_command (char *arg, int from_tty); static void free_bp_location (struct bp_location *loc); @@ -278,8 +263,6 @@ static void insert_breakpoint_locations (void); static void info_tracepoints_command (char *, int); -static void delete_trace_command (char *, int); - static void enable_trace_command (char *, int); static void disable_trace_command (char *, int); @@ -6679,7 +6662,7 @@ pending_breakpoint_p (struct breakpoint *b) breakpoints listed. */ static int -breakpoint_1 (char *args, int allflag, +breakpoint_1 (const char *args, int allflag, int (*filter) (const struct breakpoint *)) { struct breakpoint *b; @@ -6864,7 +6847,7 @@ info_watchpoints_command (char *args, int from_tty) } static void -maintenance_info_breakpoints (char *args, int from_tty) +maintenance_info_breakpoints (const char *args, int from_tty) { breakpoint_1 (args, 1, NULL); @@ -9795,7 +9778,7 @@ Usage: stop in \n\ } static void -stopin_command (char *arg, int from_tty) +stopin_command (const char *arg, int from_tty) { int badInput = 0; @@ -9803,7 +9786,7 @@ stopin_command (char *arg, int from_tty) badInput = 1; else if (*arg != '*') { - char *argptr = arg; + const char *argptr = arg; int hasColon = 0; /* Look for a ':'. If this is a line number specification, then @@ -9828,7 +9811,7 @@ stopin_command (char *arg, int from_tty) } static void -stopat_command (char *arg, int from_tty) +stopat_command (const char *arg, int from_tty) { int badInput = 0; @@ -9836,7 +9819,7 @@ stopat_command (char *arg, int from_tty) badInput = 1; else { - char *argptr = arg; + const char *argptr = arg; int hasColon = 0; /* Look for a ':'. If there is a '::' then get out, otherwise @@ -13568,7 +13551,7 @@ iterate_over_related_breakpoints (struct breakpoint *b, while (related != b); } -void +static void delete_command (char *arg, int from_tty) { struct breakpoint *b, *b_tmp; @@ -14596,7 +14579,7 @@ enable_command (char *args, int from_tty) } static void -enable_once_command (char *args, int from_tty) +enable_once_command (const char *args, int from_tty) { map_breakpoint_numbers (args, [&] (breakpoint *b) @@ -14610,7 +14593,7 @@ enable_once_command (char *args, int from_tty) } static void -enable_count_command (char *args, int from_tty) +enable_count_command (const char *args, int from_tty) { int count; @@ -14631,7 +14614,7 @@ enable_count_command (char *args, int from_tty) } static void -enable_delete_command (char *args, int from_tty) +enable_delete_command (const char *args, int from_tty) { map_breakpoint_numbers (args, [&] (breakpoint *b) @@ -15014,7 +14997,7 @@ disable_trace_command (char *args, int from_tty) /* Remove a tracepoint (or all if no argument). */ static void -delete_trace_command (char *arg, int from_tty) +delete_trace_command (const char *arg, int from_tty) { struct breakpoint *b, *b_tmp; @@ -15208,7 +15191,7 @@ print_recreate_thread (struct breakpoint *b, struct ui_file *fp) non-zero. */ static void -save_breakpoints (char *filename, int from_tty, +save_breakpoints (const char *filename, int from_tty, int (*filter) (const struct breakpoint *)) { struct breakpoint *tp; @@ -15326,7 +15309,7 @@ save_breakpoints (char *filename, int from_tty, /* The `save breakpoints' command. */ static void -save_breakpoints_command (char *args, int from_tty) +save_breakpoints_command (const char *args, int from_tty) { save_breakpoints (args, from_tty, NULL); } @@ -15334,7 +15317,7 @@ save_breakpoints_command (char *args, int from_tty) /* The `save tracepoints' command. */ static void -save_tracepoints_command (char *args, int from_tty) +save_tracepoints_command (const char *args, int from_tty) { save_breakpoints (args, from_tty, is_tracepoint); } diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index 154ff3aaff9..ff49cd2ab4b 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -1521,10 +1521,6 @@ extern int is_catchpoint (struct breakpoint *); extern void add_solib_catchpoint (const char *arg, int is_load, int is_temp, int enabled); -/* Enable breakpoints and delete when hit. Called with ARG == NULL - deletes all breakpoints. */ -extern void delete_command (char *arg, int from_tty); - /* Create and insert a new software single step breakpoint for the current thread. May be called multiple times; each time will add a new location to the set of potential addresses the next instruction -- 2.30.2