Remove MAX #define from Linux Std.h
authorRobert Tivy <rtivy@ti.com>
Wed, 18 Mar 2015 21:04:54 +0000 (14:04 -0700)
committerRobert Tivy <rtivy@ti.com>
Fri, 20 Mar 2015 00:16:42 +0000 (17:16 -0700)
The MAX() macro that is #defined in Std.h for Linux can either conflict
or be a duplicate of other MAX() macro definitions.  GCC's glibc defines
MAX in sys/param.h, and code builds will either warn abour or error on
a duplicate or conflicting definition.

Remove the usage of MAX completely to avoid this.  Use a locally-defined,
differently-named macro in C file if appropriate.

linux/include/ti/ipc/Std.h
linux/src/daemon/NameServer_daemon.c
linux/src/tests/MessageQBench.c
linux/src/transport/TransportRpmsg.c

index 7bef96fe070daa90b261f37f962f705b37961318..63adabf7ac3fd0482d3fcc4060fe58785d6f791f 100644 (file)
@@ -94,12 +94,6 @@ typedef PVOID           HANDLE;         /* h    */
 #define FAIL   -1
 //#define NULL            '\0'
 
-#define MAX(a,b) (((a)>(b))?(a):(b))
-
-#if defined (IPC_BUILDOS_ANDROID)
-#define MAX(a,b) (((a)>(b))?(a):(b))
-#endif
-
 #if defined (__cplusplus)
 }
 #endif
index 2fb200c80957cc7f8dcb2c0da2bf2e63a2d61b87..7b327ee6dde98be924057e0418ebcdb4040537f7 100644 (file)
@@ -338,7 +338,7 @@ static void *listener_cb(void *arg)
             }
             sock = NameServer_module->recvSock[i];
             FD_SET(sock, &rfds);
-            maxfd = MAX(sock, maxfd);
+            maxfd = sock > maxfd ? sock : maxfd;
         }
 
         maxfd = maxfd + 1;
index 5fbfcbef3de3b098b7c7b52109fada825b392cef..27edff998551a2b01eb33cead14c6d9062adc4ec 100644 (file)
@@ -216,7 +216,9 @@ int main (int argc, char * argv[])
     }
 
     if (argc > 2) {
-        payloadSize = MAX(strtoul(argv[2], NULL, 0), MINPAYLOADSIZE);
+        payloadSize = strtoul(argv[2], NULL, 0);
+        payloadSize = payloadSize > MINPAYLOADSIZE ? payloadSize :
+                                                     MINPAYLOADSIZE;
     }
 
     if (argc > 3) {
index 2d9898d724dff922f7de5092253206fe12760c74..ae44f7526d04e2c07cc4c82a06e1d6adcaf7aec7 100644 (file)
@@ -66,6 +66,8 @@
 
 #define TransportRpmsg_GROWSIZE 32
 
+#define _MAX(a,b) (((a)>(b))?(a):(b))
+
 /* traces in this file are controlled via _TransportMessageQ_verbose */
 Bool _TransportMessageQ_verbose = FALSE;
 #define verbose _TransportMessageQ_verbose
@@ -325,7 +327,7 @@ Int TransportRpmsg_bind(Void *handle, UInt32 queueId)
 
     /* add to our fat fd array and update select() parameters */
     TransportRpmsg_module->inFds[TransportRpmsg_module->nInFds++] = fd;
-    TransportRpmsg_module->maxFd = MAX(TransportRpmsg_module->maxFd, fd);
+    TransportRpmsg_module->maxFd = _MAX(TransportRpmsg_module->maxFd, fd);
     FD_SET(fd, &TransportRpmsg_module->rfds);
 
     pthread_mutex_unlock(&TransportRpmsg_module->gate);
@@ -390,7 +392,7 @@ Int TransportRpmsg_unbind(Void *handle, UInt32 queueId)
                 /* find new max fd */
                 maxFd = TransportRpmsg_module->unblockEvent;
                 for (j = 0; j < TransportRpmsg_module->nInFds; j++) {
-                    maxFd = MAX(TransportRpmsg_module->inFds[j], maxFd);
+                    maxFd = _MAX(TransportRpmsg_module->inFds[j], maxFd);
                 }
                 TransportRpmsg_module->maxFd = maxFd;
             }