diff options
author | Tom Tromey <tromey@redhat.com> | 2014-07-22 10:47:53 -0600 |
---|---|---|
committer | Tom Tromey <tromey@redhat.com> | 2014-07-30 08:02:52 -0600 |
commit | e799154c3bf1aac0bffd869df5eed7a959305d00 (patch) | |
tree | 5f73896f9f771cfa01e7b2bfddcf234632ed6bf4 /gdb | |
parent | 5f08566b92f63e60559577dedfabf9ed517a3ddf (diff) | |
download | gdb-e799154c3bf1aac0bffd869df5eed7a959305d00.zip gdb-e799154c3bf1aac0bffd869df5eed7a959305d00.tar.gz gdb-e799154c3bf1aac0bffd869df5eed7a959305d00.tar.bz2 |
constify some cli-utils stuff
This constifies a few functions in cli-utils -- get_number_trailer and
friends -- and then fixes the fallout.
2014-07-30 Tom Tromey <tromey@redhat.com>
* breakpoint.c (map_breakpoint_numbers): Update.
* cli/cli-utils.c (get_number_trailer): Make "pp" const. Update.
(get_number_const): New function.
(get_number): Rewrite using get_number_const.
(init_number_or_range): Make "string" const.
(number_is_in_list): Make "list" const.
* cli/cli-utils.h (get_number_const): Declare.
(struct get_number_or_range_state) <string, end_ptr>: Now const.
(init_number_or_range, number_is_in_list): Update.
* printcmd.c (map_display_numbers): Update.
* value.c (value_from_history_ref): Constify.
* value.h (value_from_history_ref): Update.
Diffstat (limited to 'gdb')
-rw-r--r-- | gdb/ChangeLog | 15 | ||||
-rw-r--r-- | gdb/breakpoint.c | 2 | ||||
-rw-r--r-- | gdb/cli/cli-utils.c | 33 | ||||
-rw-r--r-- | gdb/cli/cli-utils.h | 12 | ||||
-rw-r--r-- | gdb/printcmd.c | 2 | ||||
-rw-r--r-- | gdb/value.c | 16 | ||||
-rw-r--r-- | gdb/value.h | 2 |
7 files changed, 62 insertions, 20 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 14aaa2c..24d8673 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,20 @@ 2014-07-30 Tom Tromey <tromey@redhat.com> + * breakpoint.c (map_breakpoint_numbers): Update. + * cli/cli-utils.c (get_number_trailer): Make "pp" const. Update. + (get_number_const): New function. + (get_number): Rewrite using get_number_const. + (init_number_or_range): Make "string" const. + (number_is_in_list): Make "list" const. + * cli/cli-utils.h (get_number_const): Declare. + (struct get_number_or_range_state) <string, end_ptr>: Now const. + (init_number_or_range, number_is_in_list): Update. + * printcmd.c (map_display_numbers): Update. + * value.c (value_from_history_ref): Constify. + * value.h (value_from_history_ref): Update. + +2014-07-30 Tom Tromey <tromey@redhat.com> + * corefile.c (hook_type, call_extra_exec_file_hooks) (specify_exec_file_hook): Constify. * exec.c (exec_file_attach): Make "filename" const. diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 262e992..8cd7660 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -14788,7 +14788,7 @@ map_breakpoint_numbers (char *args, void (*function) (struct breakpoint *, while (!state.finished) { - char *p = state.string; + const char *p = state.string; match = 0; diff --git a/gdb/cli/cli-utils.c b/gdb/cli/cli-utils.c index a0ebc11..819c94c 100644 --- a/gdb/cli/cli-utils.c +++ b/gdb/cli/cli-utils.c @@ -35,10 +35,10 @@ commonly this is `-'. If you don't want a trailer, use \0. */ static int -get_number_trailer (char **pp, int trailer) +get_number_trailer (const char **pp, int trailer) { int retval = 0; /* default */ - char *p = *pp; + const char *p = *pp; if (*p == '$') { @@ -59,7 +59,7 @@ get_number_trailer (char **pp, int trailer) /* Internal variable. Make a copy of the name, so we can null-terminate it to pass to lookup_internalvar(). */ char *varname; - char *start = ++p; + const char *start = ++p; LONGEST val; while (isalnum (*p) || *p == '_') @@ -102,7 +102,7 @@ get_number_trailer (char **pp, int trailer) ++p; retval = 0; } - p = skip_spaces (p); + p = skip_spaces_const (p); *pp = p; return retval; } @@ -110,16 +110,29 @@ get_number_trailer (char **pp, int trailer) /* See documentation in cli-utils.h. */ int -get_number (char **pp) +get_number_const (const char **pp) { return get_number_trailer (pp, '\0'); } /* See documentation in cli-utils.h. */ +int +get_number (char **pp) +{ + int result; + const char *p = *pp; + + result = get_number_trailer (&p, '\0'); + *pp = (char *) p; + return result; +} + +/* See documentation in cli-utils.h. */ + void init_number_or_range (struct get_number_or_range_state *state, - char *string) + const char *string) { memset (state, 0, sizeof (*state)); state->string = string; @@ -137,15 +150,15 @@ get_number_or_range (struct get_number_or_range_state *state) state->last_retval = get_number_trailer (&state->string, '-'); if (*state->string == '-') { - char **temp; + const char **temp; /* This is the start of a range (<number1> - <number2>). Skip the '-', parse and remember the second number, and also remember the end of the final token. */ temp = &state->end_ptr; - state->end_ptr = skip_spaces (state->string + 1); - state->end_value = get_number (temp); + state->end_ptr = skip_spaces_const (state->string + 1); + state->end_value = get_number_const (temp); if (state->end_value < state->last_retval) { error (_("inverted range")); @@ -191,7 +204,7 @@ get_number_or_range (struct get_number_or_range_state *state) no arguments. */ int -number_is_in_list (char *list, int number) +number_is_in_list (const char *list, int number) { struct get_number_or_range_state state; diff --git a/gdb/cli/cli-utils.h b/gdb/cli/cli-utils.h index fed876b..1f0f3df 100644 --- a/gdb/cli/cli-utils.h +++ b/gdb/cli/cli-utils.h @@ -26,6 +26,10 @@ Currently the string can either be a number, or "$" followed by the name of a convenience variable, or ("$" or "$$") followed by digits. */ +extern int get_number_const (const char **); + +/* Like get_number_const, but takes a non-const "char **". */ + extern int get_number (char **); /* An object of this type is passed to get_number_or_range. It must @@ -40,7 +44,7 @@ struct get_number_or_range_state /* The string being parsed. When parsing has finished, this points past the last parsed token. */ - char *string; + const char *string; /* Last value returned. */ int last_retval; @@ -50,7 +54,7 @@ struct get_number_or_range_state /* When parsing a range, a pointer past the final token in the range. */ - char *end_ptr; + const char *end_ptr; /* Non-zero when parsing a range. */ int in_range; @@ -60,7 +64,7 @@ struct get_number_or_range_state get_number_or_range_state. STRING is the string to be parsed. */ extern void init_number_or_range (struct get_number_or_range_state *state, - char *string); + const char *string); /* Parse a number or a range. A number will be of the form handled by get_number. @@ -87,7 +91,7 @@ extern int get_number_or_range (struct get_number_or_range_state *state); be interpreted as typing a command such as "delete break" with no arguments. */ -extern int number_is_in_list (char *list, int number); +extern int number_is_in_list (const char *list, int number); /* Skip leading whitespace characters in INP, returning an updated pointer. If INP is NULL, return NULL. */ diff --git a/gdb/printcmd.c b/gdb/printcmd.c index 228d4ad..b4fd894 100644 --- a/gdb/printcmd.c +++ b/gdb/printcmd.c @@ -1618,7 +1618,7 @@ map_display_numbers (char *args, while (!state.finished) { - char *p = state.string; + const char *p = state.string; num = get_number_or_range (&state); if (num == 0) diff --git a/gdb/value.c b/gdb/value.c index 41ca250..19e391c 100644 --- a/gdb/value.c +++ b/gdb/value.c @@ -3449,7 +3449,7 @@ value_from_decfloat (struct type *type, const gdb_byte *dec) for details. */ struct value * -value_from_history_ref (char *h, char **endp) +value_from_history_ref (const char *h, const char **endp) { int index, len; @@ -3480,7 +3480,12 @@ value_from_history_ref (char *h, char **endp) *endp += len; } else - index = -strtol (&h[2], endp, 10); + { + char *local_end; + + index = -strtol (&h[2], &local_end, 10); + *endp = local_end; + } } else { @@ -3491,7 +3496,12 @@ value_from_history_ref (char *h, char **endp) *endp += len; } else - index = strtol (&h[1], endp, 10); + { + char *local_end; + + index = strtol (&h[1], &local_end, 10); + *endp = local_end; + } } return access_value_history (index); diff --git a/gdb/value.h b/gdb/value.h index 86ebd70..4654042 100644 --- a/gdb/value.h +++ b/gdb/value.h @@ -569,7 +569,7 @@ extern struct value *value_from_pointer (struct type *type, CORE_ADDR addr); extern struct value *value_from_double (struct type *type, DOUBLEST num); extern struct value *value_from_decfloat (struct type *type, const gdb_byte *decbytes); -extern struct value *value_from_history_ref (char *, char **); +extern struct value *value_from_history_ref (const char *, const char **); extern struct value *value_at (struct type *type, CORE_ADDR addr); extern struct value *value_at_lazy (struct type *type, CORE_ADDR addr); |