From: Tom Tromey Date: Fri, 25 Jul 2014 15:20:03 +0000 (-0600) Subject: properly parenthesize two macros X-Git-Tag: binutils-2_25~736 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d8be293957340f1cbe16d65d78d64aebc01df18f;p=external%2Fbinutils.git 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. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index f265915..11cdbf8 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 bcf97d9..29a1f01 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