Fix sparse warnings
[platform/upstream/glib.git] / gio / gfileinfo.c
index 6986615..264ccf1 100644 (file)
 
 /**
  * SECTION:gfileinfo
- * @short_description: File Information and Attributes.
- * @see_also: #GFile.
- * 
+ * @short_description: File Information and Attributes
+ * @include: gio/gio.h
+ * @see_also: #GFile, <link linkend="gio-GFileAttribute">GFileAttribute</link>
+ *
  * Functionality for manipulating basic metadata for files. #GFileInfo
- * implements methods for getting information that all files should 
- * contain, and allows for manipulation of extended attributes. 
- * #GFileAttributeMatcher allows for searching through a #GFileInfo for 
+ * implements methods for getting information that all files should
+ * contain, and allows for manipulation of extended attributes.
+ *
+ * See <link linkend="gio-GFileAttribute">GFileAttribute</link> for more
+ * information on how GIO handles file attributes.
+ *
+ * To obtain a #GFileInfo for a #GFile, use g_file_query_info() (or its
+ * async variant). To obtain a #GFileInfo for a file input or output
+ * stream, use g_file_input_stream_query_info() or
+ * g_file_output_stream_query_info() (or their async variants).
+ *
+ * To change the actual attributes of a file, you should then set the
+ * attribute in the #GFileInfo and call g_file_set_attributes_from_info()
+ * or g_file_set_attributes_async() on a GFile.
+ *
+ * However, not all attributes can be changed in the file. For instance,
+ * the actual size of a file cannot be changed via g_file_info_set_size().
+ * You may call g_file_query_settable_attributes() and
+ * g_file_query_writable_namespaces() to discover the settable attributes
+ * of a particular file at runtime.
+ *
+ * #GFileAttributeMatcher allows for searching through a #GFileInfo for
  * attributes.
- * 
- * 
- * 
  **/
 
-#include <config.h>
+#include "config.h"
 
 #include <string.h>
 
 #include "gfileinfo.h"
+#include "gfileinfo-priv.h"
+#include "gfileattribute-priv.h"
+#include "gicon.h"
 #include "glibintl.h"
 
+
 /* We use this nasty thing, because NULL is a valid attribute matcher (matches nothing) */
 #define NO_ATTRIBUTE_MASK ((GFileAttributeMatcher *)1)
 
@@ -63,8 +84,6 @@ struct _GFileInfoClass
   GObjectClass parent_class;
 };
 
-static gboolean g_file_attribute_matcher_matches_id (GFileAttributeMatcher *matcher,
-                                                    guint32 id);
 
 G_DEFINE_TYPE (GFileInfo, g_file_info, G_TYPE_OBJECT);
 
@@ -119,6 +138,133 @@ _lookup_namespace (const char *namespace)
 }
 
 static guint32
