Changeset: 4395aa6ba22e for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=4395aa6ba22e
Modified Files:
        gdk/gdk_search.h
Branch: default
Log Message:

use paratheses around macro parameters


diffs (80 lines):

diff --git a/gdk/gdk_search.h b/gdk/gdk_search.h
--- a/gdk/gdk_search.h
+++ b/gdk/gdk_search.h
@@ -34,47 +34,47 @@ gdk_export BUN HASHprobe(Hash *h, const 
 gdk_export BUN HASHlist(Hash *h, BUN i);
 
 
-#define HASHnil(H)     H->nil
+#define HASHnil(H)     (H)->nil
 
 /* play around with h->Hash[i] and h->Link[j] */
 #if SIZEOF_BUN <= 4
 #define HASHget(h,i)   \
-       ((BUN)  (h->width == BUN4 ? ((BUN4type*) h->Hash)[i] : \
-               (h->width == BUN2 ? ((BUN2type*) h->Hash)[i] : \
-                                   ((BUN1type*) h->Hash)[i] )))
+       ((BUN)  ((h)->width == BUN4 ? ((BUN4type*) (h)->Hash)[i] : \
+               ((h)->width == BUN2 ? ((BUN2type*) (h)->Hash)[i] : \
+                                     ((BUN1type*) (h)->Hash)[i] )))
 #define HASHput(h,i,v) \
-       (void)  (h->width == BUN4 ? (((BUN4type*) h->Hash)[i] = (BUN4type) v) : 
\
-               (h->width == BUN2 ? (((BUN2type*) h->Hash)[i] = (BUN2type) v) : 
\
-                                   (((BUN1type*) h->Hash)[i] = (BUN1type) v) ))
+       (void)  ((h)->width == BUN4 ? (((BUN4type*) (h)->Hash)[i] = (BUN4type) 
(v)) : \
+               ((h)->width == BUN2 ? (((BUN2type*) (h)->Hash)[i] = (BUN2type) 
(v)) : \
+                                     (((BUN1type*) (h)->Hash)[i] = (BUN1type) 
(v)) ))
 #define HASHgetlink(h,i)       \
-       ((BUN)  (h->width == BUN4 ? ((BUN4type*) h->Link)[i] : \
-               (h->width == BUN2 ? ((BUN2type*) h->Link)[i] : \
-                                   ((BUN1type*) h->Link)[i] )))
+       ((BUN)  ((h)->width == BUN4 ? ((BUN4type*) (h)->Link)[i] : \
+               ((h)->width == BUN2 ? ((BUN2type*) (h)->Link)[i] : \
+                                     ((BUN1type*) (h)->Link)[i] )))
 #define HASHputlink(h,i,v)     \
-       (void)  (h->width == BUN4 ? (((BUN4type*) h->Link)[i] = (BUN4type) v) : 
\
-               (h->width == BUN2 ? (((BUN2type*) h->Link)[i] = (BUN2type) v) : 
\
-                                   (((BUN1type*) h->Link)[i] = (BUN1type) v) ))
+       (void)  ((h)->width == BUN4 ? (((BUN4type*) (h)->Link)[i] = (BUN4type) 
(v)) : \
+               ((h)->width == BUN2 ? (((BUN2type*) (h)->Link)[i] = (BUN2type) 
(v)) : \
+                                     (((BUN1type*) (h)->Link)[i] = (BUN1type) 
(v)) ))
 #else
 #define HASHget(h,i)   \
-       ((BUN)  (h->width == BUN8 ? ((BUN8type*) h->Hash)[i] : \
-               (h->width == BUN4 ? ((BUN4type*) h->Hash)[i] : \
-               (h->width == BUN2 ? ((BUN2type*) h->Hash)[i] : \
-                                   ((BUN1type*) h->Hash)[i] ))))
+       ((BUN)  ((h)->width == BUN8 ? ((BUN8type*) (h)->Hash)[i] : \
+               ((h)->width == BUN4 ? ((BUN4type*) (h)->Hash)[i] : \
+               ((h)->width == BUN2 ? ((BUN2type*) (h)->Hash)[i] : \
+                                     ((BUN1type*) (h)->Hash)[i] ))))
 #define HASHput(h,i,v) \
-       (void)  (h->width == BUN8 ? (((BUN8type*) h->Hash)[i] = (BUN8type) v) : 
\
-               (h->width == BUN4 ? (((BUN4type*) h->Hash)[i] = (BUN4type) v) : 
\
-               (h->width == BUN2 ? (((BUN2type*) h->Hash)[i] = (BUN2type) v) : 
\
-                                   (((BUN1type*) h->Hash)[i] = (BUN1type) v) 
)))
+       (void)  ((h)->width == BUN8 ? (((BUN8type*) (h)->Hash)[i] = (BUN8type) 
(v)) : \
+               ((h)->width == BUN4 ? (((BUN4type*) (h)->Hash)[i] = (BUN4type) 
(v)) : \
+               ((h)->width == BUN2 ? (((BUN2type*) (h)->Hash)[i] = (BUN2type) 
(v)) : \
+                                     (((BUN1type*) (h)->Hash)[i] = (BUN1type) 
(v)) )))
 #define HASHgetlink(h,i)       \
-       ((BUN)  (h->width == BUN8 ? ((BUN8type*) h->Link)[i] : \
-               (h->width == BUN4 ? ((BUN4type*) h->Link)[i] : \
-               (h->width == BUN2 ? ((BUN2type*) h->Link)[i] : \
-                                   ((BUN1type*) h->Link)[i] ))))
+       ((BUN)  ((h)->width == BUN8 ? ((BUN8type*) (h)->Link)[i] : \
+               ((h)->width == BUN4 ? ((BUN4type*) (h)->Link)[i] : \
+               ((h)->width == BUN2 ? ((BUN2type*) (h)->Link)[i] : \
+                                     ((BUN1type*) (h)->Link)[i] ))))
 #define HASHputlink(h,i,v)     \
-       (void)  (h->width == BUN8 ? (((BUN8type*) h->Link)[i] = (BUN8type) v) : 
\
-               (h->width == BUN4 ? (((BUN4type*) h->Link)[i] = (BUN4type) v) : 
\
-               (h->width == BUN2 ? (((BUN2type*) h->Link)[i] = (BUN2type) v) : 
\
-                                   (((BUN1type*) h->Link)[i] = (BUN1type) v) 
)))
+       (void)  ((h)->width == BUN8 ? (((BUN8type*) (h)->Link)[i] = (BUN8type) 
(v)) : \
+               ((h)->width == BUN4 ? (((BUN4type*) (h)->Link)[i] = (BUN4type) 
(v)) : \
+               ((h)->width == BUN2 ? (((BUN2type*) (h)->Link)[i] = (BUN2type) 
(v)) : \
+                                     (((BUN1type*) (h)->Link)[i] = (BUN1type) 
(v)) )))
 #endif
 
 #define mix_sht(X)            (((X)>>7)^(X))
_______________________________________________
checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list

Reply via email to