[2/5] Change argument 'args' of get_tracepoint_by_number to char *
Commit Message
We pass the reference to a char pointer to get_tracepoint_by_number,
but the char pointer isn't used afterwards, so looks it should be
fine to pass a char pointer to get_tracepoint_by_number.
gdb:
2014-03-13 Yao Qi <yao@codesourcery.com>
* breakpoint.c (get_tracepoint_by_number): Change type of
'args' to 'char *'. Update pointer checking. All callers
updated.
* breakpoint.h (get_tracepoint_by_number): Update declaration.
---
gdb/breakpoint.c | 10 +++++-----
gdb/breakpoint.h | 2 +-
gdb/tracepoint.c | 2 +-
3 files changed, 7 insertions(+), 7 deletions(-)
@@ -15532,7 +15532,7 @@ trace_pass_command (char *args, int from_tty)
}
else if (*args == '\0')
{
- t1 = get_tracepoint_by_number (&args);
+ t1 = get_tracepoint_by_number (args);
if (t1)
trace_pass_set_count (t1, count, from_tty);
}
@@ -15592,12 +15592,12 @@ get_tracepoint_by_number_on_target (int num)
(tracepoint_count) is returned. */
struct tracepoint *
-get_tracepoint_by_number (char **arg)
+get_tracepoint_by_number (char *arg)
{
struct breakpoint *t;
int tpnum;
- if (arg == NULL || *arg == NULL || ! **arg)
+ if (arg == NULL || *arg == 0)
{
if (tracepoint_count == 0)
{
@@ -15609,9 +15609,9 @@ get_tracepoint_by_number (char **arg)
}
else
{
- char *instring = *arg;
+ char *instring = arg;
- tpnum = get_number (arg);
+ tpnum = get_number (&arg);
if (tpnum == 0)
{
@@ -1490,7 +1490,7 @@ extern struct tracepoint *get_tracepoint_by_number_on_target (int num);
/* Find a tracepoint by parsing a number in the supplied string. */
extern struct tracepoint *
- get_tracepoint_by_number (char **arg);
+ get_tracepoint_by_number (char *arg);
/* Return a vector of all tracepoints currently defined. The vector
is newly allocated; the caller should free when done with it. */
@@ -653,7 +653,7 @@ trace_actions_command (char *args, int from_tty)
struct tracepoint *t;
struct command_line *l;
- t = get_tracepoint_by_number (&args);
+ t = get_tracepoint_by_number (args);
if (t)
{
char *tmpbuf =