Use range-based for loop in remote_add_target_side_condition

Using a range based for loop makes this code a bit more clean and
readable.

The comment above is clearly erroneous, so I've updated it.

gdb/ChangeLog:

	* remote.c (remote_add_target_side_condition): Use range-based
	for loop.  Update comment.
This commit is contained in:
Simon Marchi 2017-02-27 13:54:53 -05:00
parent 2123df0ebf
commit 8362122330
2 changed files with 7 additions and 4 deletions

View File

@ -1,3 +1,8 @@
2017-02-27 Simon Marchi <simon.marchi@ericsson.com>
* remote.c (remote_add_target_side_condition): Use range-based
for loop. Update comment.
2017-02-27 Yao Qi <yao.qi@linaro.org>
* f-typeprint.c (f_print_type): Check "varstring" is empty first.

View File

@ -9664,11 +9664,9 @@ remote_add_target_side_condition (struct gdbarch *gdbarch,
xsnprintf (buf, buf_end - buf, "%s", ";");
buf++;
/* Send conditions to the target and free the vector. */
for (int ix = 0; ix < bp_tgt->conditions.size (); ix++)
/* Send conditions to the target. */
for (struct agent_expr *aexpr : bp_tgt->conditions)
{
struct agent_expr *aexpr = bp_tgt->conditions[ix];
xsnprintf (buf, buf_end - buf, "X%x,", aexpr->len);
buf += strlen (buf);
for (int i = 0; i < aexpr->len; ++i)