projects / psensor.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
refactoring
[psensor.git] / src / cfg.c
diff --git a/src/cfg.c b/src/cfg.c
index 606ecc4..548323a 100644 (file)
--- a/src/cfg.c
+++ b/src/cfg.c
@@ -71,7 +71,10 @@
 #define KEY_INTERFACE_WINDOW_DIVIDER_POS \
 "/apps/psensor/interface/window_divider_pos"
 
-GConfClient *client;
+#define KEY_INTERFACE_TEMPERATURE_UNIT \
+"/apps/psensor/interface/temperature_unit"
+
+static GConfClient *client;
 
 static char *get_string(char *key, char *default_value)
 {
@@ -92,7 +95,7 @@ static struct color *config_get_background_color()
 {
 
        char *scolor = get_string(KEY_GRAPH_BACKGROUND_COLOR,
-                                        DEFAULT_GRAPH_BACKGROUND_COLOR);
+                                 DEFAULT_GRAPH_BACKGROUND_COLOR);
 
        struct color *c = string_to_color(scolor);
 
@@ -145,7 +148,6 @@ static double config_get_graph_background_alpha()
        double a = gconf_client_get_float(client,
                                          KEY_GRAPH_BACKGROUND_ALPHA,
                                          NULL);
-
        if (a == 0)
                gconf_client_set_float(client,
                                       KEY_GRAPH_BACKGROUND_ALPHA, 1.0, NULL);
@@ -183,15 +185,17 @@ static void config_set_foreground_color(struct color *color)
        free(scolor);
 }
 
-static char *config_get_sensor_key(char *sensor_name)
+static char *get_sensor_att_key(const char *name, const char *att)
 {
-       char *escaped_name = gconf_escape_key(sensor_name, -1);
-       /* /apps/psensor/sensors/[sensor_name]/color */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 6 + 1);
+       char *esc_name, *key;
 
-       sprintf(key, "/apps/psensor/sensors/%s/color", escaped_name);
+       esc_name = gconf_escape_key(name, -1);
+       /* /apps/psensor/sensors/[esc_name]/[att] */
+       key = malloc(22 + 2 * strlen(esc_name) + 1 + strlen(att) + 1);
 
-       free(escaped_name);
+       sprintf(key, "/apps/psensor/sensors/%s/%s", esc_name, att);
+
+       free(esc_name);
 
        return key;
 }
