Imported Upstream version 2.61.2
[platform/upstream/glib.git] / glib / gbookmarkfile.c
index 596e370..88808e7 100644 (file)
@@ -162,24 +162,24 @@ struct _BookmarkAppInfo
 {
   gchar *name;
   gchar *exec;
-  
+
   guint count;
-  
+
   time_t stamp;
 };
 
 struct _BookmarkMetadata
 {
   gchar *mime_type;
-  
+
   GList *groups;
-  
+
   GList *applications;
   GHashTable *apps_by_name;
-  
+
   gchar *icon_href;
   gchar *icon_mime;
-  
+
   guint is_private : 1;
 };
 
@@ -210,10 +210,10 @@ struct _GBookmarkFile
 };
 
 /* parser state machine */
-enum
+typedef enum
 {
   STATE_STARTED        = 0,
-  
+
   STATE_ROOT,
   STATE_BOOKMARK,
   STATE_TITLE,
@@ -226,9 +226,9 @@ enum
   STATE_GROUP,
   STATE_MIME,
   STATE_ICON,
-  
+
   STATE_FINISHED
-};
+} ParserState;
 
 static void          g_bookmark_file_init        (GBookmarkFile  *bookmark);
 static void          g_bookmark_file_clear       (GBookmarkFile  *bookmark);
@@ -244,9 +244,11 @@ static BookmarkItem *g_bookmark_file_lookup_item (GBookmarkFile  *bookmark,
 static void          g_bookmark_file_add_item    (GBookmarkFile  *bookmark,
                                                  BookmarkItem   *item,
                                                  GError        **error);
-                                                      
-static time_t  timestamp_from_iso8601 (const gchar *iso_date);
-static gchar * timestamp_to_iso8601   (time_t       timestamp);
+
+static gboolean  timestamp_from_iso8601 (const gchar  *iso_date,
+                                         time_t       *out_timestamp,
+                                         GError      **error);
+static gchar    *timestamp_to_iso8601   (time_t        timestamp);
 
 /********************************
  * BookmarkAppInfo              *
@@ -257,16 +259,16 @@ static BookmarkAppInfo *
 bookmark_app_info_new (const gchar *name)
 {
   BookmarkAppInfo *retval;
+
   g_warn_if_fail (name != NULL);
-  
+
   retval = g_slice_new (BookmarkAppInfo);
-  
+
   retval->name = g_strdup (name);
   retval->exec = NULL;
   retval->count = 0;
   retval->stamp = 0;
-  
+
   return retval;
 }
 
@@ -275,10 +277,10 @@ bookmark_app_info_free (BookmarkAppInfo *app_info)
 {
   if (!app_info)
     return;
-  
+
   g_free (app_info->name);
   g_free (app_info->exec);
-  
+
   g_slice_free (BookmarkAppInfo, app_info);
 }
 
@@ -324,24 +326,24 @@ static BookmarkMetadata *
 bookmark_metadata_new (void)
 {
   BookmarkMetadata *retval;
-  
+
   retval = g_slice_new (BookmarkMetadata);
 
   retval->mime_type = NULL;
-  
+
   retval->groups = NULL;
-  
+
   retval->applications = NULL;
   retval->apps_by_name = g_hash_table_new_full (g_str_hash,
                                                 g_str_equal,
                                                 NULL,
                                                 NULL);
-  
+
   retval->is_private = FALSE;
-  
+
   retval->icon_href = NULL;
   retval->icon_mime = NULL;
-  
+
   return retval;
 }
 
@@ -350,9 +352,9 @@ bookmark_metadata_free (BookmarkMetadata *metadata)
 {
   if (!metadata)
     return;
-  
+
   g_free (metadata->mime_type);
-    
+
   g_list_free_full (metadata->groups, g_free);
   g_list_free_full (metadata->applications, (GDestroyNotify) bookmark_app_info_free);
 
@@ -360,7 +362,7 @@ bookmark_metadata_free (BookmarkMetadata *metadata)
 
   g_free (metadata->icon_href);
   g_free (metadata->icon_mime);
-  
+
   g_slice_free (BookmarkMetadata, metadata);
 }
 
@@ -369,12 +371,12 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
 {
   GString *retval;
   gchar *buffer;
+
   if (!metadata->applications)
     return NULL;
-  
+
   retval = g_string_sized_new (1024);
-  
+
   /* metadata container */
   g_string_append (retval,
                   "      "
@@ -387,7 +389,7 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
     buffer = g_strconcat ("        "
                          "<" MIME_NAMESPACE_NAME ":" MIME_TYPE_ELEMENT " "
                          MIME_TYPE_ATTRIBUTE "=\"", metadata->mime_type, "\"/>\n",
-                         NULL);    
+                         NULL);
     g_string_append (retval, buffer);
     g_free (buffer);
   }
@@ -395,13 +397,13 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
   if (metadata->groups)
     {
       GList *l;
-      
+
       /* open groups container */
       g_string_append (retval,
                       "        "
                       "<" BOOKMARK_NAMESPACE_NAME
                       ":" BOOKMARK_GROUPS_ELEMENT ">\n");
-      
+
       for (l = g_list_last (metadata->groups); l != NULL; l = l->prev)
         {
           gchar *group_name;
@@ -418,31 +420,31 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
          g_free (buffer);
          g_free (group_name);
         }
-      
+
       /* close groups container */
       g_string_append (retval,
                       "        "
                       "</" BOOKMARK_NAMESPACE_NAME
                       ":" BOOKMARK_GROUPS_ELEMENT ">\n");
     }
-  
+
   if (metadata->applications)
     {
       GList *l;
-      
+
       /* open applications container */
       g_string_append (retval,
                       "        "
                       "<" BOOKMARK_NAMESPACE_NAME
                       ":" BOOKMARK_APPLICATIONS_ELEMENT ">\n");
-      
+
       for (l = g_list_last (metadata->applications); l != NULL; l = l->prev)
         {
           BookmarkAppInfo *app_info = (BookmarkAppInfo *) l->data;
           gchar *app_data;
 
          g_warn_if_fail (app_info != NULL);
-          
+
           app_data = bookmark_app_info_dump (app_info);
 
          if (app_data)
@@ -452,14 +454,14 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
              g_free (app_data);
            }
         }
-      
+
       /* close applications container */
       g_string_append (retval,
                       "        "
                       "</" BOOKMARK_NAMESPACE_NAME
                       ":" BOOKMARK_APPLICATIONS_ELEMENT ">\n");
     }
-  
+
   /* icon */
   if (metadata->icon_href)
     {
@@ -475,19 +477,19 @@ bookmark_metadata_dump (BookmarkMetadata *metadata)
 
       g_free (buffer);
     }
-  
+
   /* private hint */
   if (metadata->is_private)
     g_string_append (retval,
                     "        "
                     "<" BOOKMARK_NAMESPACE_NAME
                     ":" BOOKMARK_PRIVATE_ELEMENT "/>\n");
-  
+
   /* close metadata container */
   g_string_append (retval,
                   "      "
                   "</" XBEL_METADATA_ELEMENT ">\n");
-                          
+
   return g_string_free (retval, FALSE);
 }
 
@@ -500,21 +502,21 @@ static BookmarkItem *
 bookmark_item_new (const gchar *uri)
 {
   BookmarkItem *item;
+
   g_warn_if_fail (uri != NULL);
-  
+
   item = g_slice_new (BookmarkItem);
   item->uri = g_strdup (uri);
-  
+
   item->title = NULL;
   item->description = NULL;
-  
+
   item->added = (time_t) -1;
   item->modified = (time_t) -1;
   item->visited = (time_t) -1;
-  
+
   item->metadata = NULL;
-  
+
   return item;
 }
 
@@ -527,10 +529,10 @@ bookmark_item_free (BookmarkItem *item)
   g_free (item->uri);
   g_free (item->title);
   g_free (item->description);
