Remove g_file_attribute_value_destroy (replace with clear)

Add const attributes in GFileAttributeValue methods

Original git commit by Alexander Larsson <alexl@redhat.com> at 1183712949 +0200

svn path=/trunk/; revision=655

5 files changed, 35 insertions, 41 deletions

diff --git a/common/gvfsdaemonprotocol.c b/common/gvfsdaemonprotocol.c
index ebd0806a..dc1f6b21 100644
--- a/common/gvfsdaemonprotocol.c
+++ b/common/gvfsdaemonprotocol.c
@@ -247,7 +247,7 @@ _g_dbus_get_file_attribute (DBusMessageIter *iter,
   const gchar *attribute_temp;
   dbus_uint32_t obj_type;
 
-  g_file_attribute_value_destroy (value);
+  g_file_attribute_value_clear (value);
   
   dbus_message_iter_recurse (iter, &inner_struct_iter);
       
diff --git a/daemon/gvfsjobsetattribute.c b/daemon/gvfsjobsetattribute.c
index 70c3a843..2a1d0020 100644
--- a/daemon/gvfsjobsetattribute.c
+++ b/daemon/gvfsjobsetattribute.c
@@ -30,7 +30,7 @@ g_vfs_job_set_attribute_finalize (GObject *object)
 
   g_free (job->filename);
   g_free (job->attribute);
-  g_file_attribute_value_destroy (&job->value);
+  g_file_attribute_value_clear (&job->value);
   
   if (G_OBJECT_CLASS (g_vfs_job_set_attribute_parent_class)->finalize)
     (*G_OBJECT_CLASS (g_vfs_job_set_attribute_parent_class)->finalize) (object);
diff --git a/gio/gfileattribute.c b/gio/gfileattribute.c
index eaa41268..7d2d2625 100644
--- a/gio/gfileattribute.c
+++ b/gio/gfileattribute.c
@@ -8,12 +8,12 @@
 void
 g_file_attribute_value_free (GFileAttributeValue *attr)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   g_free (attr);
 }
 
 void
-g_file_attribute_value_destroy (GFileAttributeValue *attr)
+g_file_attribute_value_clear (GFileAttributeValue *attr)
 {
   if (attr->type == G_FILE_ATTRIBUTE_TYPE_STRING ||
       attr->type == G_FILE_ATTRIBUTE_TYPE_BYTE_STRING)
@@ -22,20 +22,15 @@ g_file_attribute_value_destroy (GFileAttributeValue *attr)
   if (attr->type == G_FILE_ATTRIBUTE_TYPE_OBJECT &&
       attr->u.obj != NULL)
     g_object_unref (attr->u.obj);
-}
-
-void
-g_file_attribute_value_clear (GFileAttributeValue *attr)
-{
-  g_file_attribute_value_destroy (attr);
+  
   attr->type = G_FILE_ATTRIBUTE_TYPE_INVALID;
 }
 
 void
 g_file_attribute_value_set (GFileAttributeValue *attr,
-			    GFileAttributeValue *new_value)
+			    const GFileAttributeValue *new_value)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   *attr = *new_value;
 
   if (attr->type == G_FILE_ATTRIBUTE_TYPE_STRING ||
@@ -58,7 +53,7 @@ g_file_attribute_value_new (void)
 }
 
 GFileAttributeValue *
-g_file_attribute_value_dup (GFileAttributeValue *old_attr)
+g_file_attribute_value_dup (const GFileAttributeValue *old_attr)
 {
   GFileAttributeValue *attr;
 
@@ -118,7 +113,7 @@ escape_byte_string (const char *str)
 }
 
 char *
-g_file_attribute_value_as_string (GFileAttributeValue *attr)
+g_file_attribute_value_as_string (const GFileAttributeValue *attr)
 {
   char *str;
 
@@ -152,7 +147,7 @@ g_file_attribute_value_as_string (GFileAttributeValue *attr)
 }
 
 const char *
-g_file_attribute_value_get_string (GFileAttributeValue *attr)
+g_file_attribute_value_get_string (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_STRING)
     {
@@ -164,7 +159,7 @@ g_file_attribute_value_get_string (GFileAttributeValue *attr)
 }
 
 const char *
-g_file_attribute_value_get_byte_string (GFileAttributeValue *attr)
+g_file_attribute_value_get_byte_string (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_BYTE_STRING)
     {
@@ -176,7 +171,7 @@ g_file_attribute_value_get_byte_string (GFileAttributeValue *attr)
 }
   
 guint32
-g_file_attribute_value_get_uint32 (GFileAttributeValue *attr)
+g_file_attribute_value_get_uint32 (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_UINT32)
     {
@@ -188,7 +183,7 @@ g_file_attribute_value_get_uint32 (GFileAttributeValue *attr)
 }
   
 gint32
-g_file_attribute_value_get_int32 (GFileAttributeValue *attr)
+g_file_attribute_value_get_int32 (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_INT32)
     {
@@ -200,7 +195,7 @@ g_file_attribute_value_get_int32 (GFileAttributeValue *attr)
 }
   
 guint64
-g_file_attribute_value_get_uint64 (GFileAttributeValue *attr)
+g_file_attribute_value_get_uint64 (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_UINT64)
     {
@@ -212,7 +207,7 @@ g_file_attribute_value_get_uint64 (GFileAttributeValue *attr)
 }
   
 gint64
-g_file_attribute_value_get_int64 (GFileAttributeValue *attr)
+g_file_attribute_value_get_int64 (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_INT64)
     {
@@ -224,7 +219,7 @@ g_file_attribute_value_get_int64 (GFileAttributeValue *attr)
 }
 
 GObject *
