OpenE2K
/
binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

gdb/gdbserver/ 

* tracepoint.c (cur_action, cur_step_action): Make them unsigned.
	(cmd_qtfp): Initialize cur_action and cur_step_action 0 instead
	of -1.
	(cmd_qtsp): Adjust condition.  Do post increment.
	Set cur_action and cur_step_action back to 0.
This commit is contained in:
Yao Qi 2013-03-07 13:17:28 +00:00
parent 77dec115af
commit e64f749990
2 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
gdb/gdbserver/ChangeLog
View File

@ -1,3 +1,11 @@
2013-03-07 Yao Qi <yao@codesourcery.com>
* tracepoint.c (cur_action, cur_step_action): Make them unsigned.
(cmd_qtfp): Initialize cur_action and cur_step_action 0 instead
of -1.
(cmd_qtsp): Adjust condition. Do post increment.
Set cur_action and cur_step_action back to 0.
2013-03-07 Jeremy Bennett <jeremy.bennett@embecosm.com> 2013-03-07 Jeremy Bennett <jeremy.bennett@embecosm.com>
PR server/15236 PR server/15236

16
gdb/gdbserver/tracepoint.c
View File

@ -3684,8 +3684,8 @@ cmd_qtp (char *own_buf)
/* State variables to help return all the tracepoint bits. */ /* State variables to help return all the tracepoint bits. */
static struct tracepoint *cur_tpoint; static struct tracepoint *cur_tpoint;
static int cur_action; static unsigned int cur_action;
static int cur_step_action; static unsigned int cur_step_action;
static struct source_string *cur_source_string; static struct source_string *cur_source_string;
static struct trace_state_variable *cur_tsv; static struct trace_state_variable *cur_tsv;
@ -3759,7 +3759,7 @@ cmd_qtfp (char *packet)
trace_debug ("Returning first tracepoint definition piece"); trace_debug ("Returning first tracepoint definition piece");
cur_tpoint = tracepoints; cur_tpoint = tracepoints;
cur_action = cur_step_action = -1; cur_action = cur_step_action = 0;
cur_source_string = NULL; cur_source_string = NULL;
if (cur_tpoint) if (cur_tpoint)
@ -3784,17 +3784,17 @@ cmd_qtsp (char *packet)
GDB misbehavior. */ GDB misbehavior. */
strcpy (packet, "l"); strcpy (packet, "l");
} }
else if (cur_action < cur_tpoint->numactions - 1) else if (cur_action < cur_tpoint->numactions)
{ {
++cur_action;
response_action (packet, cur_tpoint, response_action (packet, cur_tpoint,
cur_tpoint->actions_str[cur_action], 0); cur_tpoint->actions_str[cur_action], 0);
++cur_action;
} }
else if (cur_step_action < cur_tpoint->num_step_actions - 1) else if (cur_step_action < cur_tpoint->num_step_actions)
{ {
++cur_step_action;
response_action (packet, cur_tpoint, response_action (packet, cur_tpoint,
cur_tpoint->step_actions_str[cur_step_action], 1); cur_tpoint->step_actions_str[cur_step_action], 1);
++cur_step_action;
} }
else if ((cur_source_string else if ((cur_source_string
? cur_source_string->next ? cur_source_string->next
@ -3809,7 +3809,7 @@ cmd_qtsp (char *packet)
else else
{ {
cur_tpoint = cur_tpoint->next; cur_tpoint = cur_tpoint->next;
cur_action = cur_step_action = -1; cur_action = cur_step_action = 0;
cur_source_string = NULL; cur_source_string = NULL;
if (cur_tpoint) if (cur_tpoint)
response_tracepoint (packet, cur_tpoint); response_tracepoint (packet, cur_tpoint);