properly parenthesize two macros
authorTom Tromey <tromey@redhat.com>
Fri, 25 Jul 2014 15:20:03 +0000 (09:20 -0600)
committerTom Tromey <tromey@redhat.com>
Fri, 25 Jul 2014 15:20:03 +0000 (09:20 -0600)
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  <tromey@redhat.com>

* target.h (target_stopped_data_address)
(target_watchpoint_addr_within_range): Use "->", not ".".  Fix
parentheses.

gdb/ChangeLog
gdb/target.h

index f265915153907271ac8d38897cab123cae646ef5..11cdbf861f3b762b1d8705c6a32eb3f0704e6af6 100644 (file)
@@ -1,3 +1,9 @@
+2014-07-25  Tom Tromey  <tromey@redhat.com>
+
+       * target.h (target_stopped_data_address)
+       (target_watchpoint_addr_within_range): Use "->", not ".".  Fix
+       parentheses.
+
 2014-07-25  Pierre Langlois  <pierre.langlois@embecosm.com>
 
        * avr-tdep.c (avr_address_to_pointer): Clarify the conversion in the
index bcf97d9ab4797209d1475456734ed49653ddab2f..29a1f01ccb852a3846c54a9959079cd85474e7e5 100644 (file)
@@ -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