Implement "DEF" configuration.

[collection4.git] / graph_list.c

diff --git a/graph_list.c b/graph_list.c
index c9fbbab..eff134f 100644 (file)
--- a/graph_list.c
+++ b/graph_list.c
@@ -616,72 +616,80 @@ static int gl_clear_instances (void) /* {{{ */
 /*
  * Config functions
  */
-static int config_get_string (const oconfig_item_t *ci, /* {{{ */
-    char **ret_str)
+static graph_ident_t *graph_config_get_selector (const oconfig_item_t *ci) /* {{{ */
 {
-  char *tmp;
+  char *host = NULL;
+  char *plugin = NULL;
+  char *plugin_instance = NULL;
+  char *type = NULL;
+  char *type_instance = NULL;
+  graph_ident_t *ret;
+  int i;
 
-  if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING))
-    return (EINVAL);
+  for (i = 0; i < ci->children_num; i++)
+  {
+    oconfig_item_t *child;
 
-  tmp = strdup (ci->values[0].value.string);
-  if (tmp == NULL)
-    return (ENOMEM);
+    child = ci->children + i;
 
-  free (*ret_str);
-  *ret_str = tmp;
+    if (strcasecmp ("Host", child->key) == 0)
+      graph_config_get_string (child, &host);
+    else if (strcasecmp ("Plugin", child->key) == 0)
+      graph_config_get_string (child, &plugin);
+    else if (strcasecmp ("PluginInstance", child->key) == 0)
+      graph_config_get_string (child, &plugin_instance);
+    else if (strcasecmp ("Type", child->key) == 0)
+      graph_config_get_string (child, &type);
+    else if (strcasecmp ("TypeInstance", child->key) == 0)
+      graph_config_get_string (child, &type_instance);
+    /* else: ignore all other directives here. */
+  } /* for */
 
-  return (0);
-} /* }}} int config_get_string */
+  ret = ident_create (host, plugin, plugin_instance, type, type_instance);
+
+  free (host);
+  free (plugin);
+  free (plugin_instance);
+  free (type);
+  free (type_instance);
+
+  return (ret);
+} /* }}} int graph_config_get_selector */
 
 /*
  * Global functions
  */
+
 int graph_config_add (const oconfig_item_t *ci) /* {{{ */
 {
-  char *host = NULL;
-  char *plugin = NULL;
-  char *plugin_instance = NULL;
-  char *type = NULL;
-  char *type_instance = NULL;
+  graph_ident_t *select;
   graph_config_t *cfg = NULL;
   int i;
 
+  select = graph_config_get_selector (ci);
+  if (select == NULL)
+    return (EINVAL);
+
   cfg = graph_create (/* selector = */ NULL);
   if (cfg == NULL)
     return (ENOMEM);
 
+  cfg->select = select;
+
   for (i = 0; i < ci->children_num; i++)
   {
     oconfig_item_t *child;
 
     child = ci->children + i;
 
-    if (strcasecmp ("Host", child->key) == 0)
-      config_get_string (child, &host);
-    else if (strcasecmp ("Plugin", child->key) == 0)
-      config_get_string (child, &plugin);
-    else if (strcasecmp ("PluginInstance", child->key) == 0)
-      config_get_string (child, &plugin_instance);
-    else if (strcasecmp ("Type", child->key) == 0)
-      config_get_string (child, &type);
-    else if (strcasecmp ("TypeInstance", child->key) == 0)
-      config_get_string (child, &type_instance);
-    else if (strcasecmp ("Title", child->key) == 0)
-      config_get_string (child, &cfg->title);
+    if (strcasecmp ("Title", child->key) == 0)
+      graph_config_get_string (child, &cfg->title);
     else if (strcasecmp ("VerticalLabel", child->key) == 0)
-      config_get_string (child, &cfg->vertical_label);
-    /* TODO: DEFs! */
+      graph_config_get_string (child, &cfg->vertical_label);
+    else if (strcasecmp ("DEF", child->key) == 0)
+      def_config (cfg, child);
   } /* for */
 
-  cfg->select = ident_create (host, plugin, plugin_instance,
-      type, type_instance);
-  if (cfg->select == NULL)
-  {
-    graph_destroy (cfg);
-    return (EINVAL);
-  }
-
   graph_append (&graph_config_staging, cfg);
 
   return (0);
@@ -889,6 +897,21 @@ graph_ident_t *gl_graph_get_selector (graph_config_t *cfg) /* {{{ */
   return (ident_clone (cfg->select));
 } /* }}} graph_ident_t *gl_graph_get_selector */
 
+int gl_graph_add_def (graph_config_t *cfg, graph_def_t *def) /* {{{ */
+{
+  if ((cfg == NULL) || (def == NULL))
+    return (EINVAL);
+
+  if (cfg->defs == NULL)
+  {
+    cfg->defs = def;
+    return (0);
+  }
+
+  return (def_append (cfg->defs, def));
+} /* }}} int gl_graph_add_def */
+
+
 int gl_instance_get_all (gl_inst_callback callback, /* {{{ */
     void *user_data)
 {