From: Simon Marchi <simon.marchi@ericsson.com>
Date: Mon, 27 Feb 2017 18:54:53 +0000 (-0500)
Subject: Use range-based for loop in remote_add_target_side_condition
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8362122330c97c8c4a15da4e7ba8aa29f0c11157;p=binutils-gdb.git

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.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index a7be8266ea9..a872f1bfb98 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -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.
diff --git a/gdb/remote.c b/gdb/remote.c
index 87fb6e02ea3..aedbd08cc43 100644
--- a/gdb/remote.c
+++ b/gdb/remote.c
@@ -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)