-g_file_attribute_value_get_object (GFileAttributeValue *attr)
+g_file_attribute_value_get_object (const GFileAttributeValue *attr)
 {
   if (attr == NULL || attr->type != G_FILE_ATTRIBUTE_TYPE_OBJECT)
     {
@@ -242,7 +237,7 @@ void
 g_file_attribute_value_set_string (GFileAttributeValue *attr,
 				   const char          *string)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_STRING;
   attr->u.string = g_strdup (string);
 }
@@ -252,7 +247,7 @@ void
 g_file_attribute_value_set_byte_string (GFileAttributeValue *attr,
 					const char          *string)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_BYTE_STRING;
   attr->u.string = g_strdup (string);
 }
@@ -261,7 +256,7 @@ void
 g_file_attribute_value_set_uint32 (GFileAttributeValue *attr,
 				   guint32              value)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_UINT32;
   attr->u.uint32 = value;
 }
@@ -270,7 +265,7 @@ void
 g_file_attribute_value_set_int32 (GFileAttributeValue *attr,
 				  gint32               value)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_INT32;
   attr->u.int32 = value;
 }
@@ -279,7 +274,7 @@ void
 g_file_attribute_value_set_uint64 (GFileAttributeValue *attr,
 				   guint64              value)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_UINT64;
   attr->u.uint64 = value;
 }
@@ -288,7 +283,7 @@ void
 g_file_attribute_value_set_int64 (GFileAttributeValue *attr,
 				  gint64               value)
 {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_INT64;
   attr->u.int64 = value;
 }
@@ -297,7 +292,7 @@ void
 g_file_attribute_value_set_object (GFileAttributeValue *attr,
 				   GObject             *obj)
   {
-  g_file_attribute_value_destroy (attr);
+  g_file_attribute_value_clear (attr);
   attr->type = G_FILE_ATTRIBUTE_TYPE_OBJECT;
   if (obj)
     attr->u.obj = g_object_ref (obj);
diff --git a/gio/gfileattribute.h b/gio/gfileattribute.h
index eb34b17d..b3edb76e 100644
--- a/gio/gfileattribute.h
+++ b/gio/gfileattribute.h
@@ -35,21 +35,20 @@ typedef struct  {
 
 GFileAttributeValue *g_file_attribute_value_new             (void);
 void                 g_file_attribute_value_free            (GFileAttributeValue *attr);
-void                 g_file_attribute_value_destroy         (GFileAttributeValue *attr);
 void                 g_file_attribute_value_clear           (GFileAttributeValue *attr);
 void                 g_file_attribute_value_set             (GFileAttributeValue *attr,
-							     GFileAttributeValue *new_value);
-GFileAttributeValue *g_file_attribute_value_dup             (GFileAttributeValue *attr);
+							     const GFileAttributeValue *new_value);
+GFileAttributeValue *g_file_attribute_value_dup             (const GFileAttributeValue *attr);
 
-char *               g_file_attribute_value_as_string       (GFileAttributeValue *attr);
+char *               g_file_attribute_value_as_string       (const GFileAttributeValue *attr);
 
-const char *         g_file_attribute_value_get_string      (GFileAttributeValue *attr);
-const char *         g_file_attribute_value_get_byte_string (GFileAttributeValue *attr);
-guint32              g_file_attribute_value_get_uint32      (GFileAttributeValue *attr);
-gint32               g_file_attribute_value_get_int32       (GFileAttributeValue *attr);
-guint64              g_file_attribute_value_get_uint64      (GFileAttributeValue *attr);
-gint64               g_file_attribute_value_get_int64       (GFileAttributeValue *attr);
-GObject *            g_file_attribute_value_get_object      (GFileAttributeValue *attr);
+const char *         g_file_attribute_value_get_string      (const GFileAttributeValue *attr);
+const char *         g_file_attribute_value_get_byte_string (const GFileAttributeValue *attr);
+guint32              g_file_attribute_value_get_uint32      (const GFileAttributeValue *attr);
+gint32               g_file_attribute_value_get_int32       (const GFileAttributeValue *attr);
+guint64              g_file_attribute_value_get_uint64      (const GFileAttributeValue *attr);
+gint64               g_file_attribute_value_get_int64       (const GFileAttributeValue *attr);
+GObject *            g_file_attribute_value_get_object      (const GFileAttributeValue *attr);
 
 void                 g_file_attribute_value_set_string      (GFileAttributeValue *attr,
 							     const char          *value);
diff --git a/gio/gfileinfo.c b/gio/gfileinfo.c
index 133ca1b0..8b0e16da 100644
--- a/gio/gfileinfo.c
+++ b/gio/gfileinfo.c
@@ -159,7 +159,7 @@ g_file_info_finalize (GObject *object)
 
   attrs = (GFileAttribute *)info->attributes->data;
   for (i = 0; i < info->attributes->len; i++)
-    g_file_attribute_value_destroy (&attrs[i].value);
+    g_file_attribute_value_clear (&attrs[i].value);
   g_array_free (info->attributes, TRUE);  
   
   if (G_OBJECT_CLASS (g_file_info_parent_class)->finalize)
@@ -332,7 +332,7 @@ g_file_info_remove_attribute (GFileInfo  *info,
   if (i < info->attributes->len &&
       attrs[i].attribute == attr_id)
     {
-      g_file_attribute_value_destroy (&attrs[i].value);
+      g_file_attribute_value_clear (&attrs[i].value);
       g_array_remove_index (info->attributes, i);
     }
 }