From: Hallvard B Furuseth <h.b.furuseth@usit.uio.no>
Date: Fri, 19 Aug 2011 19:56:10 +0000 (+0200)
Subject: Macro cleanup: Parenthesize, simplify, remove a ;
X-Git-Tag: OPENLDAP_REL_ENG_2_4_27~148^2~86
X-Git-Url: https://git.sur5r.net/?p=openldap;a=commitdiff_plain;h=3029bb694de9128b6539c924b3385686dbc427bb

Macro cleanup: Parenthesize, simplify, remove a ;
---

diff --git a/libraries/libmdb/mdb.c b/libraries/libmdb/mdb.c
index 3003784985..3de7e1d71b 100644
--- a/libraries/libmdb/mdb.c
+++ b/libraries/libmdb/mdb.c
@@ -178,7 +178,7 @@ typedef struct MDB_page {		/* represents a page of storage */
 #define IS_BRANCH(p)	 F_ISSET((p)->mp_flags, P_BRANCH)
 #define IS_OVERFLOW(p)	 F_ISSET((p)->mp_flags, P_OVERFLOW)
 
-#define OVPAGES(size, psize)	(PAGEHDRSZ + size + psize - 1) / psize;
+#define OVPAGES(size, psize)	((PAGEHDRSZ-1 + (size)) / (psize) + 1)
 
 typedef struct MDB_db {
 	uint32_t	md_pad;
@@ -252,7 +252,7 @@ struct MDB_cursor {
 	struct MDB_xcursor	*mc_xcursor;
 };
 
-#define METADATA(p)	 ((void *)((char *)p + PAGEHDRSZ))
+#define METADATA(p)	 ((void *)((char *)(p) + PAGEHDRSZ))
 
 typedef struct MDB_node {
 #define mn_pgno		 mn_p.np_pgno
diff --git a/libraries/libmdb/mdb.h b/libraries/libmdb/mdb.h
index 372dfc1364..9d6ebf429e 100644
--- a/libraries/libmdb/mdb.h
+++ b/libraries/libmdb/mdb.h
@@ -35,7 +35,7 @@
 #define MDB_VERSION_MAJOR	0
 #define MDB_VERSION_MINOR	8
 #define MDB_VERSION_PATCH	0
-#define MDB_VERINT(a,b,c)	((a << 24) | (b << 16) | c)
+#define MDB_VERINT(a,b,c)	(((a) << 24) | ((b) << 16) | (c))
 #define MDB_VERSION_FULL	\
 	MDB_VERINT(MDB_VERSION_MAJOR,MDB_VERSION_MINOR,MDB_VERSION_PATCH)
 #define MDB_VERSION_DATE	"August 11, 2011"
diff --git a/libraries/libmdb/midl.c b/libraries/libmdb/midl.c
index 136798f75f..64eefe82c8 100644
--- a/libraries/libmdb/midl.c
+++ b/libraries/libmdb/midl.c
@@ -22,10 +22,10 @@
 typedef unsigned long pgno_t;
 
 /* Sort the IDLs from highest to lowest */
-#define IDL_CMP(x,y)	 ( x > y ? -1 : ( x < y ? 1 : 0 ) )
+#define IDL_CMP(x,y)	 ( (x) > (y) ? -1 : (x) < (y) )
 
 /* Sort the IDL2s from lowest to highest */
-#define IDL2_CMP(x,y)	 ( x < y ? -1 : ( x > y ? 1 : 0 ) )
+#define IDL2_CMP(x,y)	 ( (x) < (y) ? -1 : (x) > (y) )
 
 unsigned mdb_midl_search( ID *ids, ID id )
 {
diff --git a/libraries/libmdb/midl.h b/libraries/libmdb/midl.h
index 31a4b67bd2..0836760ab5 100644
--- a/libraries/libmdb/midl.h
+++ b/libraries/libmdb/midl.h
@@ -18,7 +18,7 @@
 #define _MDB_MIDL_H_
 
 #define	ID	unsigned long
-#define	NOID	((ID)~0)
+#define	NOID	(~(ID)0)
 
 /* IDL sizes - likely should be even bigger
  *   limiting factors: sizeof(ID), thread stack size
@@ -64,12 +64,12 @@
 #define MDB_IDL_ID( bdb, ids, id ) MDB_IDL_RANGE( ids, id, ((bdb)->bi_lastid) )
 #define MDB_IDL_ALL( bdb, ids ) MDB_IDL_RANGE( ids, 1, ((bdb)->bi_lastid) )
 
-#define MDB_IDL_FIRST( ids )	( ids[1] )
+#define MDB_IDL_FIRST( ids )	( (ids)[1] )
 #define MDB_IDL_LAST( ids )		( MDB_IDL_IS_RANGE(ids) \
-	? ids[2] : ids[ids[0]] )
+	? (ids)[2] : (ids)[(ids)[0]] )
 
 #define MDB_IDL_N( ids )		( MDB_IDL_IS_RANGE(ids) \
-	? (ids[2]-ids[1])+1 : ids[0] )
+	? ((ids)[2]-(ids)[1])+1 : (ids)[0] )
 
 int mdb_midl_insert( ID *ids, ID id );