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  <tromey@redhat.com>

	* 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 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -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

diff --git a/gdb/target.h b/gdb/target.h
index bcf97d9..29a1f01 100644 (file)
--- 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