X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fui%2Fgui%2Fpsppire-conf.c;h=d0fc6e9de712ca11ba90c39f6d3b73d1564c1c86;hb=53241f59ac766997bf67870639f70afb82396640;hp=ed8077db2d8c52933e3b04455cf92221f3488337;hpb=158b6b7f7f3c1ea4ea4443ecb87d2a980808577b;p=pspp

diff --git a/src/ui/gui/psppire-conf.c b/src/ui/gui/psppire-conf.c
index ed8077db2d..d0fc6e9de7 100644
--- a/src/ui/gui/psppire-conf.c
+++ b/src/ui/gui/psppire-conf.c
@@ -28,44 +28,13 @@
 
 #include "psppire-conf.h"
 
-static void psppire_conf_init            (PsppireConf      *conf);
-static void psppire_conf_class_init      (PsppireConfClass *class);
+G_DEFINE_TYPE (PsppireConf, psppire_conf, G_TYPE_OBJECT)
 
 static void psppire_conf_finalize        (GObject   *object);
 static void psppire_conf_dispose        (GObject   *object);
 
 static GObjectClass *parent_class = NULL;
 
-
-GType
-psppire_conf_get_type (void)
-{
-  static GType conf_type = 0;
-
-  if (!conf_type)
-    {
-      static const GTypeInfo conf_info =
-      {
-	sizeof (PsppireConfClass),
-	NULL,		/* base_init */
-	NULL,		/* base_finalize */
-        (GClassInitFunc) psppire_conf_class_init,
-	NULL,		/* class_finalize */
-	NULL,		/* class_data */
-        sizeof (PsppireConf),
-	0,
-        (GInstanceInitFunc) psppire_conf_init,
-      };
-
-      conf_type = g_type_register_static (G_TYPE_OBJECT,
-						"PsppireConf",
-						&conf_info, 0);
-    }
-
-  return conf_type;
-}
-
-
 static void
 conf_read (PsppireConf *conf)
 {
@@ -81,10 +50,12 @@ flush_conf (PsppireConf *conf)
   gsize length = 0;
 
   gchar *kf = g_key_file_to_data  (conf->keyfile, &length, NULL);
+  GError *err = NULL;
 
-  if ( ! g_file_set_contents (conf->filename, kf, length, NULL) )
+  if (! g_file_set_contents (conf->filename, kf, length, &err))
     {
-      g_warning ("Cannot open %s for writing", conf->filename);
+      g_warning ("Cannot open %s for writing: %s", conf->filename, err->message);
+      g_error_free (err);
     }
 
   g_free (kf);
@@ -95,8 +66,8 @@ flush_conf (PsppireConf *conf)
 static void
 conf_write (PsppireConf *conf)
 {
-  if ( conf->idle == 0)
-    conf->idle = g_idle_add_full (G_PRIORITY_LOW, 
+  if (conf->idle == 0)
+    conf->idle = g_idle_add_full (G_PRIORITY_LOW,
 				  (GSourceFunc) flush_conf, conf, NULL);
 }
 
@@ -143,12 +114,11 @@ psppire_conf_class_init (PsppireConfClass *class)
   GObjectClass *object_class;
 
   parent_class = g_type_class_peek_parent (class);
-  object_class = (GObjectClass*) class;
+  object_class = G_OBJECT_CLASS (class);
 
   object_class->finalize = psppire_conf_finalize;
   object_class->dispose = psppire_conf_dispose;
   object_class->constructor = psppire_conf_construct;
-
 }
 
 
@@ -169,7 +139,6 @@ psppire_conf_init (PsppireConf *conf)
 
   conf->keyfile = g_key_file_new ();
 
-  conf->dispose_has_run = FALSE;
   conf->idle = 0;
 }
 
@@ -194,7 +163,7 @@ psppire_conf_get_int (PsppireConf *conf, const gchar *base,
 				   name, &err);
 
   ok = (err == NULL);
-  if ( err != NULL )
+  if (err != NULL)
     g_error_free (err);
 
   return ok;
@@ -213,7 +182,7 @@ psppire_conf_get_boolean (PsppireConf *conf, const gchar *base,
 			      name, &err);
 
   ok = (err == NULL);
-  if ( err != NULL )
+  if (err != NULL)
     g_error_free (err);
 
   if (ok)
@@ -237,7 +206,7 @@ psppire_conf_get_string (PsppireConf *conf, const gchar *base,
 			     name, &err);
 
   ok = (err == NULL);
-  if ( err != NULL )
+  if (err != NULL)
     g_error_free (err);
 
   if (ok)
@@ -262,7 +231,7 @@ psppire_conf_get_variant (PsppireConf *conf, const gchar *base,
 			     name, &err);
 
   ok = (err == NULL);
-  if ( err != NULL )
+  if (err != NULL)
     g_error_free (err);
 
   if (ok)
@@ -289,7 +258,7 @@ psppire_conf_get_enum (PsppireConf *conf, const gchar *base,
 			     name, &err);
 
   ok = (err == NULL);
-  if ( err != NULL )
+  if (err != NULL)
     g_error_free (err);
 
   if (ok)
@@ -351,12 +320,12 @@ psppire_conf_set_enum (PsppireConf *conf,
 {
   GEnumClass *ec = g_type_class_ref (enum_type);
   GEnumValue *ev = g_enum_get_value (ec, value);
-  
+
   g_key_file_set_string (conf->keyfile, base, name,
 			 ev->value_nick);
 
   g_type_class_unref (ec);
-  
+
   conf_write (conf);
 }
 
@@ -377,19 +346,19 @@ psppire_conf_set_window_geometry (PsppireConf *conf,
 
   if (psppire_conf_get_int (conf, base, "height", &height)
       &&
-      psppire_conf_get_int (conf, base, "width", &width) )
+      psppire_conf_get_int (conf, base, "width", &width))
     {
       gtk_window_set_default_size (window, width, height);
     }
 
-  if ( psppire_conf_get_int (conf, base, "x", &x)
+  if (psppire_conf_get_int (conf, base, "x", &x)
        &&
-       psppire_conf_get_int (conf, base, "y", &y) )
+       psppire_conf_get_int (conf, base, "y", &y))
     {
       gtk_window_move (window, x, y);
     }
 
-  if ( psppire_conf_get_boolean (conf, base, "maximize", &maximize))
+  if (psppire_conf_get_boolean (conf, base, "maximize", &maximize))
     {
       if (maximize)
 	gtk_window_maximize (window);