fixed memory leaks (gconf keys)
[psensor.git] / src / cfg.c
index 73cc7e8..daa0a89 100644 (file)
--- a/src/cfg.c
+++ b/src/cfg.c
@@ -1,22 +1,21 @@
 /*
-    Copyright (C) 2010-2011 jeanfi@gmail.com
-
-    This program is free software; you can redistribute it and/or modify
-    it under the terms of the GNU General Public License as published by
-    the Free Software Foundation; either version 2 of the License, or
-    (at your option) any later version.
-
-    This program is distributed in the hope that it will be useful,
-    but WITHOUT ANY WARRANTY; without even the implied warranty of
-    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-    GNU General Public License for more details.
-
-    You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
-    02110-1301 USA
-*/
-
+ * Copyright (C) 2010-2012 jeanfi@gmail.com
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
+ * 02110-1301 USA
+ */
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 #define KEY_INTERFACE_WINDOW_W "/apps/psensor/interface/window_w"
 #define KEY_INTERFACE_WINDOW_H "/apps/psensor/interface/window_h"
 
+#define KEY_INTERFACE_WINDOW_DIVIDER_POS \
+"/apps/psensor/interface/window_divider_pos"
+
+#define KEY_INTERFACE_TEMPERATURE_UNIT \
+"/apps/psensor/interface/temperature_unit"
 
 GConfClient *client;
 
@@ -144,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);
@@ -226,84 +229,99 @@ 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 = config_get_sensor_key(sensor_name);
+       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;
@@ -312,44 +330,52 @@ int config_is_sensor_enabled(char *sid)
 
 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()
@@ -480,12 +506,19 @@ struct config *config_load()
        c->window_h = gconf_client_get_int(client,
                                           KEY_INTERFACE_WINDOW_H,
                                           NULL);
+       c->window_divider_pos
+               = gconf_client_get_int(client,
+                                      KEY_INTERFACE_WINDOW_DIVIDER_POS,
+                                      NULL);
 
-       if (!c->window_w || !c->window_h) {
+       if (!c->window_restore_enabled || !c->window_w || !c->window_h) {
                c->window_w = 800;
                c->window_h = 200;
        }
 
+       c->temperature_unit = gconf_client_get_int
+               (client, KEY_INTERFACE_TEMPERATURE_UNIT, NULL);
+
        return c;
 }
 
@@ -543,4 +576,15 @@ void config_save(struct config *c)
                             KEY_INTERFACE_WINDOW_H,
                             c->window_h,
                             NULL);
+
+       gconf_client_set_int(client,
+                            KEY_INTERFACE_WINDOW_DIVIDER_POS,
+                            c->window_divider_pos,
+                            NULL);
+
+       gconf_client_set_int(client,
+                            KEY_INTERFACE_TEMPERATURE_UNIT,
+                            c->temperature_unit,
+                            NULL);
+
 }