From d538e36decd5628c084dbd5a7de13719b8b92121 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Mon, 27 Feb 2017 14:09:34 -0500 Subject: [PATCH] Remove struct keyword from range-based for loop MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The previous patch introduced this error with recent-ish GCCs: ../../binutils-gdb/gdb/remote.c: In function ‘int remote_add_target_side_condition(gdbarch*, bp_target_info*, char*, char*)’: ../../binutils-gdb/gdb/remote.c:9668:8: error: types may not be defined in a for-range-declaration [-Werror] for (struct agent_expr *aexpr : bp_tgt->conditions) ^~~~~~ Removing the struct keyword fixes the error. gdb/ChangeLog: * remote.c (remote_add_target_side_condition): Remove "struct" keyword from range-based for loop. --- gdb/ChangeLog | 5 +++++ gdb/remote.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a872f1b..fafa10c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2017-02-27 Simon Marchi + * remote.c (remote_add_target_side_condition): Remove "struct" + keyword from range-based for loop. + +2017-02-27 Simon Marchi + * remote.c (remote_add_target_side_condition): Use range-based for loop. Update comment. diff --git a/gdb/remote.c b/gdb/remote.c index aedbd08..5da8319 100644 --- a/gdb/remote.c +++ b/gdb/remote.c @@ -9665,7 +9665,7 @@ remote_add_target_side_condition (struct gdbarch *gdbarch, buf++; /* Send conditions to the target. */ - for (struct agent_expr *aexpr : bp_tgt->conditions) + for (agent_expr *aexpr : bp_tgt->conditions) { xsnprintf (buf, buf_end - buf, "X%x,", aexpr->len); buf += strlen (buf); -- 2.7.4