The integer plugin parameter handler should be used for dyn gw's
authorEric Malkowski <eric@bvwireless.net>
Thu, 4 Sep 2008 05:06:20 +0000 (07:06 +0200)
committerEric Malkowski <eric@bvwireless.net>
Thu, 4 Sep 2008 05:06:20 +0000 (07:06 +0200)
"interval" as it's more correct since it's the number of whole seconds
between pings, not a floating point number (the nanosleep used to
implement it wants a struct timespec and the number of seconds is
integer and nanoseconds is set to 0L in the code).

lib/dyn_gw/src/olsrd_dyn_gw.c
lib/dyn_gw/src/olsrd_dyn_gw.h

index 9bd091d..94a5b57 100644 (file)
@@ -133,20 +133,6 @@ olsr_event_doing_hna(void *);
 /**
  * read config file parameters
  */
-static int set_plugin_double(const char *value, void *data, set_plugin_parameter_addon addon __attribute__((unused)))
-{
-    double d = 0.0;
-    sscanf(value, "%lf", &d);
-    if (data != NULL) {
-        double *v = data;
-        *v = d;
-        OLSR_PRINTF(1, "%s double %lf\n", "Got", d);
-    } else {
-        OLSR_PRINTF(0, "%s double %lf\n", "Ignored", d);
-    }
-    return 0;
-}
-
 static int set_plugin_ping(const char *value, void *data __attribute__((unused)), set_plugin_parameter_addon addon __attribute__((unused)))
 {
     union olsr_ip_addr foo_addr;
@@ -206,7 +192,7 @@ static int set_plugin_hna(const char *value, void *data __attribute__((unused)),
 }
 
 static const struct olsrd_plugin_parameters plugin_parameters[] = {
-    { .name = "interval", .set_plugin_parameter = &set_plugin_double, .data = &check_interval },
+    { .name = "interval", .set_plugin_parameter = &set_plugin_int, .data = &check_interval },
     { .name = "ping",     .set_plugin_parameter = &set_plugin_ping,   .data = NULL },
     { .name = "hna",      .set_plugin_parameter = &set_plugin_hna,    .data = NULL },
 };
index 0499f0b..7ba1162 100644 (file)
@@ -43,6 +43,7 @@
 #define _OLSRD_PLUGIN_TEST
 
 #include "olsrd_plugin.h"
+#include "plugin_util.h"
 
 #define INET_NET       0
 #define INET_PREFIX    0