+_lookup_attribute (const char *attribute)
+{
+  guint32 attr_id, id;
+  char *ns;
+  const char *colon;
+  NSInfo *ns_info;
+  
+  attr_id = GPOINTER_TO_UINT (g_hash_table_lookup (attribute_hash, attribute));
+
+  if (attr_id != 0)
+    return attr_id;
+
+  colon = strstr (attribute, "::");
+  if (colon)
+    ns = g_strndup (attribute, colon - attribute);
+  else
+    ns = g_strdup ("");
+
+  ns_info = _lookup_namespace (ns);
+  g_free (ns);
+
+  id = ++ns_info->attribute_id_counter;
+  attributes[ns_info->id] = g_realloc (attributes[ns_info->id], (id + 1) * sizeof (char *));
+  attributes[ns_info->id][id] = g_strdup (attribute);
+  
+  attr_id = MAKE_ATTR_ID (ns_info->id, id);
+
+  g_hash_table_insert (attribute_hash, attributes[ns_info->id][id], GUINT_TO_POINTER (attr_id));
+
+  return attr_id;
+}
+
+static void
+ensure_attribute_hash (void)
+{
+  if (attribute_hash != NULL)
+    return;
+
+  ns_hash = g_hash_table_new (g_str_hash, g_str_equal);
+  attribute_hash = g_hash_table_new (g_str_hash, g_str_equal);
+
+#define REGISTER_ATTRIBUTE(name) G_STMT_START{\
+  guint _u = _lookup_attribute (G_FILE_ATTRIBUTE_ ## name); \
+  /* use for generating the ID: g_print ("#define G_FILE_ATTRIBUTE_ID_%s (%u + %u)\n", #name + 17, _u & ~ID_MASK, _u & ID_MASK); */ \
+  g_assert (_u == G_FILE_ATTRIBUTE_ID_ ## name); \
+}G_STMT_END
+
+  REGISTER_ATTRIBUTE (STANDARD_TYPE);
+  REGISTER_ATTRIBUTE (STANDARD_IS_HIDDEN);
+  REGISTER_ATTRIBUTE (STANDARD_IS_BACKUP);
+  REGISTER_ATTRIBUTE (STANDARD_IS_SYMLINK);
+  REGISTER_ATTRIBUTE (STANDARD_IS_VIRTUAL);
+  REGISTER_ATTRIBUTE (STANDARD_NAME);
+  REGISTER_ATTRIBUTE (STANDARD_DISPLAY_NAME);
+  REGISTER_ATTRIBUTE (STANDARD_EDIT_NAME);
+  REGISTER_ATTRIBUTE (STANDARD_COPY_NAME);
+  REGISTER_ATTRIBUTE (STANDARD_DESCRIPTION);
+  REGISTER_ATTRIBUTE (STANDARD_ICON);
+  REGISTER_ATTRIBUTE (STANDARD_CONTENT_TYPE);
+  REGISTER_ATTRIBUTE (STANDARD_FAST_CONTENT_TYPE);
+  REGISTER_ATTRIBUTE (STANDARD_SIZE);
+  REGISTER_ATTRIBUTE (STANDARD_ALLOCATED_SIZE);
+  REGISTER_ATTRIBUTE (STANDARD_SYMLINK_TARGET);
+  REGISTER_ATTRIBUTE (STANDARD_TARGET_URI);
+  REGISTER_ATTRIBUTE (STANDARD_SORT_ORDER);
+  REGISTER_ATTRIBUTE (ETAG_VALUE);
+  REGISTER_ATTRIBUTE (ID_FILE);
+  REGISTER_ATTRIBUTE (ID_FILESYSTEM);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_READ);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_WRITE);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_EXECUTE);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_DELETE);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_TRASH);
+  REGISTER_ATTRIBUTE (ACCESS_CAN_RENAME);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_MOUNT);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_UNMOUNT);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_EJECT);
+  REGISTER_ATTRIBUTE (MOUNTABLE_UNIX_DEVICE);
+  REGISTER_ATTRIBUTE (MOUNTABLE_UNIX_DEVICE_FILE);
+  REGISTER_ATTRIBUTE (MOUNTABLE_HAL_UDI);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_START);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_START_DEGRADED);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_STOP);
+  REGISTER_ATTRIBUTE (MOUNTABLE_START_STOP_TYPE);
+  REGISTER_ATTRIBUTE (MOUNTABLE_CAN_POLL);
+  REGISTER_ATTRIBUTE (MOUNTABLE_IS_MEDIA_CHECK_AUTOMATIC);
+  REGISTER_ATTRIBUTE (TIME_MODIFIED);
+  REGISTER_ATTRIBUTE (TIME_MODIFIED_USEC);
+  REGISTER_ATTRIBUTE (TIME_ACCESS);
+  REGISTER_ATTRIBUTE (TIME_ACCESS_USEC);
+  REGISTER_ATTRIBUTE (TIME_CHANGED);
+  REGISTER_ATTRIBUTE (TIME_CHANGED_USEC);
+  REGISTER_ATTRIBUTE (TIME_CREATED);
+  REGISTER_ATTRIBUTE (TIME_CREATED_USEC);
+  REGISTER_ATTRIBUTE (UNIX_DEVICE);
+  REGISTER_ATTRIBUTE (UNIX_INODE);
+  REGISTER_ATTRIBUTE (UNIX_MODE);
+  REGISTER_ATTRIBUTE (UNIX_NLINK);
+  REGISTER_ATTRIBUTE (UNIX_UID);
+  REGISTER_ATTRIBUTE (UNIX_GID);
+  REGISTER_ATTRIBUTE (UNIX_RDEV);
+  REGISTER_ATTRIBUTE (UNIX_BLOCK_SIZE);
+  REGISTER_ATTRIBUTE (UNIX_BLOCKS);
+  REGISTER_ATTRIBUTE (UNIX_IS_MOUNTPOINT);
+  REGISTER_ATTRIBUTE (DOS_IS_ARCHIVE);
+  REGISTER_ATTRIBUTE (DOS_IS_SYSTEM);
+  REGISTER_ATTRIBUTE (OWNER_USER);
+  REGISTER_ATTRIBUTE (OWNER_USER_REAL);
+  REGISTER_ATTRIBUTE (OWNER_GROUP);
+  REGISTER_ATTRIBUTE (THUMBNAIL_PATH);
+  REGISTER_ATTRIBUTE (THUMBNAILING_FAILED);
+  REGISTER_ATTRIBUTE (PREVIEW_ICON);
+  REGISTER_ATTRIBUTE (FILESYSTEM_SIZE);
+  REGISTER_ATTRIBUTE (FILESYSTEM_FREE);
+  REGISTER_ATTRIBUTE (FILESYSTEM_TYPE);
+  REGISTER_ATTRIBUTE (FILESYSTEM_READONLY);
+  REGISTER_ATTRIBUTE (FILESYSTEM_USE_PREVIEW);
+  REGISTER_ATTRIBUTE (GVFS_BACKEND);
+  REGISTER_ATTRIBUTE (SELINUX_CONTEXT);
+  REGISTER_ATTRIBUTE (TRASH_ITEM_COUNT);
+  REGISTER_ATTRIBUTE (TRASH_ORIG_PATH);
+  REGISTER_ATTRIBUTE (TRASH_DELETION_DATE);
+
+#undef REGISTER_ATTRIBUTE
+}
+
+static guint32
 lookup_namespace (const char *namespace)
 {
   NSInfo *ns_info;
@@ -126,11 +272,7 @@ lookup_namespace (const char *namespace)
   
   G_LOCK (attribute_hash);
   
-  if (attribute_hash == NULL)
-    {
-      ns_hash = g_hash_table_new (g_str_hash, g_str_equal);
-      attribute_hash = g_hash_table_new (g_str_hash, g_str_equal);
-    }
+  ensure_attribute_hash ();
 
   ns_info = _lookup_namespace (namespace);
   id = 0;
@@ -155,42 +297,12 @@ get_attribute_for_id (int attribute)
 static guint32
 lookup_attribute (const char *attribute)
 {
-  guint32 attr_id, id;
-  char *ns;
-  const char *colon;
-  NSInfo *ns_info;
+  guint32 attr_id;
   
   G_LOCK (attribute_hash);
-  if (attribute_hash == NULL)
-    {
-      ns_hash = g_hash_table_new (g_str_hash, g_str_equal);
-      attribute_hash = g_hash_table_new (g_str_hash, g_str_equal);
-    }
-
-  attr_id = GPOINTER_TO_UINT (g_hash_table_lookup (attribute_hash, attribute));
-
-  if (attr_id != 0)
-    {
-      G_UNLOCK (attribute_hash);
-      return attr_id;
-    }
-
-  colon = strchr (attribute, ':');
-  if (colon)
-    ns = g_strndup (attribute, colon - attribute);
-  else
-    ns = g_strdup ("");
-
-  ns_info = _lookup_namespace (ns);
-  g_free (ns);
+  ensure_attribute_hash ();
 
-  id = ++ns_info->attribute_id_counter;
-  attributes[ns_info->id] = g_realloc (attributes[ns_info->id], (id + 1) * sizeof (char *));
-  attributes[ns_info->id][id] = g_strdup (attribute);
-  
-  attr_id = MAKE_ATTR_ID (ns_info->id, id);
-
-  g_hash_table_insert (attribute_hash, attributes[ns_info->id][id], GUINT_TO_POINTER (attr_id));
+  attr_id = _lookup_attribute (attribute);
   
   G_UNLOCK (attribute_hash);
   
@@ -208,14 +320,13 @@ g_file_info_finalize (GObject *object)
 
   attrs = (GFileAttribute *)info->attributes->data;
   for (i = 0; i < info->attributes->len; i++)
-    g_file_attribute_value_clear (&attrs[i].value);
-  g_array_free (info->attributes, TRUE);  
+    _g_file_attribute_value_clear (&attrs[i].value);
+  g_array_free (info->attributes, TRUE);
 
   if (info->mask != NO_ATTRIBUTE_MASK)
     g_file_attribute_matcher_unref (info->mask);
-  
-  if (G_OBJECT_CLASS (g_file_info_parent_class)->finalize)
-    (*G_OBJECT_CLASS (g_file_info_parent_class)->finalize) (object);
+
+  G_OBJECT_CLASS (g_file_info_parent_class)->finalize (object);
 }
 
 static void
@@ -255,7 +366,8 @@ g_file_info_new (void)
  * Copies all of the #GFileAttribute<!-- -->s from @src_info to @dest_info.
  **/
 void
-g_file_info_copy_into (GFileInfo *src_info, GFileInfo *dest_info)
+g_file_info_copy_into (GFileInfo *src_info, 
+                       GFileInfo *dest_info)
 {
   GFileAttribute *source, *dest;
   int i;
@@ -265,7 +377,7 @@ g_file_info_copy_into (GFileInfo *src_info, GFileInfo *dest_info)
 
   dest = (GFileAttribute *)dest_info->attributes->data;
   for (i = 0; i < dest_info->attributes->len; i++)
-    g_file_attribute_value_clear (&dest[i].value);
+    _g_file_attribute_value_clear (&dest[i].value);
   
   g_array_set_size (dest_info->attributes,
                    src_info->attributes->len);
@@ -277,9 +389,12 @@ g_file_info_copy_into (GFileInfo *src_info, GFileInfo *dest_info)
     {
       dest[i].attribute = source[i].attribute;
       dest[i].value.type = G_FILE_ATTRIBUTE_TYPE_INVALID;
-      g_file_attribute_value_set (&dest[i].value, &source[i].value);
+      _g_file_attribute_value_set (&dest[i].value, &source[i].value);
     }
 
+  if (dest_info->mask != NO_ATTRIBUTE_MASK)
+    g_file_attribute_matcher_unref (dest_info->mask);
+
   if (src_info->mask == NO_ATTRIBUTE_MASK)
     dest_info->mask = NO_ATTRIBUTE_MASK;
   else
@@ -295,7 +410,7 @@ g_file_info_copy_into (GFileInfo *src_info, GFileInfo *dest_info)
  * Returns: a duplicate #GFileInfo of @other.
  **/
 GFileInfo *
-g_file_info_dup (GFileInfo  *other)
+g_file_info_dup (GFileInfo *other)
 {
   GFileInfo *new;
   
@@ -314,14 +429,13 @@ g_file_info_dup (GFileInfo  *other)
  * Sets @mask on @info to match specific attribute types.
  **/
 void
-g_file_info_set_attribute_mask (GFileInfo *info,
+g_file_info_set_attribute_mask (GFileInfo             *info,
                                GFileAttributeMatcher *mask)
 {
   GFileAttribute *attr;
   int i;
   
   g_return_if_fail (G_IS_FILE_INFO (info));
-  g_return_if_fail (mask != NULL);
   
   if (mask != info->mask)
     {
@@ -333,10 +447,10 @@ g_file_info_set_attribute_mask (GFileInfo *info,
       for (i = 0; i < info->attributes->len; i++)
        {
          attr = &g_array_index (info->attributes, GFileAttribute, i);
-         if (!g_file_attribute_matcher_matches_id (mask,
+         if (!_g_file_attribute_matcher_matches_id (mask,
                                                    attr->attribute))
            {
-             g_file_attribute_value_clear (&attr->value);
+             _g_file_attribute_value_clear (&attr->value);
              g_array_remove_index (info->attributes, i);
              i--;
            }
@@ -382,11 +496,11 @@ g_file_info_clear_status (GFileInfo  *info)
 
 static int
 g_file_info_find_place (GFileInfo  *info,
-                       guint32 attribute)
+                       guint32     attribute)
 {
   int min, max, med;
   GFileAttribute *attrs;
-  /* Binary search for the place where attribute would be, if its
+  /* Binary search for the place where attribute would be, if it's
      in the array */
 
   min = 0;
@@ -413,7 +527,7 @@ g_file_info_find_place (GFileInfo  *info,
 
 static GFileAttributeValue *
 g_file_info_find_value (GFileInfo *info,
-                       guint32 attr_id)
+                       guint32    attr_id)
 {
   GFileAttribute *attrs;
   int i;
@@ -428,7 +542,7 @@ g_file_info_find_value (GFileInfo *info,
 }
 
 static GFileAttributeValue *
-g_file_info_find_value_by_name (GFileInfo *info,
+g_file_info_find_value_by_name (GFileInfo  *info,
                                const char *attribute)
 {
   guint32 attr_id;
@@ -444,8 +558,8 @@ g_file_info_find_value_by_name (GFileInfo *info,
  * 
  * Checks if a file info structure has an attribute named @attribute.
  * 
- * Returns: %TRUE if @GFileInfo has an attribute named @attribute, 
- * %FALSE otherwise.
+ * Returns: %TRUE if @Ginfo has an attribute named @attribute, 
+ *     %FALSE otherwise.
  **/
 gboolean
 g_file_info_has_attribute (GFileInfo  *info,
@@ -461,13 +575,49 @@ g_file_info_has_attribute (GFileInfo  *info,
 }
 
 /**
+ * g_file_info_has_namespace:
+ * @info: a #GFileInfo.
+ * @name_space: a file attribute namespace.
+ *
+ * Checks if a file info structure has an attribute in the
+ * specified @name_space.
+ *
+ * Returns: %TRUE if @Ginfo has an attribute in @name_space,
+ *     %FALSE otherwise.
+ *
+ * Since: 2.22
+ **/
+gboolean
+g_file_info_has_namespace (GFileInfo  *info,
+                          const char *name_space)
+{
+  GFileAttribute *attrs;
+  guint32 ns_id;
+  int i;
+
+  g_return_val_if_fail (G_IS_FILE_INFO (info), FALSE);
+  g_return_val_if_fail (name_space != NULL, FALSE);
+
+  ns_id = lookup_namespace (name_space);
+
+  attrs = (GFileAttribute *)info->attributes->data;
+  for (i = 0; i < info->attributes->len; i++)
+    {
+      if (GET_NS (attrs[i].attribute) == ns_id)
+       return TRUE;
+    }
+
+  return FALSE;
+}
+
+/**
  * g_file_info_list_attributes:
  * @info: a #GFileInfo.
  * @name_space: a file attribute key's namespace.
  * 
  * Lists the file info structure's attributes.
  * 
- * Returns: a null-terminated array of strings of all of the 
+ * Returns: (array zero-terminated=1) (transfer full): a null-terminated array of strings of all of the 
  * possible attribute types for the given @name_space, or 
  * %NULL on error.
  **/
@@ -506,7 +656,7 @@ g_file_info_list_attributes (GFileInfo  *info,
  * Gets the attribute type for an attribute key.
  * 
  * Returns: a #GFileAttributeType for the given @attribute, or 
- * %G_FILE_ATTRIBUTE_TYPE_INVALID if the key is invalid.
+ * %G_FILE_ATTRIBUTE_TYPE_INVALID if the key is not set.
  **/
 GFileAttributeType
 g_file_info_get_attribute_type (GFileInfo  *info,
@@ -530,7 +680,6 @@ g_file_info_get_attribute_type (GFileInfo  *info,
  * @attribute: a file attribute key.
  * 
  * Removes all cases of @attribute from @info if it exists.
- *
  **/
 void
 g_file_info_remove_attribute (GFileInfo  *info,
@@ -550,24 +699,116 @@ g_file_info_remove_attribute (GFileInfo  *info,
   if (i < info->attributes->len &&
       attrs[i].attribute == attr_id)
     {
-      g_file_attribute_value_clear (&attrs[i].value);
+      _g_file_attribute_value_clear (&attrs[i].value);
       g_array_remove_index (info->attributes, i);
     }
 }
 
 /**
- * g_file_info_get_attribute:
- * @info: a #GFileInfo.
- * @attribute: a file attribute key.
- * 
- * Gets an attribute value from a file info structure.
- * 
- * Returns: a #GFileAttributeValue for the given @attribute, or 
- * %NULL otherwise.
- **/
+ * g_file_info_get_attribute_data:
+ * @info: a #GFileInfo
+ * @attribute: a file attribute key
+ * @type: (out) (allow-none): return location for the attribute type, or %NULL
+ * @value_pp: (out) (allow-none): return location for the attribute value, or %NULL
+ * @status: (out) (allow-none): return location for the attribute status, or %NULL
+ *
+ * Gets the attribute type, value and status for an attribute key.
+ *
+ * Returns: (transfer none): %TRUE if @info has an attribute named @attribute, 
+ *      %FALSE otherwise.
+ */
+gboolean
+g_file_info_get_attribute_data (GFileInfo            *info,
+                               const char           *attribute,
+                               GFileAttributeType   *type,
+                               gpointer             *value_pp,
+                               GFileAttributeStatus *status)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_find_value_by_name (info, attribute);
+  if (value == NULL)
+    return FALSE;
+
+  if (status)
+    *status = value->status;
+
+  if (type)
+    *type = value->type;
+
+  if (value_pp)
+    *value_pp = _g_file_attribute_value_peek_as_pointer (value);
+  
+  return TRUE;
+}
+
+/** 
+ * g_file_info_get_attribute_status:
+ * @info: a #GFileInfo
+ * @attribute: a file attribute key
+ *
+ * Gets the attribute status for an attribute key.
+ *
+ * Returns: a #GFileAttributeStatus for the given @attribute, or 
+ *    %G_FILE_ATTRIBUTE_STATUS_UNSET if the key is invalid.
+ *
+ */
+GFileAttributeStatus
+g_file_info_get_attribute_status (GFileInfo  *info,
+                                 const char *attribute)
+{
+  GFileAttributeValue *val;
+  
+  g_return_val_if_fail (G_IS_FILE_INFO (info), 0);
+  g_return_val_if_fail (attribute != NULL && *attribute != '\0', 0);
+
+  val = g_file_info_find_value_by_name (info, attribute);
+  if (val)
+    return val->status;
+
+  return G_FILE_ATTRIBUTE_STATUS_UNSET;
+}
+
+/**
+ * g_file_info_set_attribute_status:
+ * @info: a #GFileInfo
+ * @attribute: a file attribute key
+ * @status: a #GFileAttributeStatus
+ *
+ * Sets the attribute status for an attribute key. This is only
+ * needed by external code that implement g_file_set_attributes_from_info()
+ * or similar functions.
+ *
+ * The attribute must exist in @info for this to work. Otherwise %FALSE
+ * is returned and @info is unchanged.
+ *
+ * Returns: %TRUE if the status was changed, %FALSE if the key was not set.
+ *
+ * Since: 2.22
+ */
+gboolean
+g_file_info_set_attribute_status (GFileInfo  *info,
+                                 const char *attribute,
+                                 GFileAttributeStatus status)
+{
+  GFileAttributeValue *val;
+
+  g_return_val_if_fail (G_IS_FILE_INFO (info), FALSE);
+  g_return_val_if_fail (attribute != NULL && *attribute != '\0', FALSE);
+
+  val = g_file_info_find_value_by_name (info, attribute);
+  if (val)
+    {
+      val->status = status;
+      return TRUE;
+    }
+
+  return FALSE;
+}
+
 GFileAttributeValue *
-g_file_info_get_attribute (GFileInfo  *info,
-                          const char *attribute)
+_g_file_info_get_attribute_value (GFileInfo  *info,
+                                 const char *attribute)
   
 {
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
@@ -577,6 +818,30 @@ g_file_info_get_attribute (GFileInfo  *info,
 }
 
 /**
+ * g_file_info_get_attribute_as_string:
+ * @info: a #GFileInfo.
+ * @attribute: a file attribute key.
+ * 
+ * Gets the value of a attribute, formated as a string.
+ * This escapes things as needed to make the string valid
+ * utf8.
+ * 
+ * Returns: a UTF-8 string associated with the given @attribute.
+ *    When you're done with the string it must be freed with g_free().
+ **/
+char *
+g_file_info_get_attribute_as_string (GFileInfo  *info,
+                                    const char *attribute)
+{
+  GFileAttributeValue *val;
+  val = _g_file_info_get_attribute_value (info, attribute);
+  if (val) 
+    return _g_file_attribute_value_as_string (val);
+  return NULL;
+}
+
+
+/**
  * g_file_info_get_attribute_object:
  * @info: a #GFileInfo.
  * @attribute: a file attribute key.
@@ -584,7 +849,7 @@ g_file_info_get_attribute (GFileInfo  *info,
  * Gets the value of a #GObject attribute. If the attribute does 
  * not contain a #GObject, %NULL will be returned.
  * 
- * Returns: a #GObject associated with the given @attribute, or
+ * Returns: (transfer none): a #GObject associated with the given @attribute, or
  * %NULL otherwise.
  **/
 GObject *
@@ -597,7 +862,7 @@ g_file_info_get_attribute_object (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', NULL);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_object (value);
+  return _g_file_attribute_value_get_object (value);
 }
 
 /**
@@ -621,7 +886,7 @@ g_file_info_get_attribute_string (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', NULL);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_string (value);
+  return _g_file_attribute_value_get_string (value);
 }
 
 /**
@@ -645,7 +910,33 @@ g_file_info_get_attribute_byte_string (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', NULL);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_byte_string (value);
+  return _g_file_attribute_value_get_byte_string (value);
+}
+
+/**
+ * g_file_info_get_attribute_stringv:
+ * @info: a #GFileInfo.
+ * @attribute: a file attribute key.
+ *
+ * Gets the value of a stringv attribute. If the attribute does
+ * not contain a stringv, %NULL will be returned.
+ *
+ * Returns: the contents of the @attribute value as a stringv, or
+ * %NULL otherwise. Do not free.
+ *
+ * Since: 2.22
+ **/
+char **
+g_file_info_get_attribute_stringv (GFileInfo  *info,
+                                  const char *attribute)
+{
+  GFileAttributeValue *value;
+
+  g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
+  g_return_val_if_fail (attribute != NULL && *attribute != '\0', NULL);
+
+  value = g_file_info_find_value_by_name (info, attribute);
+  return _g_file_attribute_value_get_stringv (value);
 }
 
 /**
@@ -668,7 +959,7 @@ g_file_info_get_attribute_boolean (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', FALSE);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_boolean (value);
+  return _g_file_attribute_value_get_boolean (value);
 }
 
 /**
@@ -678,7 +969,7 @@ g_file_info_get_attribute_boolean (GFileInfo  *info,
  * 
  * Gets an unsigned 32-bit integer contained within the attribute. If the 
  * attribute does not contain an unsigned 32-bit integer, or is invalid, 
- * %0 will be returned.
+ * 0 will be returned.
  * 
  * Returns: an unsigned 32-bit integer from the attribute. 
  **/
@@ -692,7 +983,7 @@ g_file_info_get_attribute_uint32 (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', 0);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_uint32 (value);
+  return _g_file_attribute_value_get_uint32 (value);
 }
 
 /**
@@ -702,7 +993,7 @@ g_file_info_get_attribute_uint32 (GFileInfo  *info,
  * 
  * Gets a signed 32-bit integer contained within the attribute. If the 
  * attribute does not contain a signed 32-bit integer, or is invalid, 
- * %0 will be returned.
+ * 0 will be returned.
  * 
  * Returns: a signed 32-bit integer from the attribute.
  **/
@@ -716,7 +1007,7 @@ g_file_info_get_attribute_int32 (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', 0);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_int32 (value);
+  return _g_file_attribute_value_get_int32 (value);
 }
 
 /**
@@ -726,7 +1017,7 @@ g_file_info_get_attribute_int32 (GFileInfo  *info,
  * 
  * Gets a unsigned 64-bit integer contained within the attribute. If the 
  * attribute does not contain an unsigned 64-bit integer, or is invalid, 
- * %0 will be returned.
+ * 0 will be returned.
  * 
  * Returns: a unsigned 64-bit integer from the attribute. 
  **/
@@ -740,7 +1031,7 @@ g_file_info_get_attribute_uint64 (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', 0);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_uint64 (value);
+  return _g_file_attribute_value_get_uint64 (value);
 }
 
 /**
@@ -750,7 +1041,7 @@ g_file_info_get_attribute_uint64 (GFileInfo  *info,
  * 
  * Gets a signed 64-bit integer contained within the attribute. If the 
  * attribute does not contain an signed 64-bit integer, or is invalid, 
- * %0 will be returned.
+ * 0 will be returned.
  * 
  * Returns: a signed 64-bit integer from the attribute. 
  **/
@@ -764,7 +1055,7 @@ g_file_info_get_attribute_int64  (GFileInfo  *info,
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', 0);
 
   value = g_file_info_find_value_by_name (info, attribute);
-  return g_file_attribute_value_get_int64 (value);
+  return _g_file_attribute_value_get_int64 (value);
 }
 
 static GFileAttributeValue *
@@ -772,11 +1063,10 @@ g_file_info_create_value (GFileInfo *info,
                          guint32 attr_id)
 {
   GFileAttribute *attrs;
-  GFileAttribute attr;
   int i;
 
   if (info->mask != NO_ATTRIBUTE_MASK &&
-      !g_file_attribute_matcher_matches_id (info->mask, attr_id))
+      !_g_file_attribute_matcher_matches_id (info->mask, attr_id))
     return NULL;
   
   i = g_file_info_find_place (info, attr_id);
@@ -787,8 +1077,8 @@ g_file_info_create_value (GFileInfo *info,
     return &attrs[i].value;
   else
     {
+      GFileAttribute attr = { 0 };
       attr.attribute = attr_id;
-      attr.value.type = G_FILE_ATTRIBUTE_TYPE_INVALID;
       g_array_insert_val (info->attributes, i, attr);
 
       attrs = (GFileAttribute *)info->attributes->data;
@@ -796,41 +1086,51 @@ g_file_info_create_value (GFileInfo *info,
     }
 }
 
-static GFileAttributeValue *
-g_file_info_create_value_by_name (GFileInfo *info,
-                                 const char *attribute)
+void
+_g_file_info_set_attribute_by_id (GFileInfo                 *info,
+                                  guint32                    attribute,
+                                  GFileAttributeType         type,
+                                  gpointer                   value_p)
 {
-  guint32 attr_id;
+  GFileAttributeValue *value;
 
-  attr_id = lookup_attribute (attribute);
+  value = g_file_info_create_value (info, attribute);
 
-  return g_file_info_create_value (info, attr_id);
+  if (value)
+    _g_file_attribute_value_set_from_pointer (value, type, value_p, TRUE);
 }
 
 /**
  * g_file_info_set_attribute:
  * @info: a #GFileInfo.
  * @attribute: a file attribute key.
- * @attr_value: a #GFileAttributeValue.
+ * @type: a #GFileAttributeType
+ * @value_p: pointer to the value
  * 
- * Sets the @attribute to contain the given @attr_value, 
- * if possible.
- *
+ * Sets the @attribute to contain the given value, if possible.
  **/
 void
-g_file_info_set_attribute (GFileInfo  *info,
-                          const char *attribute,
-                          const GFileAttributeValue *attr_value)
+g_file_info_set_attribute (GFileInfo                 *info,
+                          const char                *attribute,
+                          GFileAttributeType         type,
+                          gpointer                   value_p)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
-  g_return_if_fail (attr_value != NULL);
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_by_id (info, lookup_attribute (attribute), type, value_p);
+}
+
+void
+_g_file_info_set_attribute_object_by_id (GFileInfo *info,
+                                         guint32    attribute,
+                                        GObject   *attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set (value, attr_value);
+    _g_file_attribute_value_set_object (value, attr_value);
 }
 
 /**
@@ -841,22 +1141,68 @@ g_file_info_set_attribute (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
 void
 g_file_info_set_attribute_object (GFileInfo  *info,
                                  const char *attribute,
-                                 GObject *attr_value)
+                                 GObject    *attr_value)
+{
+  g_return_if_fail (G_IS_FILE_INFO (info));
+  g_return_if_fail (attribute != NULL && *attribute != '\0');
+  g_return_if_fail (G_IS_OBJECT (attr_value));
+
+  _g_file_info_set_attribute_object_by_id (info,
+                                           lookup_attribute (attribute),
+                                           attr_value);
+}
+
+void
+_g_file_info_set_attribute_stringv_by_id (GFileInfo *info,
+                                          guint32    attribute,
+                                         char     **attr_value)
 {
   GFileAttributeValue *value;
 
+  value = g_file_info_create_value (info, attribute);
+  if (value)
+    _g_file_attribute_value_set_stringv (value, attr_value);
+}
+
+/**
+ * g_file_info_set_attribute_stringv:
+ * @info: a #GFileInfo.
+ * @attribute: a file attribute key.
+ * @attr_value: a %NULL terminated string array
+ *
+ * Sets the @attribute to contain the given @attr_value,
+ * if possible.
+ *
+ * Sinze: 2.22
+ **/
+void
+g_file_info_set_attribute_stringv (GFileInfo  *info,
+                                  const char *attribute,
+                                  char      **attr_value)
+{
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
-  g_return_if_fail (G_IS_OBJECT (attr_value));
+  g_return_if_fail (attr_value != NULL);
+
+  _g_file_info_set_attribute_stringv_by_id (info, 
+                                            lookup_attribute (attribute),
+                                            attr_value);
+}
+
+void
+_g_file_info_set_attribute_string_by_id (GFileInfo  *info,
+                                         guint32     attribute,
+                                        const char *attr_value)
+{
+  GFileAttributeValue *value;
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_object (value, attr_value);
+    _g_file_attribute_value_set_string (value, attr_value);
 }
 
 /**
@@ -867,22 +1213,31 @@ g_file_info_set_attribute_object (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- *
  **/
 void
 g_file_info_set_attribute_string (GFileInfo  *info,
                                  const char *attribute,
                                  const char *attr_value)
 {
-  GFileAttributeValue *value;
-  
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
   g_return_if_fail (attr_value != NULL);
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_string_by_id (info,
+                                           lookup_attribute (attribute),
+                                           attr_value);
+}
+
+void
+_g_file_info_set_attribute_byte_string_by_id (GFileInfo  *info,
+                                              guint32     attribute,
+                                             const char *attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_string (value, attr_value);
+    _g_file_attribute_value_set_byte_string (value, attr_value);
 }
 
 /**
@@ -893,22 +1248,31 @@ g_file_info_set_attribute_string (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
 void
 g_file_info_set_attribute_byte_string (GFileInfo  *info,
                                       const char *attribute,
                                       const char *attr_value)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
   g_return_if_fail (attr_value != NULL);
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_byte_string_by_id (info,
+                                                lookup_attribute (attribute),
+                                                attr_value);
+}
+
+void
+_g_file_info_set_attribute_boolean_by_id (GFileInfo *info,
+                                          guint32    attribute,
+                                         gboolean   attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_byte_string (value, attr_value);
+    _g_file_attribute_value_set_boolean (value, attr_value);
 }
 
 /**
@@ -919,21 +1283,30 @@ g_file_info_set_attribute_byte_string (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
 void
 g_file_info_set_attribute_boolean (GFileInfo  *info,
                                   const char *attribute,
-                                  gboolean attr_value)
+                                  gboolean    attr_value)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_boolean_by_id (info,
+                                            lookup_attribute (attribute),
+                                            attr_value);
+}
+
+void
+_g_file_info_set_attribute_uint32_by_id (GFileInfo *info,
+                                         guint32    attribute,
+                                        guint32    attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_boolean (value, attr_value);
+    _g_file_attribute_value_set_uint32 (value, attr_value);
 }
 
 /**
@@ -944,24 +1317,31 @@ g_file_info_set_attribute_boolean (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
-
 void
 g_file_info_set_attribute_uint32 (GFileInfo  *info,
                                  const char *attribute,
                                  guint32     attr_value)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
 
-  value = g_file_info_create_value_by_name (info, attribute);
-  if (value)
-    g_file_attribute_value_set_uint32 (value, attr_value);
+  _g_file_info_set_attribute_uint32_by_id (info,
+                                           lookup_attribute (attribute),
+                                           attr_value);
 }
 
+void
+_g_file_info_set_attribute_int32_by_id (GFileInfo *info,
+                                        guint32    attribute,
+                                       gint32     attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
+  if (value)
+    _g_file_attribute_value_set_int32 (value, attr_value);
+}
 
 /**
  * g_file_info_set_attribute_int32:
@@ -971,21 +1351,30 @@ g_file_info_set_attribute_uint32 (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
 void
-g_file_info_set_attribute_int32  (GFileInfo  *info,
-                                 const char *attribute,
-                                 gint32      attr_value)
+g_file_info_set_attribute_int32 (GFileInfo  *info,
+                                 const char *attribute,
+                                 gint32      attr_value)
 {
-  GFileAttributeValue *value;
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_int32_by_id (info,
+                                          lookup_attribute (attribute),
+                                          attr_value);
+}
+
+void
+_g_file_info_set_attribute_uint64_by_id (GFileInfo *info,
+                                         guint32    attribute,
+                                        guint64    attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_int32 (value, attr_value);
+    _g_file_attribute_value_set_uint64 (value, attr_value);
 }
 
 /**
@@ -996,21 +1385,30 @@ g_file_info_set_attribute_int32  (GFileInfo  *info,
  * 
  * Sets the @attribute to contain the given @attr_value, 
  * if possible.
- * 
  **/
 void
 g_file_info_set_attribute_uint64 (GFileInfo  *info,
                                  const char *attribute,
                                  guint64     attr_value)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
 
-  value = g_file_info_create_value_by_name (info, attribute);
+  _g_file_info_set_attribute_uint64_by_id (info,
+                                           lookup_attribute (attribute),
+                                           attr_value);
+}
+
+void
+_g_file_info_set_attribute_int64_by_id (GFileInfo *info,
+                                        guint32    attribute,
+                                       gint64     attr_value)
+{
+  GFileAttributeValue *value;
+
+  value = g_file_info_create_value (info, attribute);
   if (value)
-    g_file_attribute_value_set_uint64 (value, attr_value);
+    _g_file_attribute_value_set_int64 (value, attr_value);
 }
 
 /**
@@ -1028,14 +1426,12 @@ g_file_info_set_attribute_int64  (GFileInfo  *info,
                                  const char *attribute,
                                  gint64      attr_value)
 {
-  GFileAttributeValue *value;
-
   g_return_if_fail (G_IS_FILE_INFO (info));
   g_return_if_fail (attribute != NULL && *attribute != '\0');
 
-  value = g_file_info_create_value_by_name (info, attribute);
-  if (value)
-    g_file_attribute_value_set_int64 (value, attr_value);
+  _g_file_info_set_attribute_int64_by_id (info,
+                                          lookup_attribute (attribute),
+                                          attr_value);
 }
 
 /* Helper getters */
@@ -1057,10 +1453,10 @@ g_file_info_get_file_type (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), G_FILE_TYPE_UNKNOWN);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_TYPE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_TYPE);
   
   value = g_file_info_find_value (info, attr);
-  return (GFileType)g_file_attribute_value_get_uint32 (value);
+  return (GFileType)_g_file_attribute_value_get_uint32 (value);
 }
 
 /**
@@ -1080,10 +1476,10 @@ g_file_info_get_is_hidden (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), FALSE);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_IS_HIDDEN);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_IS_HIDDEN);
   
   value = g_file_info_find_value (info, attr);
-  return (GFileType)g_file_attribute_value_get_boolean (value);
+  return (GFileType)_g_file_attribute_value_get_boolean (value);
 }
 
 /**
@@ -1103,10 +1499,10 @@ g_file_info_get_is_backup (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), FALSE);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_IS_BACKUP);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_IS_BACKUP);
   
   value = g_file_info_find_value (info, attr);
-  return (GFileType)g_file_attribute_value_get_boolean (value);
+  return (GFileType)_g_file_attribute_value_get_boolean (value);
 }
 
 /**
@@ -1126,10 +1522,10 @@ g_file_info_get_is_symlink (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), FALSE);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_IS_SYMLINK);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_IS_SYMLINK);
   
   value = g_file_info_find_value (info, attr);
-  return (GFileType)g_file_attribute_value_get_boolean (value);
+  return (GFileType)_g_file_attribute_value_get_boolean (value);
 }
 
 /**
@@ -1149,10 +1545,10 @@ g_file_info_get_name (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_NAME);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_byte_string (value);
+  return _g_file_attribute_value_get_byte_string (value);
 }
 
 /**
@@ -1172,10 +1568,10 @@ g_file_info_get_display_name (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_DISPLAY_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_DISPLAY_NAME);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_string (value);
+  return _g_file_attribute_value_get_string (value);
 }
 
 /**
@@ -1195,10 +1591,10 @@ g_file_info_get_edit_name (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_EDIT_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_string (value);
+  return _g_file_attribute_value_get_string (value);
 }
 
 /**
@@ -1207,7 +1603,7 @@ g_file_info_get_edit_name (GFileInfo *info)
  * 
  * Gets the icon for a file.
  * 
- * Returns: #GIcon for the given @info.
+ * Returns: (transfer none): #GIcon for the given @info.
  **/
 GIcon *
 g_file_info_get_icon (GFileInfo *info)
@@ -1219,11 +1615,11 @@ g_file_info_get_icon (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_ICON);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_ICON);
   
   value = g_file_info_find_value (info, attr);
-  obj = g_file_attribute_value_get_object (value);
-  if (obj != NULL && G_IS_ICON (obj))
+  obj = _g_file_attribute_value_get_object (value);
+  if (G_IS_ICON (obj))
     return G_ICON (obj);
   return NULL;
 }
@@ -1234,7 +1630,7 @@ g_file_info_get_icon (GFileInfo *info)
  * 
  * Gets the file's content type.
  * 
- * Returns: a string containing the file's content type.s
+ * Returns: a string containing the file's content type.
  **/
 const char *
 g_file_info_get_content_type (GFileInfo *info)
@@ -1245,10 +1641,10 @@ g_file_info_get_content_type (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_CONTENT_TYPE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_string (value);
+  return _g_file_attribute_value_get_string (value);
 }
 
 /**
@@ -1268,10 +1664,10 @@ g_file_info_get_size (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), (goffset) 0);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SIZE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SIZE);
   
   value = g_file_info_find_value (info, attr);
-  return (goffset) g_file_attribute_value_get_uint64 (value);
+  return (goffset) _g_file_attribute_value_get_uint64 (value);
 }
 
 /**
@@ -1281,9 +1677,7 @@ g_file_info_get_size (GFileInfo *info)
  * 
  * Gets the modification time of the current @info and sets it
  * in @result.
- * 
  **/
-
 void
 g_file_info_get_modification_time (GFileInfo *info,
                                   GTimeVal  *result)
@@ -1301,9 +1695,9 @@ g_file_info_get_modification_time (GFileInfo *info,
     }
   
   value = g_file_info_find_value (info, attr_mtime);
-  result->tv_sec = g_file_attribute_value_get_uint64 (value);
+  result->tv_sec = _g_file_attribute_value_get_uint64 (value);
   value = g_file_info_find_value (info, attr_mtime_usec);
-  result->tv_usec = g_file_attribute_value_get_uint32 (value);
+  result->tv_usec = _g_file_attribute_value_get_uint32 (value);
 }
 
 /**
@@ -1323,18 +1717,18 @@ g_file_info_get_symlink_target (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SYMLINK_TARGET);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SYMLINK_TARGET);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_byte_string (value);
+  return _g_file_attribute_value_get_byte_string (value);
 }
 
 /**
  * g_file_info_get_etag:
  * @info: a #GFileInfo.
  * 
- * Gets the entity tag for a given #GFileInfo. 
- * See %G_FILE_ATTRIBUTE_ETAG_VALUE.
+ * Gets the <link linkend="gfile-etag">entity tag</link> for a given 
+ * #GFileInfo. See %G_FILE_ATTRIBUTE_ETAG_VALUE.
  * 
  * Returns: a string containing the value of the "etag:value" attribute.
  **/
@@ -1350,7 +1744,7 @@ g_file_info_get_etag (GFileInfo *info)
     attr = lookup_attribute (G_FILE_ATTRIBUTE_ETAG_VALUE);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_string (value);
+  return _g_file_attribute_value_get_string (value);
 }
 
 /**
@@ -1358,9 +1752,9 @@ g_file_info_get_etag (GFileInfo *info)
  * @info: a #GFileInfo.
  * 
  * Gets the value of the sort_order attribute from the #GFileInfo.
- * See %G_FILE_ATTRIBUTE_STD_SORT_ORDER.
+ * See %G_FILE_ATTRIBUTE_STANDARD_SORT_ORDER.
  * 
- * Returns: a #gint32 containing the value of the "std:sort_order" attribute.
+ * Returns: a #gint32 containing the value of the "standard::sort_order" attribute.
  **/
 gint32
 g_file_info_get_sort_order (GFileInfo *info)
@@ -1371,10 +1765,10 @@ g_file_info_get_sort_order (GFileInfo *info)
   g_return_val_if_fail (G_IS_FILE_INFO (info), 0);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SORT_ORDER);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SORT_ORDER);
   
   value = g_file_info_find_value (info, attr);
-  return g_file_attribute_value_get_int32 (value);
+  return _g_file_attribute_value_get_int32 (value);
 }
 
 /* Helper setters: */
@@ -1384,11 +1778,11 @@ g_file_info_get_sort_order (GFileInfo *info)
  * @type: a #GFileType.
  * 
  * Sets the file type in a #GFileInfo to @type.
- * See %G_FILE_ATTRIBUTE_STD_TYPE.
+ * See %G_FILE_ATTRIBUTE_STANDARD_TYPE.
  **/
 void
-g_file_info_set_file_type (GFileInfo         *info,
-                          GFileType          type)
+g_file_info_set_file_type (GFileInfo *info,
+                          GFileType  type)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1396,11 +1790,11 @@ g_file_info_set_file_type (GFileInfo         *info,
   g_return_if_fail (G_IS_FILE_INFO (info));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_TYPE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_TYPE);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_uint32 (value, type);
+    _g_file_attribute_value_set_uint32 (value, type);
 }
 
 /**
@@ -1409,7 +1803,7 @@ g_file_info_set_file_type (GFileInfo         *info,
  * @is_hidden: a #gboolean.
  * 
  * Sets the "is_hidden" attribute in a #GFileInfo according to @is_symlink.
- * See %G_FILE_ATTRIBUTE_STD_IS_HIDDEN.
+ * See %G_FILE_ATTRIBUTE_STANDARD_IS_HIDDEN.
  **/
 void
 g_file_info_set_is_hidden (GFileInfo *info,
@@ -1421,11 +1815,11 @@ g_file_info_set_is_hidden (GFileInfo *info,
   g_return_if_fail (G_IS_FILE_INFO (info));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_IS_HIDDEN);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_IS_HIDDEN);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_boolean (value, is_hidden);
+    _g_file_attribute_value_set_boolean (value, is_hidden);
 }
 
 /**
@@ -1434,8 +1828,7 @@ g_file_info_set_is_hidden (GFileInfo *info,
  * @is_symlink: a #gboolean.
  * 
  * Sets the "is_symlink" attribute in a #GFileInfo according to @is_symlink.
- * See %G_FILE_ATTRIBUTE_STD_IS_SYMLINK.
- * 
+ * See %G_FILE_ATTRIBUTE_STANDARD_IS_SYMLINK.
  **/
 void
 g_file_info_set_is_symlink (GFileInfo *info,
@@ -1447,11 +1840,11 @@ g_file_info_set_is_symlink (GFileInfo *info,
   g_return_if_fail (G_IS_FILE_INFO (info));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_IS_SYMLINK);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_IS_SYMLINK);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_boolean (value, is_symlink);
+    _g_file_attribute_value_set_boolean (value, is_symlink);
 }
 
 /**
@@ -1460,12 +1853,11 @@ g_file_info_set_is_symlink (GFileInfo *info,
  * @name: a string containing a name.
  * 
  * Sets the name attribute for the current #GFileInfo. 
- * See %G_FILE_ATTRIBUTE_STD_NAME.
- * 
+ * See %G_FILE_ATTRIBUTE_STANDARD_NAME.
  **/
 void
-g_file_info_set_name (GFileInfo         *info,
-                     const char        *name)
+g_file_info_set_name (GFileInfo  *info,
+                     const char *name)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1474,11 +1866,11 @@ g_file_info_set_name (GFileInfo         *info,
   g_return_if_fail (name != NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_NAME);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_byte_string (value, name);
+    _g_file_attribute_value_set_byte_string (value, name);
 }
 
 /**
@@ -1487,12 +1879,11 @@ g_file_info_set_name (GFileInfo         *info,
  * @display_name: a string containing a display name.
  * 
  * Sets the display name for the current #GFileInfo.
- * See %G_FILE_ATTRIBUTE_STD_DISPLAY_NAME.
- *  
+ * See %G_FILE_ATTRIBUTE_STANDARD_DISPLAY_NAME.
  **/
 void
-g_file_info_set_display_name (GFileInfo         *info,
-                             const char        *display_name)
+g_file_info_set_display_name (GFileInfo  *info,
+                             const char *display_name)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1501,11 +1892,11 @@ g_file_info_set_display_name (GFileInfo         *info,
   g_return_if_fail (display_name != NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_DISPLAY_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_DISPLAY_NAME);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_string (value, display_name);
+    _g_file_attribute_value_set_string (value, display_name);
 }
 
 /**
@@ -1514,13 +1905,11 @@ g_file_info_set_display_name (GFileInfo         *info,
  * @edit_name: a string containing an edit name.
  * 
  * Sets the edit name for the current file.
- * See %G_FILE_ATTRIBUTE_STD_EDIT_NAME.
- * 
+ * See %G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME.
  **/
-
 void
-g_file_info_set_edit_name (GFileInfo         *info,
-                          const char        *edit_name)
+g_file_info_set_edit_name (GFileInfo  *info,
+                          const char *edit_name)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1529,11 +1918,11 @@ g_file_info_set_edit_name (GFileInfo         *info,
   g_return_if_fail (edit_name != NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_EDIT_NAME);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_EDIT_NAME);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_string (value, edit_name);
+    _g_file_attribute_value_set_string (value, edit_name);
 }
 
 /**
@@ -1542,12 +1931,11 @@ g_file_info_set_edit_name (GFileInfo         *info,
  * @icon: a #GIcon.
  * 
  * Sets the icon for a given #GFileInfo. 
- * See %G_FILE_ATTRIBUTE_STD_ICON.
- *
+ * See %G_FILE_ATTRIBUTE_STANDARD_ICON.
  **/
 void
-g_file_info_set_icon (GFileInfo   *info,
-                     GIcon       *icon)
+g_file_info_set_icon (GFileInfo *info,
+                     GIcon     *icon)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1556,11 +1944,11 @@ g_file_info_set_icon (GFileInfo   *info,
   g_return_if_fail (G_IS_ICON (icon));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_ICON);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_ICON);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_object (value, G_OBJECT (icon));
+    _g_file_attribute_value_set_object (value, G_OBJECT (icon));
 }
 
 /**
@@ -1569,12 +1957,11 @@ g_file_info_set_icon (GFileInfo   *info,
  * @content_type: a content type. See #GContentType.
  * 
  * Sets the content type attribute for a given #GFileInfo.
- * See %G_FILE_ATTRIBUTE_STD_CONTENT_TYPE.
- *
+ * See %G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE.
  **/
 void
-g_file_info_set_content_type (GFileInfo         *info,
-                             const char        *content_type)
+g_file_info_set_content_type (GFileInfo  *info,
+                             const char *content_type)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1583,11 +1970,11 @@ g_file_info_set_content_type (GFileInfo         *info,
   g_return_if_fail (content_type != NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_CONTENT_TYPE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_string (value, content_type);
+    _g_file_attribute_value_set_string (value, content_type);
 }
 
 /**
@@ -1595,13 +1982,12 @@ g_file_info_set_content_type (GFileInfo         *info,
  * @info: a #GFileInfo.
  * @size: a #goffset containing the file's size.
  * 
- * Sets the %G_FILE_ATTRIBUTE_STD_SIZE attribute in the file info 
+ * Sets the %G_FILE_ATTRIBUTE_STANDARD_SIZE attribute in the file info 
  * to the given size.
- * 
  **/
 void
-g_file_info_set_size (GFileInfo         *info,
-                     goffset            size)
+g_file_info_set_size (GFileInfo *info,
+                     goffset    size)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1609,11 +1995,11 @@ g_file_info_set_size (GFileInfo         *info,
   g_return_if_fail (G_IS_FILE_INFO (info));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SIZE);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SIZE);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_uint64 (value, size);
+    _g_file_attribute_value_set_uint64 (value, size);
 }
 
 /**
@@ -1623,11 +2009,10 @@ g_file_info_set_size (GFileInfo         *info,
  * 
  * Sets the %G_FILE_ATTRIBUTE_TIME_MODIFIED attribute in the file
  * info to the given time value.
- * 
  **/
 void
-g_file_info_set_modification_time (GFileInfo         *info,
-                                  GTimeVal          *mtime)
+g_file_info_set_modification_time (GFileInfo *info,
+                                  GTimeVal  *mtime)
 {
   static guint32 attr_mtime = 0, attr_mtime_usec;
   GFileAttributeValue *value;
@@ -1643,10 +2028,10 @@ g_file_info_set_modification_time (GFileInfo         *info,
   
   value = g_file_info_create_value (info, attr_mtime);
   if (value)
-    g_file_attribute_value_set_uint64 (value, mtime->tv_sec);
+    _g_file_attribute_value_set_uint64 (value, mtime->tv_sec);
   value = g_file_info_create_value (info, attr_mtime_usec);
   if (value)
-    g_file_attribute_value_set_uint32 (value, mtime->tv_usec);
+    _g_file_attribute_value_set_uint32 (value, mtime->tv_usec);
 }
 
 /**
@@ -1654,13 +2039,12 @@ g_file_info_set_modification_time (GFileInfo         *info,
  * @info: a #GFileInfo.
  * @symlink_target: a static string containing a path to a symlink target.
  * 
- * Sets the %G_FILE_ATTRIBUTE_STD_SYMLINK_TARGET attribute in the file info 
+ * Sets the %G_FILE_ATTRIBUTE_STANDARD_SYMLINK_TARGET attribute in the file info 
  * to the given symlink target.
- * 
  **/
 void
-g_file_info_set_symlink_target (GFileInfo         *info,
-                               const char        *symlink_target)
+g_file_info_set_symlink_target (GFileInfo  *info,
+                               const char *symlink_target)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1669,11 +2053,11 @@ g_file_info_set_symlink_target (GFileInfo         *info,
   g_return_if_fail (symlink_target != NULL);
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SYMLINK_TARGET);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SYMLINK_TARGET);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_byte_string (value, symlink_target);
+    _g_file_attribute_value_set_byte_string (value, symlink_target);
 }
 
 /**
@@ -1682,12 +2066,11 @@ g_file_info_set_symlink_target (GFileInfo         *info,
  * @sort_order: a sort order integer.
  * 
  * Sets the sort order attribute in the file info structure. See 
- * %G_FILE_ATTRIBUTE_STD_SORT_ORDER.
- * 
+ * %G_FILE_ATTRIBUTE_STANDARD_SORT_ORDER.
  **/
 void
