- the "cookies" array is used only here. So make it "static".
authorBernd Petrovitsch <bernd@firmix.at>
Tue, 2 Sep 2008 22:57:37 +0000 (00:57 +0200)
committerBernd Petrovitsch <bernd@firmix.at>
Tue, 2 Sep 2008 22:57:37 +0000 (00:57 +0200)
- the "cookies" array is a global variable and thus initialized to
  0 anyways (Google it or read only e.g.
  http://gcc.gnu.org/ml/gcc-help/2007-07/msg00097.html).
  Make it explciti so and remove the now superfluous initializetion code.

src/olsr_cookie.c

index 628f54f..484ee6f 100644 (file)
@@ -46,7 +46,7 @@
 #include <assert.h>
 
 /* Root directory of the cookies we have in the system */
-struct olsr_cookie_info *cookies[COOKIE_ID_MAX];
+static struct olsr_cookie_info *cookies[COOKIE_ID_MAX] = { 0 };
 
 /*
  * Allocate a cookie for the next available cookie id.
@@ -54,18 +54,9 @@ struct olsr_cookie_info *cookies[COOKIE_ID_MAX];
 struct olsr_cookie_info *
 olsr_alloc_cookie(const char *cookie_name, olsr_cookie_type cookie_type)
 {
-  static olsr_bool first = OLSR_TRUE;
   struct olsr_cookie_info *ci;
   int ci_index;
 
-  /* Clear the cookie root array on the first call */
-  if (first) {
-    for (ci_index = 0; ci_index < COOKIE_ID_MAX; ci_index++) {
-      cookies[ci_index] = NULL;
-    }
-    first = OLSR_FALSE;
-  }
-
   /*
    * Look for an unused index.
    * For ease of troubleshooting (non-zero patterns) we start at index 1.