From d8be293957340f1cbe16d65d78d64aebc01df18f Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Fri, 25 Jul 2014 09:20:03 -0600 Subject: [PATCH] properly parenthesize two macros I happened to notice that a couple of macros in target.h weren't properly using parens and as a result had a strange definition. This patch adds the parens and then fixes the macros to be written as must have been intended. Tested by rebuilding. I'm pushing this as obvious. 2014-07-25 Tom Tromey * target.h (target_stopped_data_address) (target_watchpoint_addr_within_range): Use "->", not ".". Fix parentheses. --- gdb/ChangeLog | 6 ++++++ gdb/target.h | 4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f2659151539..11cdbf861f3 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2014-07-25 Tom Tromey + + * target.h (target_stopped_data_address) + (target_watchpoint_addr_within_range): Use "->", not ".". Fix + parentheses. + 2014-07-25 Pierre Langlois * avr-tdep.c (avr_address_to_pointer): Clarify the conversion in the diff --git a/gdb/target.h b/gdb/target.h index bcf97d9ab47..29a1f01ccb8 100644 --- a/gdb/target.h +++ b/gdb/target.h @@ -1822,12 +1822,12 @@ extern int target_ranged_break_num_registers (void); target_stopped_by_watchpoint, in such case place it to *ADDR_P. Only the INFERIOR_PTID task is being queried. */ #define target_stopped_data_address(target, addr_p) \ - (*target.to_stopped_data_address) (target, addr_p) + (*(target)->to_stopped_data_address) (target, addr_p) /* Return non-zero if ADDR is within the range of a watchpoint spanning LENGTH bytes beginning at START. */ #define target_watchpoint_addr_within_range(target, addr, start, length) \ - (*target.to_watchpoint_addr_within_range) (target, addr, start, length) + (*(target)->to_watchpoint_addr_within_range) (target, addr, start, length) /* Return non-zero if the target is capable of using hardware to evaluate the condition expression. In this case, if the condition is false when -- 2.30.2