-g_file_info_set_sort_order (GFileInfo         *info,
-                           gint32             sort_order)
+g_file_info_set_sort_order (GFileInfo *info,
+                           gint32     sort_order)
 {
   static guint32 attr = 0;
   GFileAttributeValue *value;
@@ -1695,56 +2078,14 @@ g_file_info_set_sort_order (GFileInfo         *info,
   g_return_if_fail (G_IS_FILE_INFO (info));
   
   if (attr == 0)
-    attr = lookup_attribute (G_FILE_ATTRIBUTE_STD_SORT_ORDER);
+    attr = lookup_attribute (G_FILE_ATTRIBUTE_STANDARD_SORT_ORDER);
   
   value = g_file_info_create_value (info, attr);
   if (value)
-    g_file_attribute_value_set_int32 (value, sort_order);
+    _g_file_attribute_value_set_int32 (value, sort_order);
 }
 
 
-#define KILOBYTE_FACTOR 1024.0
-#define MEGABYTE_FACTOR (1024.0 * 1024.0)
-#define GIGABYTE_FACTOR (1024.0 * 1024.0 * 1024.0)
-
-/**
- * g_format_file_size_for_display:
- * @size: a file size.
- * 
- * Formats a file size into a human readable string. Sizes are rounded
- * to the nearest metric prefix and are displayed rounded to the nearest 
- * tenth. E.g. the file size 3292528 bytes will be converted into the string 
- * "3.1 MB".
- * 
- * Returns: a formatted string containing a human readable file size.
- **/
-char *
-g_format_file_size_for_display (goffset size)
-{
-  if (size < (goffset) KILOBYTE_FACTOR)
-    return g_strdup_printf (dngettext(GETTEXT_PACKAGE, "%u byte", "%u bytes",(guint) size), (guint) size);
-  else
-    {
-      gdouble displayed_size;
-      
-      if (size < (goffset) MEGABYTE_FACTOR)
-       {
-         displayed_size = (gdouble) size / KILOBYTE_FACTOR;
-         return g_strdup_printf (_("%.1f KB"), displayed_size);
-       }
-      else if (size < (goffset) GIGABYTE_FACTOR)
-       {
-         displayed_size = (gdouble) size / MEGABYTE_FACTOR;
-         return g_strdup_printf (_("%.1f MB"), displayed_size);
-       }
-      else
-       {
-         displayed_size = (gdouble) size / GIGABYTE_FACTOR;
-         return g_strdup_printf (_("%.1f GB"), displayed_size);
-       }
-    }
-}
-
 #define ON_STACK_MATCHERS 5
 
 typedef struct {
@@ -1765,7 +2106,8 @@ struct _GFileAttributeMatcher {
 
 static void
 matcher_add (GFileAttributeMatcher *matcher,
-            guint id, guint mask)
+            guint                  id,
+             guint                  mask)
 {
   SubMatcher *sub_matchers;
   int i;
@@ -1805,20 +2147,39 @@ matcher_add (GFileAttributeMatcher *matcher,
   g_array_append_val (matcher->more_sub_matchers, s);
 }
 
+GType
+g_file_attribute_matcher_get_type (void)
+{
+  static volatile gsize g_define_type_id__volatile = 0;
+
+  if (g_once_init_enter (&g_define_type_id__volatile))
+    {
+      GType g_define_type_id =
+        g_boxed_type_register_static (I_("GFileAttributeMatcher"),
+                                      (GBoxedCopyFunc) g_file_attribute_matcher_ref,
+                                      (GBoxedFreeFunc) g_file_attribute_matcher_unref);
+
+      g_once_init_leave (&g_define_type_id__volatile, g_define_type_id);
+    }
+
+  return g_define_type_id__volatile;
+}
+
 /**
  * g_file_attribute_matcher_new:
  * @attributes: an attribute string to match.
  * 
- * Creates a new file attribute matcher, which matches attributes against
- * a given string. #GFileAttributeMatcher<!-- -->s are reference counted structures, 
- * and are created with a reference count of 1. If the number of references
- * falls to 0, the #GFileAttributeMatcher is automatically destroyed.
+ * Creates a new file attribute matcher, which matches attributes 
+ * against a given string. #GFileAttributeMatcher<!-- -->s are reference 
+ * counted structures, and are created with a reference count of 1. If 
+ * the number of references falls to 0, the #GFileAttributeMatcher is 
+ * automatically destroyed.
  * 
  * The @attribute string should be formatted with specific keys separated
- * from namespaces with a colon. Several "namespace:key" strings may be 
- * concatenated with a single comma (e.g. "std:type,std:is_hidden"). 
- * The wildcard "*" may be used to match all keys and namespaces, or "namespace:*" will 
- * match all keys in a given namespace. 
+ * from namespaces with a double colon. Several "namespace::key" strings may be 
+ * concatenated with a single comma (e.g. "standard::type,standard::is-hidden"). 
+ * The wildcard "*" may be used to match all keys and namespaces, or 
+ * "namespace::*" will match all keys in a given namespace. 
  * 
  * Examples of strings to use:
  * <table>
@@ -1827,9 +2188,9 @@ matcher_add (GFileAttributeMatcher *matcher,
  * <row><entry> Matcher String </entry><entry> Matches </entry></row></thead>
  * <tbody>
  * <row><entry>"*"</entry><entry>matches all attributes.</entry></row>
- * <row><entry>"std:is_hidden"</entry><entry>matches only the key is_hidden in the std namespace.</entry></row>
- * <row><entry>"std:type,unix:*"</entry><entry>matches the type key in the std namespace and all keys in the unix 
- * namespace.</entry></row>
+ * <row><entry>"standard::is-hidden"</entry><entry>matches only the key is-hidden in the standard namespace.</entry></row>
+ * <row><entry>"standard::type,unix::*"</entry><entry>matches the type key in the standard namespace and
+ * all keys in the unix namespace.</entry></row>
  * </tbody></tgroup>
  * </table>
  * 
@@ -1859,11 +2220,11 @@ g_file_attribute_matcher_new (const char *attributes)
        {
          guint32 id, mask;
   
-         colon = strchr (split[i], ':');
+         colon = strstr (split[i], "::");
          if (colon != NULL &&
-             !(colon[1] == 0 ||
-               (colon[1] == '*' &&
-                colon[2] == 0)))
+             !(colon[2] == 0 ||
+               (colon[2] == '*' &&
+                colon[3] == 0)))
            {
              id = lookup_attribute (split[i]);
              mask = 0xffffffff;
@@ -1897,11 +2258,11 @@ g_file_attribute_matcher_new (const char *attributes)
 GFileAttributeMatcher *
 g_file_attribute_matcher_ref (GFileAttributeMatcher *matcher)
 {
-  g_return_val_if_fail (matcher != NULL, NULL);
-  g_return_val_if_fail (matcher->ref > 0, NULL);
-
-  g_atomic_int_inc (&matcher->ref);
-  
+  if (matcher)
+    {
+      g_return_val_if_fail (matcher->ref > 0, NULL);
+      g_atomic_int_inc (&matcher->ref);
+    }
   return matcher;
 }
 
@@ -1916,15 +2277,17 @@ g_file_attribute_matcher_ref (GFileAttributeMatcher *matcher)
 void
 g_file_attribute_matcher_unref (GFileAttributeMatcher *matcher)
 {
-  g_return_if_fail (matcher != NULL);
-  g_return_if_fail (matcher->ref > 0);
-
-  if (g_atomic_int_dec_and_test (&matcher->ref))
+  if (matcher)
     {
-      if (matcher->more_sub_matchers)
-       g_array_free (matcher->more_sub_matchers, TRUE);
+      g_return_if_fail (matcher->ref > 0);
       
-      g_free (matcher);
+      if (g_atomic_int_dec_and_test (&matcher->ref))
+       {
+         if (matcher->more_sub_matchers)
+           g_array_free (matcher->more_sub_matchers, TRUE);
+         
+         g_free (matcher);
+       }
     }
 }
 
@@ -1944,10 +2307,10 @@ g_file_attribute_matcher_matches_only (GFileAttributeMatcher *matcher,
 {
   guint32 id;
 
-  g_return_val_if_fail (matcher != NULL, FALSE);
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', FALSE);
 
-  if (matcher->all)
+  if (matcher == NULL ||
+      matcher->all)
     return FALSE;
   
   id = lookup_attribute (attribute);
@@ -1963,7 +2326,7 @@ g_file_attribute_matcher_matches_only (GFileAttributeMatcher *matcher,
 
 static gboolean
 matcher_matches_id (GFileAttributeMatcher *matcher,
-                   guint32 id)
+                    guint32                id)
 {
   SubMatcher *sub_matchers;
   int i;
@@ -1990,11 +2353,13 @@ matcher_matches_id (GFileAttributeMatcher *matcher,
   return FALSE;
 }
 
-static gboolean
-g_file_attribute_matcher_matches_id (GFileAttributeMatcher *matcher,
-                                    guint32 id)
+gboolean
+_g_file_attribute_matcher_matches_id (GFileAttributeMatcher *matcher,
+                                      guint32                id)
 {
-  g_return_val_if_fail (matcher != NULL, FALSE);
+  /* We return a NULL matcher for an empty match string, so handle this */
+  if (matcher == NULL)
+    return FALSE;
   
   if (matcher->all)
     return TRUE;
@@ -2017,9 +2382,12 @@ gboolean
 g_file_attribute_matcher_matches (GFileAttributeMatcher *matcher,
                                  const char            *attribute)
 {
-  g_return_val_if_fail (matcher != NULL, FALSE);
   g_return_val_if_fail (attribute != NULL && *attribute != '\0', FALSE);
 
+  /* We return a NULL matcher for an empty match string, so handle this */
+  if (matcher == NULL)
+    return FALSE;
+  
   if (matcher->all)
     return TRUE;
   
@@ -2034,7 +2402,7 @@ g_file_attribute_matcher_matches (GFileAttributeMatcher *matcher,
  * 
  * Checks if the matcher will match all of the keys in a given namespace.
  * This will always return %TRUE if a wildcard character is in use (e.g. if 
- * matcher was created with "std:*" and @ns is "std", or if matcher was created
+ * matcher was created with "standard::*" and @ns is "standard", or if matcher was created
  * using "*" and namespace is anything.) 
  * 
  * TODO: this is awkwardly worded.
@@ -2050,8 +2418,11 @@ g_file_attribute_matcher_enumerate_namespace (GFileAttributeMatcher *matcher,
   int ns_id;
   int i;
   
-  g_return_val_if_fail (matcher != NULL, FALSE);
   g_return_val_if_fail (ns != NULL && *ns != '\0', FALSE);
+
+  /* We return a NULL matcher for an empty match string, so handle this */
+  if (matcher == NULL)
+    return FALSE;
   
   if (matcher->all)
     return TRUE;
@@ -2094,8 +2465,10 @@ g_file_attribute_matcher_enumerate_next (GFileAttributeMatcher *matcher)
 {
   int i;
   SubMatcher *sub_matcher;
-  
-  g_return_val_if_fail (matcher != NULL, NULL);
+
+  /* We return a NULL matcher for an empty match string, so handle this */
+  if (matcher == NULL)
+    return NULL;
 
   while (1)
     {