X-Git-Url: http://git.maemo.org/git/?p=browser-switch;a=blobdiff_plain;f=config.c;h=2dcbfae26593ebaab7be61b098c01f67cf393f91;hp=663858a0d931be65d71905c697fbdffdb867f91d;hb=refs%2Ftags%2Ffremantle-package-3.3-2fremantle2;hpb=ec8b58af1b1cf22f34478faf476047e7ce996245

diff --git a/config.c b/config.c
index 663858a..2dcbfae 100644
--- a/config.c
+++ b/config.c
@@ -21,48 +21,121 @@
  */
 
 #include <stdlib.h>
+#include <stddef.h>
 #include <string.h>
 
 #include "configfile.h"
 #include "config.h"
 
-/* Initialize a swb_config struct with configuration defaults */
-void swb_config_init(struct swb_config *cfg) {
-	if (!cfg)
+/* The Browser Switchboard config file options */
+struct swb_config_option swb_config_options[] = {
+	{ "continuous_mode", SWB_CONFIG_OPT_INT, SWB_CONFIG_CONTINUOUS_MODE_SET },
+	{ "default_browser", SWB_CONFIG_OPT_STRING, SWB_CONFIG_DEFAULT_BROWSER_SET },
+	{ "other_browser_cmd", SWB_CONFIG_OPT_STRING, SWB_CONFIG_OTHER_BROWSER_CMD_SET },
+	{ "logging", SWB_CONFIG_OPT_STRING, SWB_CONFIG_LOGGING_SET },
+	{ "autostart_microb", SWB_CONFIG_OPT_INT, SWB_CONFIG_AUTOSTART_MICROB_SET },
+	{ NULL, 0, 0 },
+};
+
+/* Browser Switchboard configuration defaults */
+static struct swb_config swb_config_defaults = {
+	.flags = SWB_CONFIG_INITIALIZED,
+	.continuous_mode = 1,
+	.default_browser = "microb",
+	.other_browser_cmd = NULL,
+	.logging = "stdout",
+	.autostart_microb = -1,
+};
+
+
+/* Copy the contents of an swb_config struct
+   The entries[] array means that the standard copy will not work */
+void swb_config_copy(struct swb_config *dst, struct swb_config *src) {
+	if (!dst || !src)
 		return;
 
-	cfg->continuous_mode = 0;
-	cfg->default_browser = "microb";
-	cfg->other_browser_cmd = NULL;
-	cfg->logging = "stdout";
+	dst->entries[0] = &(dst->continuous_mode);
+	dst->entries[1] = &(dst->default_browser);
+	dst->entries[2] = &(dst->other_browser_cmd);
+	dst->entries[3] = &(dst->logging);
+	dst->entries[4] = &(dst->autostart_microb);
+
+	dst->flags = src->flags;
 
-	cfg->flags = SWB_CONFIG_INITIALIZED;
+	dst->continuous_mode = src->continuous_mode;
+	dst->default_browser = src->default_browser;
+	dst->other_browser_cmd = src->other_browser_cmd;
+	dst->logging = src->logging;
+	dst->autostart_microb = src->autostart_microb;
+}
+
+/* Initialize a swb_config struct with configuration defaults */
+void swb_config_init(struct swb_config *cfg) {
+	swb_config_copy(cfg, &swb_config_defaults);
 }
 
 /* Free all heap memory used in an swb_config struct
    This MUST NOT be done if any of the strings are being used elsewhere! */
 void swb_config_free(struct swb_config *cfg) {
+	int i;
+
 	if (!cfg)
 		return;
 	if (!(cfg->flags & SWB_CONFIG_INITIALIZED))
 		return;
 
-	if (cfg->flags & SWB_CONFIG_DEFAULT_BROWSER_SET) {
-		free(cfg->default_browser);
-		cfg->default_browser = NULL;
-	}
-	if (cfg->flags & SWB_CONFIG_OTHER_BROWSER_CMD_SET) {
-		free(cfg->other_browser_cmd);
-		cfg->other_browser_cmd = NULL;
-	}
-	if (cfg->flags & SWB_CONFIG_LOGGING_SET) {
-		free(cfg->logging);
-		cfg->logging = NULL;
+	for (i = 0; swb_config_options[i].name; ++i) {
+		if (cfg->flags & swb_config_options[i].set_mask) {
+			switch (swb_config_options[i].type) {
+			  case SWB_CONFIG_OPT_STRING:
+				free(*(char **)cfg->entries[i]);
+				*(char **)cfg->entries[i] = NULL;
+				break;
+			  default:
+				break;
+			}
+		}
 	}
 
 	cfg->flags = 0;
 }
 
+/* Load a value into the part of a struct swb_config indicated by name */
+static int swb_config_load_option(struct swb_config *cfg,
+				  char *name, char *value) {
+	struct swb_config_option *opt;
+	ptrdiff_t i;
+
+	/* Search through list of recognized config options for a match */
+	for (opt = swb_config_options; opt->name; ++opt) {
+		if (strcmp(name, opt->name))
+			continue;
+
+		if (!(cfg->flags & opt->set_mask)) {
+			i = opt - swb_config_options;
+			switch (opt->type) {
+			  case SWB_CONFIG_OPT_STRING:
+				*(char **)cfg->entries[i] = value;
+				break;
+			  case SWB_CONFIG_OPT_INT:
+				*(int *)cfg->entries[i] = atoi(value);
+				free(value);
+				break;
+			}
+			cfg->flags |= opt->set_mask;
+		} else {
+			/* Option was repeated in the config file
+			   We want the first value, so ignore this one */
+			free(value);
+		}
+		return 1;
+	}
+
+	/* Unrecognized config option */
+	free(value);
+	return 0;
+}
+
 /* Read the config file and load settings into the provided swb_config struct
    Caller is responsible for freeing allocated strings with free()
    Returns true on success, false otherwise */
@@ -81,39 +154,8 @@ int swb_config_load(struct swb_config *cfg) {
 	if (!parse_config_file_begin())
 		goto out;
 	while (!parse_config_file_line(fp, &line)) {
-		if (line.parsed) {
-			if (!strcmp(line.key, "continuous_mode")) {
-				if (!(cfg->flags &
-				      SWB_CONFIG_CONTINUOUS_MODE_SET)) {
-					cfg->continuous_mode = atoi(line.value);
-					cfg->flags |=
-						SWB_CONFIG_CONTINUOUS_MODE_SET;
-				}
-				free(line.value);
-			} else if (!strcmp(line.key, "default_browser")) {
-				if (!(cfg->flags &
-				      SWB_CONFIG_DEFAULT_BROWSER_SET)) {
-					cfg->default_browser = line.value;
-					cfg->flags |=
-						SWB_CONFIG_DEFAULT_BROWSER_SET;
-				}
-			} else if (!strcmp(line.key, "other_browser_cmd")) {
-				if (!(cfg->flags &
-				      SWB_CONFIG_OTHER_BROWSER_CMD_SET)) {
-					cfg->other_browser_cmd = line.value;
-					cfg->flags |=
-						SWB_CONFIG_OTHER_BROWSER_CMD_SET;
-				}
-			} else if (!strcmp(line.key, "logging")) {
-				if (!(cfg->flags & SWB_CONFIG_LOGGING_SET)) {
-					cfg->logging = line.value;
-					cfg->flags |= SWB_CONFIG_LOGGING_SET;
-				}
-			} else {
-				/* Don't need this line's contents */
-				free(line.value);
-			}
-		}
+		if (line.parsed)
+			swb_config_load_option(cfg, line.key, line.value);
 		free(line.key);
 	}
 	parse_config_file_end();