@@ -199,13 +203,14 @@ static char *config_get_sensor_key(char *sensor_name)
 struct color *config_get_sensor_color(char *sensor_name,
                                      struct color *default_color)
 {
-       char *key = config_get_sensor_key(sensor_name);
+       char *key, *scolor;
+       struct color *color;
+
+       key = get_sensor_att_key(sensor_name, "color");
 
-       char *scolor = gconf_client_get_string(client,
-                                              key,
-                                              NULL);
+       scolor = gconf_client_get_string(client, key, NULL);
 
-       struct color *color = NULL;
+       color = NULL;
 
        if (scolor)
                color = string_to_color(scolor);
@@ -227,130 +232,152 @@ struct color *config_get_sensor_color(char *sensor_name,
 
 void config_set_sensor_color(char *sensor_name, struct color *color)
 {
-       char *key = config_get_sensor_key(sensor_name);
+       char *key, *scolor;
 
-       char *scolor = color_to_string(color);
+       key = get_sensor_att_key(sensor_name, "color");
+       scolor = color_to_string(color);
 
        gconf_client_set_string(client, key, scolor, NULL);
 
        free(scolor);
+       free(key);
 }
 
 int config_get_sensor_alarm_limit(char *sensor_name, int def)
 {
        int res;
-       char *escaped_name = gconf_escape_key(sensor_name, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sensor_name, -1);
        /* /apps/psensor/sensors/[sensor_name]/alarmlimit */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 10 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 10 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/alarmlimit", escaped_name);
 
        res = gconf_client_get_int(client, key, NULL);
 
        free(escaped_name);
+       free(key);
 
        return res ? res : def;
 }
 
 void config_set_sensor_alarm_limit(char *sensor_name, int alarm_limit)
 {
-       char *escaped_name = gconf_escape_key(sensor_name, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sensor_name, -1);
        /* /apps/psensor/sensors/[sensor_name]/alarmlimit */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 10 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 10 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/alarmlimit", escaped_name);
 
        gconf_client_set_int(client, key, alarm_limit, NULL);
 
        free(escaped_name);
+       free(key);
 }
 
 int config_get_sensor_alarm_enabled(char *sid)
 {
        gboolean res;
-       char *escaped_name = gconf_escape_key(sid, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sid, -1);
        /* /apps/psensor/sensors/[sensor_name]/alarmenabled */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 12 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 12 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/alarmenabled", escaped_name);
 
        res = gconf_client_get_bool(client, key, NULL);
 
        free(escaped_name);
+       free(key);
 
        return res == TRUE;
 }
 
 void config_set_sensor_alarm_enabled(char *sid, int enabled)
 {
-       char *escaped_name = gconf_escape_key(sid, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sid, -1);
        /* /apps/psensor/sensors/[sensor_name]/alarmenabled */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 12 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 12 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/alarmenabled", escaped_name);
 
        gconf_client_set_bool(client, key, enabled, NULL);
 
        free(escaped_name);
+       free(key);
 }
 
 int config_is_sensor_enabled(char *sid)
 {
        gboolean res;
-       char *escaped_name = gconf_escape_key(sid, -1);
-       /* /apps/psensor/sensors/[sensor_name]/enabled */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 7 + 1);
+       char *escaped_name, *key;
 
+       escaped_name = gconf_escape_key(sid, -1);
+       /* /apps/psensor/sensors/[sensor_name]/enabled */
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 7 + 1);
        sprintf(key, "/apps/psensor/sensors/%s/enabled", escaped_name);
 
        res = gconf_client_get_bool(client, key, NULL);
-
+       free(key);
        free(escaped_name);
 
        return res == TRUE;
-
 }
 
 void config_set_sensor_enabled(char *sid, int enabled)
 {
-       char *escaped_name = gconf_escape_key(sid, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sid, -1);
        /* /apps/psensor/sensors/[sensor_name]/enabled */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 7 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 7 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/enabled", escaped_name);
 
        gconf_client_set_bool(client, key, enabled, NULL);
 
        free(escaped_name);
+       free(key);
 }
 
 char *config_get_sensor_name(char *sid)
 {
-       char *res;
-       char *escaped_name = gconf_escape_key(sid, -1);
+       char *res, *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sid, -1);
        /* /apps/psensor/sensors/[sensor_name]/name */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 4 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 4 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/name", escaped_name);
 
        res = gconf_client_get_string(client, key, NULL);
 
        free(escaped_name);
+       free(key);
 
        return res;
 }
 
 void config_set_sensor_name(char *sid, const char *name)
 {
-       char *escaped_name = gconf_escape_key(sid, -1);
+       char *escaped_name, *key;
+
+       escaped_name = gconf_escape_key(sid, -1);
        /* /apps/psensor/sensors/[sensor_name]/name */
-       char *key = malloc(22 + 2 * strlen(escaped_name) + 1 + 4 + 1);
+       key = malloc(22 + 2 * strlen(escaped_name) + 1 + 4 + 1);
 
        sprintf(key, "/apps/psensor/sensors/%s/name", escaped_name);
 
        gconf_client_set_string(client, key, name, NULL);
 
        free(escaped_name);
+       free(key);
 }
 
 static int config_is_window_decoration_enabled()
@@ -399,9 +426,13 @@ static void config_set_window_keep_below_enabled(int enabled)
                                      FALSE, NULL);
 }
 
-void config_init()
+/*
+ * Initializes the GConf client.
+ */
+static void init()
 {
-       client = gconf_client_get_default();
+       if (!client)
+               client = gconf_client_get_default();
 }
 
 void config_cleanup()
@@ -416,6 +447,8 @@ struct config *config_load()
 {
        struct config *c;
 
+       init();
+
        c = malloc(sizeof(struct config));
 
        c->graph_bgcolor = config_get_background_color();
@@ -491,6 +524,9 @@ struct config *config_load()
                c->window_h = 200;
        }
 
+       c->temperature_unit = gconf_client_get_int
+               (client, KEY_INTERFACE_TEMPERATURE_UNIT, NULL);
+
        return c;
 }
 
@@ -553,4 +589,9 @@ void config_save(struct config *c)
                             KEY_INTERFACE_WINDOW_DIVIDER_POS,
                             c->window_divider_pos,
                             NULL);
+
+       gconf_client_set_int(client,
+                            KEY_INTERFACE_TEMPERATURE_UNIT,
+                            c->temperature_unit,
+                            NULL);
 }
psensor