-  
+
   if (item->metadata)
     bookmark_metadata_free (item->metadata);
-  
+
   g_slice_free (BookmarkItem, item);
 }
 
@@ -541,18 +543,18 @@ bookmark_item_dump (BookmarkItem *item)
   gchar *added, *visited, *modified;
   gchar *escaped_uri;
   gchar *buffer;
+
   /* at this point, we must have at least a registered application; if we don't
    * we don't screw up the bookmark file, and just skip this item
    */
   if (!item->metadata || !item->metadata->applications)
     {
-      g_warning ("Item for URI '%s' has no registered applications: skipping.\n", item->uri);
+      g_warning ("Item for URI '%s' has no registered applications: skipping.", item->uri);
       return NULL;
     }
-  
+
   retval = g_string_sized_new (4096);
-  
+
   added = timestamp_to_iso8601 (item->added);
   modified = timestamp_to_iso8601 (item->modified);
   visited = timestamp_to_iso8601 (item->visited);
@@ -575,11 +577,11 @@ bookmark_item_dump (BookmarkItem *item)
   g_free (modified);
   g_free (added);
   g_free (buffer);
-  
+
   if (item->title)
     {
       gchar *escaped_title;
-      
+
       escaped_title = g_markup_escape_text (item->title, -1);
       buffer = g_strconcat ("    "
                             "<" XBEL_TITLE_ELEMENT ">",
@@ -591,11 +593,11 @@ bookmark_item_dump (BookmarkItem *item)
       g_free (escaped_title);
       g_free (buffer);
     }
-  
+
   if (item->description)
     {
       gchar *escaped_desc;
-      
+
       escaped_desc = g_markup_escape_text (item->description, -1);
       buffer = g_strconcat ("    "
                             "<" XBEL_DESC_ELEMENT ">",
@@ -607,11 +609,11 @@ bookmark_item_dump (BookmarkItem *item)
       g_free (escaped_desc);
       g_free (buffer);
     }
-  
+
   if (item->metadata)
     {
       gchar *metadata;
-      
+
       metadata = bookmark_metadata_dump (item->metadata);
       if (metadata)
         {
@@ -629,7 +631,7 @@ bookmark_item_dump (BookmarkItem *item)
     }
 
   g_string_append (retval, "  </" XBEL_BOOKMARK_ELEMENT ">\n");
-  
+
   return g_string_free (retval, FALSE);
 }
 
@@ -641,20 +643,20 @@ bookmark_item_lookup_app_info (BookmarkItem *item,
 
   if (!item->metadata)
     return NULL;
-  
+
   return g_hash_table_lookup (item->metadata->apps_by_name, app_name);
 }
 
 /*************************
  *    GBookmarkFile    *
  *************************/
+
 static void
 g_bookmark_file_init (GBookmarkFile *bookmark)
 {
   bookmark->title = NULL;
   bookmark->description = NULL;
-  
+
   bookmark->items = NULL;
   bookmark->items_by_uri = g_hash_table_new_full (g_str_hash,
                                                   g_str_equal,
@@ -670,21 +672,21 @@ g_bookmark_file_clear (GBookmarkFile *bookmark)
 
   g_list_free_full (bookmark->items, (GDestroyNotify) bookmark_item_free);
   bookmark->items = NULL;
-  
+
   if (bookmark->items_by_uri)
     {
       g_hash_table_destroy (bookmark->items_by_uri);
-      
+
       bookmark->items_by_uri = NULL;
     }
 }
 
 struct _ParseData
 {
-  gint state;
-  
+  ParserState state;
+
   GHashTable *namespaces;
-  
+
   GBookmarkFile *bookmark_file;
   BookmarkItem *current_item;
 };
@@ -693,16 +695,16 @@ static ParseData *
 parse_data_new (void)
 {
   ParseData *retval;
-  
+
   retval = g_new (ParseData, 1);
-  
+
   retval->state = STATE_STARTED;
   retval->namespaces = g_hash_table_new_full (g_str_hash, g_str_equal,
                                              (GDestroyNotify) g_free,
                                              (GDestroyNotify) g_free);
   retval->bookmark_file = NULL;
   retval->current_item = NULL;
-  
+
   return retval;
 }
 
@@ -710,7 +712,7 @@ static void
 parse_data_free (ParseData *parse_data)
 {
   g_hash_table_destroy (parse_data->namespaces);
-  
+
   g_free (parse_data);
 }
 
@@ -728,9 +730,9 @@ parse_bookmark_element (GMarkupParseContext  *context,
   gint i;
   BookmarkItem *item;
   GError *add_error;
+
   g_warn_if_fail ((parse_data != NULL) && (parse_data->state == STATE_BOOKMARK));
-  
+
   i = 0;
   uri = added = modified = visited = NULL;
   for (attr = attribute_names[i]; attr != NULL; attr = attribute_names[++i])
@@ -757,7 +759,7 @@ parse_bookmark_element (GMarkupParseContext  *context,
           return;
         }
     }
-  
+
   if (!uri)
     {
       g_set_error (error, G_MARKUP_ERROR,
@@ -767,19 +769,19 @@ parse_bookmark_element (GMarkupParseContext  *context,
                   XBEL_BOOKMARK_ELEMENT);
       return;
     }
-  
+
   g_warn_if_fail (parse_data->current_item == NULL);
-  
+
   item = bookmark_item_new (uri);
-  
-  if (added)
-    item->added = timestamp_from_iso8601 (added);
-  
-  if (modified)
-    item->modified = timestamp_from_iso8601 (modified);
-  
-  if (visited)
-    item->visited = timestamp_from_iso8601 (visited);
+
+  if (added != NULL && !timestamp_from_iso8601 (added, &item->added, error))
+    return;
+
+  if (modified != NULL && !timestamp_from_iso8601 (modified, &item->modified, error))
+    return;
+
+  if (visited != NULL && !timestamp_from_iso8601 (visited, &item->visited, error))
+    return;
 
   add_error = NULL;
   g_bookmark_file_add_item (parse_data->bookmark_file,
@@ -788,12 +790,12 @@ parse_bookmark_element (GMarkupParseContext  *context,
   if (add_error)
     {
       bookmark_item_free (item);
-      
+
       g_propagate_error (error, add_error);
-      
+
       return;
-    }                                
-  
+    }
+
   parse_data->current_item = item;
 }
 
@@ -809,7 +811,7 @@ parse_application_element (GMarkupParseContext  *context,
   gint i;
   BookmarkItem *item;
   BookmarkAppInfo *ai;
-  
+
   g_warn_if_fail ((parse_data != NULL) && (parse_data->state == STATE_APPLICATION));
 
   i = 0;
@@ -838,7 +840,7 @@ parse_application_element (GMarkupParseContext  *context,
                   BOOKMARK_APPLICATION_ELEMENT);
       return;
     }
-  
+
   if (!exec)
     {
       g_set_error (error, G_MARKUP_ERROR,
@@ -849,30 +851,34 @@ parse_application_element (GMarkupParseContext  *context,
       return;
     }
 
-  g_warn_if_fail (parse_data->current_item != NULL);  
+  g_warn_if_fail (parse_data->current_item != NULL);
   item = parse_data->current_item;
-    
+
   ai = bookmark_item_lookup_app_info (item, name);
   if (!ai)
     {
       ai = bookmark_app_info_new (name);
-      
+
       if (!item->metadata)
        item->metadata = bookmark_metadata_new ();
-      
+
       item->metadata->applications = g_list_prepend (item->metadata->applications, ai);
       g_hash_table_replace (item->metadata->apps_by_name, ai->name, ai);
     }
-      
+
+  g_free (ai->exec);
   ai->exec = g_strdup (exec);
-  
+
   if (count)
     ai->count = atoi (count);
   else
     ai->count = 1;
 
-  if (modified)
-    ai->stamp = timestamp_from_iso8601 (modified);
+  if (modified != NULL)
+    {
+      if (!timestamp_from_iso8601 (modified, &ai->stamp, error))
+        return;
+    }
   else
     {
       /* the timestamp attribute has been deprecated but we still parse
@@ -896,9 +902,9 @@ parse_mime_type_element (GMarkupParseContext  *context,
   const gchar *attr;
   gint i;
   BookmarkItem *item;
-  
+
   g_warn_if_fail ((parse_data != NULL) && (parse_data->state == STATE_MIME));
-  
+
   i = 0;
   type = NULL;
   for (attr = attribute_names[i]; attr != NULL; attr = attribute_names[++i])
@@ -910,12 +916,13 @@ parse_mime_type_element (GMarkupParseContext  *context,
   if (!type)
     type = "application/octet-stream";
 
-  g_warn_if_fail (parse_data->current_item != NULL);  
+  g_warn_if_fail (parse_data->current_item != NULL);
   item = parse_data->current_item;
-    
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
+  g_free (item->metadata->mime_type);
   item->metadata->mime_type = g_strdup (type);
 }
 
@@ -931,9 +938,9 @@ parse_icon_element (GMarkupParseContext  *context,
   const gchar *attr;
   gint i;
   BookmarkItem *item;
-  
+
   g_warn_if_fail ((parse_data != NULL) && (parse_data->state == STATE_ICON));
-  
+
   i = 0;
   href = NULL;
   type = NULL;
@@ -945,7 +952,7 @@ parse_icon_element (GMarkupParseContext  *context,
         type = attribute_values[i];
     }
 
-  /* the "href" attribute is mandatory */       
+  /* the "href" attribute is mandatory */
   if (!href)
     {
       g_set_error (error, G_MARKUP_ERROR,
@@ -959,12 +966,14 @@ parse_icon_element (GMarkupParseContext  *context,
   if (!type)
     type = "application/octet-stream";
 
-  g_warn_if_fail (parse_data->current_item != NULL);  
+  g_warn_if_fail (parse_data->current_item != NULL);
   item = parse_data->current_item;
-    
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
+  g_free (item->metadata->icon_href);
+  g_free (item->metadata->icon_mime);
   item->metadata->icon_href = g_strdup (href);
   item->metadata->icon_mime = g_strdup (type);
 }
@@ -976,7 +985,7 @@ parse_icon_element (GMarkupParseContext  *context,
  *
  * FIXME: this works on the assumption that the generator of the XBEL file
  * is either this code or is smart enough to place the namespace declarations
- * inside the main root node or inside the metadata node and does not redefine 
+ * inside the main root node or inside the metadata node and does not redefine
  * a namespace inside an inner node; this does *not* conform to the
  * XML-NS standard, although is a close approximation.  In order to make this
  * conformant to the XML-NS specification we should use a per-element
@@ -989,12 +998,12 @@ map_namespace_to_name (ParseData    *parse_data,
 {
   const gchar *attr;
   gint i;
+
   g_warn_if_fail (parse_data != NULL);
-  
+
   if (!attribute_names || !attribute_names[0])
     return;
-  
+
   i = 0;
   for (attr = attribute_names[i]; attr; attr = attribute_names[++i])
     {
@@ -1002,16 +1011,16 @@ map_namespace_to_name (ParseData    *parse_data,
         {
           gchar *namespace_name, *namespace_uri;
           gchar *p;
-          
+
           p = g_utf8_strchr (attr, -1, ':');
           if (p)
             p = g_utf8_next_char (p);
           else
             p = "default";
-          
+
           namespace_name = g_strdup (p);
           namespace_uri = g_strdup (attribute_values[i]);
-          
+
           g_hash_table_replace (parse_data->namespaces,
                                 namespace_name,
                                 namespace_uri);
@@ -1035,17 +1044,17 @@ is_element_full (ParseData   *parse_data,
   gchar *ns_uri, *ns_name;
   const gchar *p, *element_name;
   gboolean retval;
+
   g_warn_if_fail (parse_data != NULL);
   g_warn_if_fail (element_full != NULL);
-  
+
   if (!element)
     return FALSE;
-    
+
   /* no namespace requested: dumb element compare */
   if (!namespace)
     return (0 == strcmp (element_full, element));
-  
+
   /* search for namespace separator; if none found, assume we are under the
    * default namespace, and set ns_name to our "default" marker; if no default
    * namespace has been set, just do a plain comparison between @full_element
@@ -1062,27 +1071,64 @@ is_element_full (ParseData   *parse_data,
       ns_name = g_strdup ("default");
       element_name = element_full;
     }
-  
-  ns_uri = g_hash_table_lookup (parse_data->namespaces, ns_name);  
+
+  ns_uri = g_hash_table_lookup (parse_data->namespaces, ns_name);
   if (!ns_uri)
     {
       /* no default namespace found */
       g_free (ns_name);
-      
+
       return (0 == strcmp (element_full, element));
     }
 
   retval = (0 == strcmp (ns_uri, namespace) &&
             0 == strcmp (element_name, element));
-  
+
   g_free (ns_name);
-  
+
   return retval;
 }
 
 #define IS_ELEMENT(p,s,e)      (is_element_full ((p), (s), NULL, (e), '\0'))
 #define IS_ELEMENT_NS(p,s,n,e) (is_element_full ((p), (s), (n), (e), '|'))
 
+static const gchar *
+parser_state_to_element_name (ParserState state)
+{
+  switch (state)
+    {
+    case STATE_STARTED:
+    case STATE_FINISHED:
+      return "(top-level)";
+    case STATE_ROOT:
+      return XBEL_ROOT_ELEMENT;
+    case STATE_BOOKMARK:
+      return XBEL_BOOKMARK_ELEMENT;
+    case STATE_TITLE:
+      return XBEL_TITLE_ELEMENT;
+    case STATE_DESC:
+      return XBEL_DESC_ELEMENT;
+    case STATE_INFO:
+      return XBEL_INFO_ELEMENT;
+    case STATE_METADATA:
+      return XBEL_METADATA_ELEMENT;
+    case STATE_APPLICATIONS:
+      return BOOKMARK_APPLICATIONS_ELEMENT;
+    case STATE_APPLICATION:
+      return BOOKMARK_APPLICATION_ELEMENT;
+    case STATE_GROUPS:
+      return BOOKMARK_GROUPS_ELEMENT;
+    case STATE_GROUP:
+      return BOOKMARK_GROUP_ELEMENT;
+    case STATE_MIME:
+      return MIME_TYPE_ELEMENT;
+    case STATE_ICON:
+      return BOOKMARK_ICON_ELEMENT;
+    default:
+      g_assert_not_reached ();
+    }
+}
+
 static void
 start_element_raw_cb (GMarkupParseContext *context,
                       const gchar         *element_name,
@@ -1094,13 +1140,13 @@ start_element_raw_cb (GMarkupParseContext *context,
   ParseData *parse_data = (ParseData *) user_data;
 
   /* we must check for namespace declarations first
-   * 
+   *
    * XXX - we could speed up things by checking for namespace declarations
    * only on the root node, where they usually are; this would probably break
    * on streams not produced by us or by "smart" generators
    */
   map_namespace_to_name (parse_data, attribute_names, attribute_values);
-  
+
   switch (parse_data->state)
     {
     case STATE_STARTED:
@@ -1108,7 +1154,7 @@ start_element_raw_cb (GMarkupParseContext *context,
         {
           const gchar *attr;
           gint i;
-          
+
           i = 0;
           for (attr = attribute_names[i]; attr; attr = attribute_names[++i])
             {
@@ -1131,9 +1177,9 @@ start_element_raw_cb (GMarkupParseContext *context,
       else if (IS_ELEMENT (parse_data, element_name, XBEL_BOOKMARK_ELEMENT))
         {
           GError *inner_error = NULL;
-          
+
           parse_data->state = STATE_BOOKMARK;
-          
+
           parse_bookmark_element (context,
                                  parse_data,
                                  attribute_names,
@@ -1168,7 +1214,7 @@ start_element_raw_cb (GMarkupParseContext *context,
         {
           const gchar *attr;
           gint i;
-          
+
           i = 0;
           for (attr = attribute_names[i]; attr; attr = attribute_names[++i])
             {
@@ -1176,7 +1222,7 @@ start_element_raw_cb (GMarkupParseContext *context,
                   (0 == strcmp (attribute_values[i], BOOKMARK_METADATA_OWNER)))
                 {
                   parse_data->state = STATE_METADATA;
-                  
+
                   if (!parse_data->current_item->metadata)
                     parse_data->current_item->metadata = bookmark_metadata_new ();
                 }
@@ -1199,9 +1245,9 @@ start_element_raw_cb (GMarkupParseContext *context,
       else if (IS_ELEMENT_NS (parse_data, element_name, BOOKMARK_NAMESPACE_URI, BOOKMARK_ICON_ELEMENT))
         {
           GError *inner_error = NULL;
-          
+
          parse_data->state = STATE_ICON;
-          
+
           parse_icon_element (context,
                              parse_data,
                              attribute_names,
@@ -1213,9 +1259,9 @@ start_element_raw_cb (GMarkupParseContext *context,
       else if (IS_ELEMENT_NS (parse_data, element_name, MIME_NAMESPACE_URI, MIME_TYPE_ELEMENT))
         {
           GError *inner_error = NULL;
-          
+
           parse_data->state = STATE_MIME;
-          
+
           parse_mime_type_element (context,
                                   parse_data,
                                   attribute_names,
@@ -1235,9 +1281,9 @@ start_element_raw_cb (GMarkupParseContext *context,
       if (IS_ELEMENT_NS (parse_data, element_name, BOOKMARK_NAMESPACE_URI, BOOKMARK_APPLICATION_ELEMENT))
         {
           GError *inner_error = NULL;
-          
+
           parse_data->state = STATE_APPLICATION;
-          
+
           parse_application_element (context,
                                     parse_data,
                                     attribute_names,
@@ -1263,8 +1309,23 @@ start_element_raw_cb (GMarkupParseContext *context,
                     element_name,
                     BOOKMARK_GROUP_ELEMENT);
       break;
+
+    case STATE_TITLE:
+    case STATE_DESC:
+    case STATE_APPLICATION:
+    case STATE_GROUP:
+    case STATE_MIME:
+    case STATE_ICON:
+    case STATE_FINISHED:
+      g_set_error (error, G_MARKUP_ERROR,
+                   G_MARKUP_ERROR_INVALID_CONTENT,
+                   _("Unexpected tag “%s” inside “%s”"),
+                   element_name,
+                   parser_state_to_element_name (parse_data->state));
+      break;
+
     default:
-      g_warn_if_reached ();
+      g_assert_not_reached ();
       break;
     }
 }
@@ -1276,13 +1337,13 @@ end_element_raw_cb (GMarkupParseContext *context,
                     GError             **error)
 {
   ParseData *parse_data = (ParseData *) user_data;
-  
+
   if (IS_ELEMENT (parse_data, element_name, XBEL_ROOT_ELEMENT))
     parse_data->state = STATE_FINISHED;
   else if (IS_ELEMENT (parse_data, element_name, XBEL_BOOKMARK_ELEMENT))
     {
       parse_data->current_item = NULL;
-      
+
       parse_data->state = STATE_ROOT;
     }
   else if ((IS_ELEMENT (parse_data, element_name, XBEL_INFO_ELEMENT)) ||
@@ -1321,9 +1382,9 @@ text_raw_cb (GMarkupParseContext *context,
 {
   ParseData *parse_data = (ParseData *) user_data;
   gchar *payload;
-  
+
   payload = g_strndup (text, length);
-  
+
   switch (parse_data->state)
     {
     case STATE_TITLE:
@@ -1353,12 +1414,12 @@ text_raw_cb (GMarkupParseContext *context,
     case STATE_GROUP:
       {
       GList *groups;
-      
+
       g_warn_if_fail (parse_data->current_item != NULL);
-      
+
       if (!parse_data->current_item->metadata)
         parse_data->current_item->metadata = bookmark_metadata_new ();
-      
+
       groups = parse_data->current_item->metadata->groups;
       parse_data->current_item->metadata->groups = g_list_prepend (groups, g_strdup (payload));
       }
@@ -1377,7 +1438,7 @@ text_raw_cb (GMarkupParseContext *context,
       g_warn_if_reached ();
       break;
     }
-  
+
   g_free (payload);
 }
 
@@ -1400,7 +1461,7 @@ g_bookmark_file_parse (GBookmarkFile  *bookmark,
   ParseData *parse_data;
   GError *parse_error, *end_error;
   gboolean retval;
-  
+
   g_warn_if_fail (bookmark != NULL);
 
   if (!buffer)
@@ -1408,18 +1469,18 @@ g_bookmark_file_parse (GBookmarkFile  *bookmark,
 
   parse_error = NULL;
   end_error = NULL;
-  
+
   if (length == (gsize) -1)
     length = strlen (buffer);
 
   parse_data = parse_data_new ();
   parse_data->bookmark_file = bookmark;
-  
+
   context = g_markup_parse_context_new (&markup_parser,
                                        0,
                                        parse_data,
                                        (GDestroyNotify) parse_data_free);
-  
+
   retval = g_markup_parse_context_parse (context,
                                         buffer,
                                         length,
@@ -1432,7 +1493,7 @@ g_bookmark_file_parse (GBookmarkFile  *bookmark,
       if (!retval)
         g_propagate_error (error, end_error);
    }
+
   g_markup_parse_context_free (context);
 
   return retval;
@@ -1446,7 +1507,7 @@ g_bookmark_file_dump (GBookmarkFile  *bookmark,
   GString *retval;
   gchar *buffer;
   GList *l;
-  
+
   retval = g_string_sized_new (4096);
 
   g_string_append (retval,
@@ -1460,28 +1521,28 @@ g_bookmark_file_dump (GBookmarkFile  *bookmark,
                   "<" XBEL_ROOT_ELEMENT " " XBEL_VERSION_ATTRIBUTE "=\"" XBEL_VERSION "\"\n"
                   "      xmlns:" BOOKMARK_NAMESPACE_NAME "=\"" BOOKMARK_NAMESPACE_URI "\"\n"
                   "      xmlns:" MIME_NAMESPACE_NAME     "=\"" MIME_NAMESPACE_URI "\"\n>");
-  
+
   if (bookmark->title)
     {
       gchar *escaped_title;
+
       escaped_title = g_markup_escape_text (bookmark->title, -1);
 
       buffer = g_strconcat ("  "
                            "<" XBEL_TITLE_ELEMENT ">",
                            escaped_title,
                            "</" XBEL_TITLE_ELEMENT ">\n", NULL);
-      
+
       g_string_append (retval, buffer);
 
       g_free (buffer);
       g_free (escaped_title);
     }
-  
+
   if (bookmark->description)
     {
       gchar *escaped_desc;
+
       escaped_desc = g_markup_escape_text (bookmark->description, -1);
 
       buffer = g_strconcat ("  "
@@ -1493,7 +1554,7 @@ g_bookmark_file_dump (GBookmarkFile  *bookmark,
       g_free (buffer);
       g_free (escaped_desc);
     }
-  
+
   if (!bookmark->items)
     goto out;
   else
@@ -1506,57 +1567,61 @@ g_bookmark_file_dump (GBookmarkFile  *bookmark,
     {
       BookmarkItem *item = (BookmarkItem *) l->data;
       gchar *item_dump;
-      
+
       item_dump = bookmark_item_dump (item);
       if (!item_dump)
         continue;
-      
+
       retval = g_string_append (retval, item_dump);
-      
+
       g_free (item_dump);
     }
 
 out:
   g_string_append (retval, "</" XBEL_ROOT_ELEMENT ">");
-  
+
   if (length)
     *length = retval->len;
-  
+
   return g_string_free (retval, FALSE);
 }
 
 /**************
  *    Misc    *
  **************/
+
 /* converts a Unix timestamp in a ISO 8601 compliant string; you
  * should free the returned string.
  */
 static gchar *
 timestamp_to_iso8601 (time_t timestamp)
 {
-  GTimeVal stamp;
-
-  if (timestamp == (time_t) -1)
-    g_get_current_time (&stamp);
-  else
-    {
-      stamp.tv_sec = timestamp;
-      stamp.tv_usec = 0;
-    }
+  GDateTime *dt = g_date_time_new_from_unix_utc (timestamp);
+  gchar *iso8601_string = g_date_time_format_iso8601 (dt);
+  g_date_time_unref (dt);
 
-  return g_time_val_to_iso8601 (&stamp);
+  return g_steal_pointer (&iso8601_string);
 }
 
-static time_t
-timestamp_from_iso8601 (const gchar *iso_date)
+static gboolean
+timestamp_from_iso8601 (const gchar  *iso_date,
+                        time_t       *out_timestamp,
+                        GError      **error)
 {
-  GTimeVal stamp;
+  gint64 time_val;
+  GDateTime *dt = g_date_time_new_from_iso8601 (iso_date, NULL);
+  if (dt == NULL)
+    {
+      g_set_error (error, G_BOOKMARK_FILE_ERROR, G_BOOKMARK_FILE_ERROR_READ,
+                   _("Invalid date/time ‘%s’ in bookmark file"), iso_date);
+      return FALSE;
+    }
 
-  if (!g_time_val_from_iso8601 (iso_date, &stamp))
-    return (time_t) -1;
+  time_val = g_date_time_to_unix (dt);
+  g_date_time_unref (dt);
 
-  return (time_t) stamp.tv_sec;
+  *out_timestamp = time_val;
+  return TRUE;
 }
 
 G_DEFINE_QUARK (g-bookmark-file-error-quark, g_bookmark_file_error)
@@ -1582,11 +1647,11 @@ GBookmarkFile *
 g_bookmark_file_new (void)
 {
   GBookmarkFile *bookmark;
-  
+
   bookmark = g_new (GBookmarkFile, 1);
-  
+
   g_bookmark_file_init (bookmark);
-  
+
   return bookmark;
 }
 
@@ -1603,16 +1668,17 @@ g_bookmark_file_free (GBookmarkFile *bookmark)
 {
   if (!bookmark)
     return;
-  
+
   g_bookmark_file_clear (bookmark);
-  
-  g_free (bookmark);  
+
+  g_free (bookmark);
 }
 
 /**
  * g_bookmark_file_load_from_data:
  * @bookmark: an empty #GBookmarkFile struct
- * @data: desktop bookmarks loaded in memory
+ * @data: (array length=length) (element-type guint8): desktop bookmarks
+ *    loaded in memory
  * @length: the length of @data in bytes
  * @error: return location for a #GError, or %NULL
  *
@@ -1632,7 +1698,7 @@ g_bookmark_file_load_from_data (GBookmarkFile  *bookmark,
 {
   GError *parse_error;
   gboolean retval;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
 
   if (length == (gsize) -1)
@@ -1676,13 +1742,13 @@ g_bookmark_file_load_from_file (GBookmarkFile  *bookmark,
   gboolean ret = FALSE;
   gchar *buffer = NULL;
   gsize len;
-       
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (filename != NULL, FALSE);
 
   if (!g_file_get_contents (filename, &buffer, &len, error))
     goto out;
-  
+
   if (!g_bookmark_file_load_from_data (bookmark, buffer, len, error))
     goto out;
 
@@ -1754,10 +1820,10 @@ find_file_in_data_dirs (const gchar   *file,
       g_set_error_literal (error, G_BOOKMARK_FILE_ERROR,
                            G_BOOKMARK_FILE_ERROR_FILE_NOT_FOUND,
                            _("No valid bookmark file found in data dirs"));
-      
+
       return NULL;
     }
-  
+
   return path;
 }
 
@@ -1766,14 +1832,14 @@ find_file_in_data_dirs (const gchar   *file,
  * g_bookmark_file_load_from_data_dirs:
  * @bookmark: a #GBookmarkFile
  * @file: (type filename): a relative path to a filename to open and parse
- * @full_path: (type filename) (nullable): return location for a string
+ * @full_path: (out) (optional) (type filename): return location for a string
  *    containing the full path of the file, or %NULL
  * @error: return location for a #GError, or %NULL
  *
  * This function looks for a desktop bookmark file named @file in the
- * paths returned from g_get_user_data_dir() and g_get_system_data_dirs(), 
- * loads the file into @bookmark and returns the file's full path in 
- * @full_path.  If the file could not be loaded then an %error is
+ * paths returned from g_get_user_data_dir() and g_get_system_data_dirs(),
+ * loads the file into @bookmark and returns the file's full path in
+ * @full_path.  If the file could not be loaded then @error is
  * set to either a #GFileError or #GBookmarkFileError.
  *
  * Returns: %TRUE if a key file could be loaded, %FALSE otherwise
@@ -1793,10 +1859,10 @@ g_bookmark_file_load_from_data_dirs (GBookmarkFile  *bookmark,
   gsize i, j;
   gchar *output_path;
   gboolean found_file;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (!g_path_is_absolute (file), FALSE);
-  
+
   user_data_dir = g_get_user_data_dir ();
   system_data_dirs = g_get_system_data_dirs ();
   all_data_dirs = g_new0 (gchar *, g_strv_length ((gchar **)system_data_dirs) + 2);
@@ -1816,7 +1882,7 @@ g_bookmark_file_load_from_data_dirs (GBookmarkFile  *bookmark,
       g_free (output_path);
 
       output_path = find_file_in_data_dirs (file, &data_dirs, &file_error);
-      
+
       if (file_error)
         {
           g_propagate_error (error, file_error);
@@ -1835,7 +1901,7 @@ g_bookmark_file_load_from_data_dirs (GBookmarkFile  *bookmark,
 
   if (found_file && full_path)
     *full_path = output_path;
-  else 
+  else
     g_free (output_path);
 
   g_strfreev (all_data_dirs);
@@ -1852,8 +1918,8 @@ g_bookmark_file_load_from_data_dirs (GBookmarkFile  *bookmark,
  *
  * This function outputs @bookmark as a string.
  *
- * Returns: a newly allocated string holding
- *   the contents of the #GBookmarkFile
+ * Returns: (array length=length) (element-type guint8):
+ *   a newly allocated string holding the contents of the #GBookmarkFile
  *
  * Since: 2.12
  */
@@ -1864,17 +1930,17 @@ g_bookmark_file_to_data (GBookmarkFile  *bookmark,
 {
   GError *write_error = NULL;
   gchar *retval;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
-  
+
   retval = g_bookmark_file_dump (bookmark, length, &write_error);
   if (write_error)
     {
       g_propagate_error (error, write_error);
-      
+
       return NULL;
     }
-      
+
   return retval;
 }
 
@@ -1903,13 +1969,13 @@ g_bookmark_file_to_file (GBookmarkFile  *bookmark,
 
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (filename != NULL, FALSE);
-  
+
   data_error = NULL;
   data = g_bookmark_file_to_data (bookmark, &len, &data_error);
   if (data_error)
     {
       g_propagate_error (error, data_error);
-      
+
       return FALSE;
     }
 
@@ -1918,14 +1984,14 @@ g_bookmark_file_to_file (GBookmarkFile  *bookmark,
   if (write_error)
     {
       g_propagate_error (error, write_error);
-      
+
       retval = FALSE;
     }
   else
     retval = TRUE;
 
   g_free (data);
-  
+
   return retval;
 }
 
@@ -1934,7 +2000,7 @@ g_bookmark_file_lookup_item (GBookmarkFile *bookmark,
                             const gchar   *uri)
 {
   g_warn_if_fail (bookmark != NULL && uri != NULL);
-  
+
   return g_hash_table_lookup (bookmark->items_by_uri, uri);
 }
 
@@ -1958,16 +2024,16 @@ g_bookmark_file_add_item (GBookmarkFile  *bookmark,
                   item->uri);
       return;
     }
-  
+
   bookmark->items = g_list_prepend (bookmark->items, item);
-  
+
   g_hash_table_replace (bookmark->items_by_uri,
                        item->uri,
                        item);
 
   if (item->added == (time_t) -1)
     item->added = time (NULL);
-  
+
   if (item->modified == (time_t) -1)
     item->modified = time (NULL);
 }
@@ -1981,7 +2047,7 @@ g_bookmark_file_add_item (GBookmarkFile  *bookmark,
  * Removes the bookmark for @uri from the bookmark file @bookmark.
  *
  * Returns: %TRUE if the bookmark was removed successfully.
- * 
+ *
  * Since: 2.12
  */
 gboolean
@@ -1990,12 +2056,12 @@ g_bookmark_file_remove_item (GBookmarkFile  *bookmark,
                             GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
-  
+
   if (!item)
     {
       g_set_error (error, G_BOOKMARK_FILE_ERROR,
@@ -2006,8 +2072,8 @@ g_bookmark_file_remove_item (GBookmarkFile  *bookmark,
     }
 
   bookmark->items = g_list_remove (bookmark->items, item);
-  g_hash_table_remove (bookmark->items_by_uri, item->uri);  
-  
+  g_hash_table_remove (bookmark->items_by_uri, item->uri);
+
   bookmark_item_free (item);
 
   return TRUE;
@@ -2030,7 +2096,7 @@ g_bookmark_file_has_item (GBookmarkFile *bookmark,
 {
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   return (NULL != g_hash_table_lookup (bookmark->items_by_uri, uri));
 }
 
@@ -2055,26 +2121,26 @@ g_bookmark_file_get_uris (GBookmarkFile *bookmark,
   GList *l;
   gchar **uris;
   gsize i, n_items;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
-  
-  n_items = g_list_length (bookmark->items); 
+
+  n_items = g_list_length (bookmark->items);
   uris = g_new0 (gchar *, n_items + 1);
 
   /* the items are stored in reverse order, so we walk the list backward */
   for (l = g_list_last (bookmark->items), i = 0; l != NULL; l = l->prev)
     {
       BookmarkItem *item = (BookmarkItem *) l->data;
-      
+
       g_warn_if_fail (item != NULL);
-      
+
       uris[i++] = g_strdup (item->uri);
     }
   uris[i] = NULL;
-  
+
   if (length)
     *length = i;
-  
+
   return uris;
 }
 
@@ -2099,7 +2165,7 @@ g_bookmark_file_set_title (GBookmarkFile *bookmark,
                           const gchar   *title)
 {
   g_return_if_fail (bookmark != NULL);
-  
+
   if (!uri)
     {
       g_free (bookmark->title);
@@ -2108,17 +2174,17 @@ g_bookmark_file_set_title (GBookmarkFile *bookmark,
   else
     {
       BookmarkItem *item;
-      
+
       item = g_bookmark_file_lookup_item (bookmark, uri);
       if (!item)
         {
           item = bookmark_item_new (uri);
           g_bookmark_file_add_item (bookmark, item, NULL);
         }
-      
+
       g_free (item->title);
       item->title = g_strdup (title);
-      
+
       item->modified = time (NULL);
     }
 }
@@ -2147,12 +2213,12 @@ g_bookmark_file_get_title (GBookmarkFile  *bookmark,
                           GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
-  
+
   if (!uri)
     return g_strdup (bookmark->title);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2162,7 +2228,7 @@ g_bookmark_file_get_title (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   return g_strdup (item->title);
 }
 
@@ -2189,23 +2255,23 @@ g_bookmark_file_set_description (GBookmarkFile *bookmark,
 
   if (!uri)
     {
-      g_free (bookmark->description); 
+      g_free (bookmark->description);
       bookmark->description = g_strdup (description);
     }
   else
     {
       BookmarkItem *item;
-      
+
       item = g_bookmark_file_lookup_item (bookmark, uri);
       if (!item)
         {
           item = bookmark_item_new (uri);
           g_bookmark_file_add_item (bookmark, item, NULL);
         }
-      
+
       g_free (item->description);
       item->description = g_strdup (description);
-      
+
       item->modified = time (NULL);
     }
 }
@@ -2232,12 +2298,12 @@ g_bookmark_file_get_description (GBookmarkFile  *bookmark,
                                 GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
 
   if (!uri)
     return g_strdup (bookmark->description);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2247,7 +2313,7 @@ g_bookmark_file_get_description (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   return g_strdup (item->description);
 }
 
@@ -2269,23 +2335,23 @@ g_bookmark_file_set_mime_type (GBookmarkFile *bookmark,
                               const gchar   *mime_type)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
   g_return_if_fail (mime_type != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
   g_free (item->metadata->mime_type);
-  
+
   item->metadata->mime_type = g_strdup (mime_type);
   item->modified = time (NULL);
 }
@@ -2314,10 +2380,10 @@ g_bookmark_file_get_mime_type (GBookmarkFile  *bookmark,
                               GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
   g_return_val_if_fail (uri != NULL, NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2327,7 +2393,7 @@ g_bookmark_file_get_mime_type (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   if (!item->metadata)
     {
       g_set_error (error, G_BOOKMARK_FILE_ERROR,
@@ -2336,7 +2402,7 @@ g_bookmark_file_get_mime_type (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   return g_strdup (item->metadata->mime_type);
 }
 
@@ -2358,20 +2424,20 @@ g_bookmark_file_set_is_private (GBookmarkFile *bookmark,
                                gboolean       is_private)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
   item->metadata->is_private = (is_private == TRUE);
   item->modified = time (NULL);
 }
@@ -2399,10 +2465,10 @@ g_bookmark_file_get_is_private (GBookmarkFile  *bookmark,
                                GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2412,7 +2478,7 @@ g_bookmark_file_get_is_private (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   if (!item->metadata)
     {
       g_set_error (error, G_BOOKMARK_FILE_ERROR,
@@ -2421,7 +2487,7 @@ g_bookmark_file_get_is_private (GBookmarkFile  *bookmark,
                    uri);
       return FALSE;
     }
-  
+
   return item->metadata->is_private;
 }
 
@@ -2443,10 +2509,10 @@ g_bookmark_file_set_added (GBookmarkFile *bookmark,
                           time_t         added)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2456,7 +2522,7 @@ g_bookmark_file_set_added (GBookmarkFile *bookmark,
 
   if (added == (time_t) -1)
     time (&added);
-  
+
   item->added = added;
   item->modified = added;
 }
@@ -2482,10 +2548,10 @@ g_bookmark_file_get_added (GBookmarkFile  *bookmark,
                           GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, (time_t) -1);
   g_return_val_if_fail (uri != NULL, (time_t) -1);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2495,7 +2561,7 @@ g_bookmark_file_get_added (GBookmarkFile  *bookmark,
                   uri);
       return (time_t) -1;
     }
-  
+
   return item->added;
 }
 
@@ -2522,20 +2588,20 @@ g_bookmark_file_set_modified (GBookmarkFile *bookmark,
                              time_t         modified)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (modified == (time_t) -1)
     time (&modified);
-  
+
   item->modified = modified;
 }
 
@@ -2560,10 +2626,10 @@ g_bookmark_file_get_modified (GBookmarkFile  *bookmark,
                              GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, (time_t) -1);
   g_return_val_if_fail (uri != NULL, (time_t) -1);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2573,7 +2639,7 @@ g_bookmark_file_get_modified (GBookmarkFile  *bookmark,
                   uri);
       return (time_t) -1;
     }
-  
+
   return item->modified;
 }
 
@@ -2587,7 +2653,7 @@ g_bookmark_file_get_modified (GBookmarkFile  *bookmark,
  *
  * If no bookmark for @uri is found then it is created.
  *
- * The "visited" time should only be set if the bookmark was launched, 
+ * The "visited" time should only be set if the bookmark was launched,
  * either using the command line retrieved by g_bookmark_file_get_app_info()
  * or by the default application for the bookmark's MIME type, retrieved
  * using g_bookmark_file_get_mime_type().  Changing the "visited" time
@@ -2601,10 +2667,10 @@ g_bookmark_file_set_visited (GBookmarkFile *bookmark,
                             time_t         visited)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2614,7 +2680,7 @@ g_bookmark_file_set_visited (GBookmarkFile *bookmark,
 
   if (visited == (time_t) -1)
     time (&visited);
-  
+
   item->visited = visited;
 }
 
@@ -2639,10 +2705,10 @@ g_bookmark_file_get_visited (GBookmarkFile  *bookmark,
                             GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, (time_t) -1);
   g_return_val_if_fail (uri != NULL, (time_t) -1);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2652,7 +2718,7 @@ g_bookmark_file_get_visited (GBookmarkFile  *bookmark,
                   uri);
       return (time_t) -1;
     }
-  
+
   return item->visited;
 }
 
@@ -2681,10 +2747,10 @@ g_bookmark_file_has_group (GBookmarkFile  *bookmark,
 {
   BookmarkItem *item;
   GList *l;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2694,16 +2760,16 @@ g_bookmark_file_has_group (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   if (!item->metadata)
     return FALSE;
-   
+
   for (l = item->metadata->groups; l != NULL; l = l->next)
     {
       if (strcmp (l->data, group) == 0)
         return TRUE;
     }
-  
+
   return FALSE;
 
 }
@@ -2727,26 +2793,26 @@ g_bookmark_file_add_group (GBookmarkFile *bookmark,
                           const gchar   *group)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
   g_return_if_fail (group != NULL && group[0] != '\0');
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
   if (!g_bookmark_file_has_group (bookmark, uri, group, NULL))
     {
       item->metadata->groups = g_list_prepend (item->metadata->groups,
                                                g_strdup (group));
-      
+
       item->modified = time (NULL);
     }
 }
@@ -2778,10 +2844,10 @@ g_bookmark_file_remove_group (GBookmarkFile  *bookmark,
 {
   BookmarkItem *item;
   GList *l;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2791,7 +2857,7 @@ g_bookmark_file_remove_group (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   if (!item->metadata)
     {
       g_set_error (error, G_BOOKMARK_FILE_ERROR,
@@ -2800,7 +2866,7 @@ g_bookmark_file_remove_group (GBookmarkFile  *bookmark,
                    uri);
       return FALSE;
     }
-  
+
   for (l = item->metadata->groups; l != NULL; l = l->next)
     {
       if (strcmp (l->data, group) == 0)
@@ -2808,13 +2874,13 @@ g_bookmark_file_remove_group (GBookmarkFile  *bookmark,
           item->metadata->groups = g_list_remove_link (item->metadata->groups, l);
           g_free (l->data);
          g_list_free_1 (l);
-          
-          item->modified = time (NULL);          
-          
+
+          item->modified = time (NULL);
+
           return TRUE;
         }
     }
-  
+
   return FALSE;
 }
 
@@ -2822,7 +2888,8 @@ g_bookmark_file_remove_group (GBookmarkFile  *bookmark,
  * g_bookmark_file_set_groups:
  * @bookmark: a #GBookmarkFile
  * @uri: an item's URI
- * @groups: (nullable): an array of group names, or %NULL to remove all groups
+ * @groups: (nullable) (array length=length) (element-type utf8): an array of
+ *    group names, or %NULL to remove all groups
  * @length: number of group name values in @groups
  *
  * Sets a list of group names for the item with URI @uri.  Each previously
@@ -2840,27 +2907,27 @@ g_bookmark_file_set_groups (GBookmarkFile  *bookmark,
 {
   BookmarkItem *item;
   gsize i;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
   g_return_if_fail (groups != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
 
   g_list_free_full (item->metadata->groups, g_free);
   item->metadata->groups = NULL;
-  
+
   if (groups)
     {
-      for (i = 0; groups[i] != NULL && i < length; i++)
+      for (i = 0; i < length && groups[i] != NULL; i++)
         item->metadata->groups = g_list_append (item->metadata->groups,
                                                g_strdup (groups[i]));
     }
@@ -2898,10 +2965,10 @@ g_bookmark_file_get_groups (GBookmarkFile  *bookmark,
   GList *l;
   gsize len, i;
   gchar **retval;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
   g_return_val_if_fail (uri != NULL, NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -2911,15 +2978,15 @@ g_bookmark_file_get_groups (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   if (!item->metadata)
     {
       if (length)
        *length = 0;
-      
+
       return NULL;
     }
-  
+
   len = g_list_length (item->metadata->groups);
   retval = g_new0 (gchar *, len + 1);
   for (l = g_list_last (item->metadata->groups), i = 0;
@@ -2927,16 +2994,16 @@ g_bookmark_file_get_groups (GBookmarkFile  *bookmark,
        l = l->prev)
     {
       gchar *group_name = (gchar *) l->data;
-      
+
       g_warn_if_fail (group_name != NULL);
-      
+
       retval[i++] = g_strdup (group_name);
     }
   retval[i] = NULL;
-  
+
   if (length)
     *length = len;
-  
+
   return retval;
 }
 
@@ -2981,22 +3048,22 @@ g_bookmark_file_add_application (GBookmarkFile *bookmark,
 {
   BookmarkItem *item;
   gchar *app_name, *app_exec;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (name && name[0] != '\0')
     app_name = g_strdup (name);
   else
     app_name = g_strdup (g_get_application_name ());
-  
+
   if (exec && exec[0] != '\0')
     app_exec = g_strdup (exec);
   else
@@ -3008,7 +3075,7 @@ g_bookmark_file_add_application (GBookmarkFile *bookmark,
                                 -1,
                                 (time_t) -1,
                                 NULL);
-  
+
   g_free (app_exec);
   g_free (app_name);
 }
@@ -3041,11 +3108,11 @@ g_bookmark_file_remove_application (GBookmarkFile  *bookmark,
 {
   GError *set_error;
   gboolean retval;
-    
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
   g_return_val_if_fail (name != NULL, FALSE);
-  
+
   set_error = NULL;
   retval = g_bookmark_file_set_app_info (bookmark, uri,
                                         name,
@@ -3056,10 +3123,10 @@ g_bookmark_file_remove_application (GBookmarkFile  *bookmark,
   if (set_error)
     {
       g_propagate_error (error, set_error);
-      
+
       return FALSE;
     }
-  
+
   return retval;
 }
 
@@ -3087,11 +3154,11 @@ g_bookmark_file_has_application (GBookmarkFile  *bookmark,
                                 GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
   g_return_val_if_fail (name != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -3101,7 +3168,7 @@ g_bookmark_file_has_application (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   return (NULL != bookmark_item_lookup_app_info (item, name));
 }
 
@@ -3160,12 +3227,12 @@ g_bookmark_file_set_app_info (GBookmarkFile  *bookmark,
 {
   BookmarkItem *item;
   BookmarkAppInfo *ai;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
   g_return_val_if_fail (name != NULL, FALSE);
   g_return_val_if_fail (exec != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -3183,7 +3250,7 @@ g_bookmark_file_set_app_info (GBookmarkFile  *bookmark,
          g_bookmark_file_add_item (bookmark, item, NULL);
        }
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
 
@@ -3202,7 +3269,7 @@ g_bookmark_file_set_app_info (GBookmarkFile  *bookmark,
       else
         {
           ai = bookmark_app_info_new (name);
-          
+
           item->metadata->applications = g_list_prepend (item->metadata->applications, ai);
           g_hash_table_replace (item->metadata->apps_by_name, ai->name, ai);
         }
@@ -3215,27 +3282,27 @@ g_bookmark_file_set_app_info (GBookmarkFile  *bookmark,
       bookmark_app_info_free (ai);
 
       item->modified = time (NULL);
-          
+
       return TRUE;
     }
   else if (count > 0)
     ai->count = count;
   else
     ai->count += 1;
-      
+
   if (stamp != (time_t) -1)
     ai->stamp = stamp;
   else
     ai->stamp = time (NULL);
-  
+
   if (exec && exec[0] != '\0')
     {
       g_free (ai->exec);
       ai->exec = g_shell_quote (exec);
     }
-  
+
   item->modified = time (NULL);
-  
+
   return TRUE;
 }
 
@@ -3246,7 +3313,7 @@ expand_exec_line (const gchar *exec_fmt,
 {
   GString *exec;
   gchar ch;
-  
+
   exec = g_string_sized_new (512);
   while ((ch = *exec_fmt++) != '\0')
    {
@@ -3255,7 +3322,7 @@ expand_exec_line (const gchar *exec_fmt,
          exec = g_string_append_c (exec, ch);
          continue;
        }
-     
+
      ch = *exec_fmt++;
      switch (ch)
        {
@@ -3287,7 +3354,7 @@ expand_exec_line (const gchar *exec_fmt,
          break;
        }
    }
-   
+
  out:
   return g_string_free (exec, FALSE);
 }
@@ -3302,8 +3369,8 @@ expand_exec_line (const gchar *exec_fmt,
  * @stamp: (out) (optional): return location for the last registration time, or %NULL
  * @error: return location for a #GError, or %NULL
  *
- * Gets the registration informations of @app_name for the bookmark for
- * @uri.  See g_bookmark_file_set_app_info() for more informations about
+ * Gets the registration information of @app_name for the bookmark for
+ * @uri.  See g_bookmark_file_set_app_info() for more information about
  * the returned data.
  *
  * The string returned in @app_exec must be freed.
@@ -3331,11 +3398,11 @@ g_bookmark_file_get_app_info (GBookmarkFile  *bookmark,
 {
   BookmarkItem *item;
   BookmarkAppInfo *ai;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
   g_return_val_if_fail (name != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -3345,7 +3412,7 @@ g_bookmark_file_get_app_info (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   ai = bookmark_item_lookup_app_info (item, name);
   if (!ai)
     {
@@ -3356,12 +3423,12 @@ g_bookmark_file_get_app_info (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   if (exec)
     {
       GError *unquote_error = NULL;
       gchar *command_line;
-      
+
       command_line = g_shell_unquote (ai->exec, &unquote_error);
       if (unquote_error)
         {
@@ -3382,14 +3449,14 @@ g_bookmark_file_get_app_info (GBookmarkFile  *bookmark,
         }
       else
         g_free (command_line);
-    } 
+    }
 
   if (count)
     *count = ai->count;
-  
+
   if (stamp)
     *stamp = ai->stamp;
-  
+
   return TRUE;
 }
 
@@ -3402,7 +3469,7 @@ g_bookmark_file_get_app_info (GBookmarkFile  *bookmark,
  *
  * Retrieves the names of the applications that have registered the
  * bookmark for @uri.
- * 
+ *
  * In the event the URI cannot be found, %NULL is returned and
  * @error is set to #G_BOOKMARK_FILE_ERROR_URI_NOT_FOUND.
  *
@@ -3421,10 +3488,10 @@ g_bookmark_file_get_applications (GBookmarkFile  *bookmark,
   GList *l;
   gchar **apps;
   gsize i, n_apps;
-  
+
   g_return_val_if_fail (bookmark != NULL, NULL);
   g_return_val_if_fail (uri != NULL, NULL);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -3434,45 +3501,45 @@ g_bookmark_file_get_applications (GBookmarkFile  *bookmark,
                   uri);
       return NULL;
     }
-  
+
   if (!item->metadata)
-    {     
+    {
       if (length)
        *length = 0;
-      
+
       return NULL;
     }
-  
+
   n_apps = g_list_length (item->metadata->applications);
   apps = g_new0 (gchar *, n_apps + 1);
-  
+
   for (l = g_list_last (item->metadata->applications), i = 0;
        l != NULL;
        l = l->prev)
     {
       BookmarkAppInfo *ai;
-      
+
       ai = (BookmarkAppInfo *) l->data;
-      
+
       g_warn_if_fail (ai != NULL);
       g_warn_if_fail (ai->name != NULL);
-      
+
       apps[i++] = g_strdup (ai->name);
     }
   apps[i] = NULL;
-  
+
   if (length)
     *length = i;
-  
+
   return apps;
 }
 
 /**
  * g_bookmark_file_get_size:
  * @bookmark: a #GBookmarkFile
- * 
+ *
  * Gets the number of bookmarks inside @bookmark.
- * 
+ *
  * Returns: the number of bookmarks
  *
  * Since: 2.12
@@ -3510,7 +3577,7 @@ g_bookmark_file_move_item (GBookmarkFile  *bookmark,
                           GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (old_uri != NULL, FALSE);
 
@@ -3526,6 +3593,9 @@ g_bookmark_file_move_item (GBookmarkFile  *bookmark,
 
   if (new_uri && new_uri[0] != '\0')
     {
+      if (g_strcmp0 (old_uri, new_uri) == 0)
+        return TRUE;
+
       if (g_bookmark_file_has_item (bookmark, new_uri))
         {
           if (!g_bookmark_file_remove_item (bookmark, new_uri, error))
@@ -3533,7 +3603,7 @@ g_bookmark_file_move_item (GBookmarkFile  *bookmark,
         }
 
       g_hash_table_steal (bookmark->items_by_uri, item->uri);
-      
+
       g_free (item->uri);
       item->uri = g_strdup (new_uri);
       item->modified = time (NULL);
@@ -3573,7 +3643,7 @@ g_bookmark_file_set_icon (GBookmarkFile *bookmark,
                          const gchar   *mime_type)
 {
   BookmarkItem *item;
-  
+
   g_return_if_fail (bookmark != NULL);
   g_return_if_fail (uri != NULL);
 
@@ -3583,20 +3653,20 @@ g_bookmark_file_set_icon (GBookmarkFile *bookmark,
       item = bookmark_item_new (uri);
       g_bookmark_file_add_item (bookmark, item, NULL);
     }
-  
+
   if (!item->metadata)
     item->metadata = bookmark_metadata_new ();
-  
+
   g_free (item->metadata->icon_href);
   g_free (item->metadata->icon_mime);
-  
+
   item->metadata->icon_href = g_strdup (href);
-  
+
   if (mime_type && mime_type[0] != '\0')
     item->metadata->icon_mime = g_strdup (mime_type);
   else
     item->metadata->icon_mime = g_strdup ("application/octet-stream");
-  
+
   item->modified = time (NULL);
 }
 
@@ -3626,10 +3696,10 @@ g_bookmark_file_get_icon (GBookmarkFile  *bookmark,
                          GError        **error)
 {
   BookmarkItem *item;
-  
+
   g_return_val_if_fail (bookmark != NULL, FALSE);
   g_return_val_if_fail (uri != NULL, FALSE);
-  
+
   item = g_bookmark_file_lookup_item (bookmark, uri);
   if (!item)
     {
@@ -3639,15 +3709,15 @@ g_bookmark_file_get_icon (GBookmarkFile  *bookmark,
                   uri);
       return FALSE;
     }
-  
+
   if ((!item->metadata) || (!item->metadata->icon_href))
     return FALSE;
-  
+
   if (href)
     *href = g_strdup (item->metadata->icon_href);
-  
+
   if (mime_type)
     *mime_type = g_strdup (item->metadata->icon_mime);
-  
+
   return TRUE;
 }