all: move definitions of INLINE into compiler.h
authorFerry Huberts <ferry.huberts@pelagic.nl>
Sat, 30 Apr 2016 20:24:51 +0000 (22:24 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Sat, 30 Apr 2016 20:41:03 +0000 (22:41 +0200)
So that there are no duplicated definitions.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/common/avl.h
src/common/list.h
src/compiler.h [new file with mode: 0644]
src/defs.h
src/olsr_cfg.h
src/olsr_protocol.h

index 7c76d00..a5bf12f 100644 (file)
 #define _AVL_H
 
 #include <stddef.h>
+#include "compiler.h"
 #include "defs.h"
 
-/* must be declared here because of circular dependency through "defs.h" */
-#define INLINE inline __attribute__((always_inline))
-
 struct avl_node {
   struct avl_node *parent;
   struct avl_node *left;
index 8f8e702..23ba171 100644 (file)
@@ -47,6 +47,7 @@
 #define _LIST_H
 
 #include <stddef.h>
+#include "compiler.h"
 #include "defs.h"
 
 struct list_node {
@@ -54,9 +55,6 @@ struct list_node {
   struct list_node *prev;
 };
 
-/* must be declared here because of circular dependency through "defs.h" */
-#define INLINE inline __attribute__((always_inline))
-
 void list_head_init(struct list_node *);
 void list_node_init(struct list_node *);
 int list_node_on_list(struct list_node *);
diff --git a/src/compiler.h b/src/compiler.h
new file mode 100644 (file)
index 0000000..ee2d139
--- /dev/null
@@ -0,0 +1,61 @@
+/*
+ * The olsr.org Optimized Link-State Routing daemon (olsrd)
+ *
+ * (c) by the OLSR project
+ *
+ * See our Git repository to find out who worked on this file
+ * and thus is a copyright holder on it.
+ *
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ * * Redistributions in binary form must reproduce the above copyright
+ *   notice, this list of conditions and the following disclaimer in
+ *   the documentation and/or other materials provided with the
+ *   distribution.
+ * * Neither the name of olsr.org, olsrd nor the names of its
+ *   contributors may be used to endorse or promote products derived
+ *   from this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+ * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+ * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+ * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+ * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+ * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
+ * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Visit http://www.olsr.org for more information.
+ *
+ * If you find this software useful feel free to make a donation
+ * to the project. For more information see the website or contact
+ * the copyright holders.
+ *
+ */
+
+#ifndef _OLSR_COMPILER_H
+#define _OLSR_COMPILER_H
+
+#define INLINE inline __attribute__((always_inline))
+
+/*
+ * On ARM, the compiler spits out additional warnings if called
+ * with -Wcast-align if you cast e.g. char* -> int*. While this
+ * is fine, most of that warnings are un-critical. Also the ARM
+ * CPU will throw BUS_ERROR if alignment does not fit. For this,
+ * we add an additional cast to (void *) to prevent the warning.
+ */
+#define ARM_NOWARN_ALIGN(x) ((void *)(x))
+#define CONST_ARM_NOWARN_ALIGN(x) ((const void *)(x))
+
+#endif /* _OLSR_COMPILER_H */
index 5575052..a132f8f 100644 (file)
@@ -61,6 +61,7 @@
 #include "compat.h"
 #endif /* _WIN32 */
 
+#include "compiler.h"
 #include "olsr_protocol.h"
 #include "olsr_cfg.h"
 
@@ -114,18 +115,6 @@ extern FILE *debug_handle;
 #define MIN(x,y)       ((x) < (y) ? (x) : (y))
 #endif /* MIN */
 
-#define INLINE inline __attribute__((always_inline))
-
-/*
- * On ARM, the compiler spits out additional warnings if called
- * with -Wcast-align if you cast e.g. char* -> int*. While this
- * is fine, most of that warnings are un-critical. Also the ARM
- * CPU will throw BUS_ERROR if alignment does not fit. For this,
- * we add an additional cast to (void *) to prevent the warning.
- */
-#define ARM_NOWARN_ALIGN(x) ((void *)(x))
-#define CONST_ARM_NOWARN_ALIGN(x) ((const void *)(x))
-
 /*
  * A somewhat safe version of strncpy and strncat. Note, that
  * BSD/Solaris strlcpy()/strlcat() differ in implementation, while
index 575589c..34d583c 100644 (file)
@@ -46,6 +46,7 @@
 #ifndef _OLSRD_CFGPARSER_H
 #define _OLSRD_CFGPARSER_H
 
+#include "compiler.h"
 #include "defs.h"
 #include "olsr_types.h"
 #include "common/autobuf.h"
@@ -53,9 +54,6 @@
 #include "pud/src/receiver.h"
 #endif /* HTTPINFO_PUD */
 
-/* must be declared here because of circular dependency through "defs.h" */
-#define INLINE inline __attribute__((always_inline))
-
 /* set to 1 to collect all startup sleep into one sleep
  * (just as long as the longest sleep)
  * useful if many errors on many interfaces */
index 2f3c396..a995db0 100644 (file)
@@ -55,6 +55,7 @@ struct olsr;
 
 #include "olsr_types.h"
 #include "olsr_cfg.h"
+#include "compiler.h"
 
 #include <string.h>
 
@@ -63,9 +64,6 @@ struct olsr;
 #define OLSR_MSGHDRSZ_IPV4 12
 #define OLSR_MSGHDRSZ_IPV6 24
 
-/* must be declared here because of circular dependency through "defs.h" */
-#define INLINE inline __attribute__((always_inline))
-
 /*
  *Emission Intervals
  */