Return a boolean from timeout func. Fix ported from gedit.

2005-07-06  Christian Persch  <chpe@cvs.gnome.org>

	* shell/ev-metadata-manager.c: (ev_metadata_manager_save):

	Return a boolean from timeout func. Fix ported from gedit.

diff --git a/ChangeLog b/ChangeLog
index db437f8acd65f1c9d47d89bcca75eb989fe20f5e..8544a85d6d127e208f83302108066d10aa4431e6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-07-06  Christian Persch  <chpe@cvs.gnome.org>
+
+       * shell/ev-metadata-manager.c: (ev_metadata_manager_save):
+
+       Return a boolean from timeout func. Fix ported from gedit.
+
 2005-07-07  Iñaki Larrañaga  <dooteo@euskalgnu.org>
 
        * configure.ac: Added "eu" (Basque) to ALL_LINGUAS.

diff --git a/shell/ev-metadata-manager.c b/shell/ev-metadata-manager.c
index be550c819763b18cb1eb98965705183ff94f976e..712f124f0b0da0e1603bed37bdf1412b0445bd07 100644 (file)
--- a/shell/ev-metadata-manager.c
+++ b/shell/ev-metadata-manager.c
@@ -67,7 +67,7 @@ struct _GeditMetadataManager
        GHashTable      *items;
 };
 
-static void ev_metadata_manager_save (gpointer data);
+static gboolean ev_metadata_manager_save (gpointer data);
 
 
 static GeditMetadataManager *ev_metadata_manager = NULL;
@@ -516,7 +516,7 @@ resize_items ()
        }
 }
 
-static void
+static gboolean
 ev_metadata_manager_save (gpointer data)
 {      
        xmlDocPtr  doc;
@@ -524,7 +524,7 @@ ev_metadata_manager_save (gpointer data)
        gchar *file_name;
 
        if (!ev_metadata_manager->modified)
-               return;
+               return TRUE;
 
        resize_items ();
                
@@ -532,7 +532,7 @@ ev_metadata_manager_save (gpointer data)
 
        doc = xmlNewDoc ((const xmlChar *)"1.0");
        if (doc == NULL)
-               return;
+               return TRUE;
 
        /* Create metadata root */
        root = xmlNewDocNode (doc, NULL, (const xmlChar *)"metadata", NULL);
@@ -549,6 +549,8 @@ ev_metadata_manager_save (gpointer data)
        xmlFreeDoc (doc); 
 
        ev_metadata_manager->modified = FALSE;
+
+       return TRUE;
 }
 
 void