Make some functions that take a GError return boolean instead of void.

2007-11-07  Matthias Clasen <mclasen@redhat.com>

        * glib/gkeyfile.[hc]: Make some functions that take
        a GError return boolean instead of void.  (#375651, Matt Barnes)



svn path=/trunk/; revision=5815
This commit is contained in:
Matthias Clasen 2007-11-08 03:19:37 +00:00 committed by Matthias Clasen
parent 5c971cb504
commit 1dc3d6f088
3 changed files with 110 additions and 76 deletions

View File

@ -1,3 +1,8 @@
2007-11-07 Matthias Clasen <mclasen@redhat.com>
* glib/gkeyfile.[hc]: Make some functions that take
a GError return boolean instead of void. (#375651, Matt Barnes)
2007-11-07 Matthias Clasen <mclasen@redhat.com>
* autogen.sh: Use automake 1.9

View File

@ -2436,7 +2436,7 @@ g_key_file_set_double_list (GKeyFile *key_file,
g_string_free (values, TRUE);
}
static void
static gboolean
g_key_file_set_key_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
@ -2455,7 +2455,7 @@ g_key_file_set_key_comment (GKeyFile *key_file,
_("Key file does not have group '%s'"),
group_name ? group_name : "(null)");
return;
return FALSE;
}
/* First find the key the comments are supposed to be
@ -2469,7 +2469,7 @@ g_key_file_set_key_comment (GKeyFile *key_file,
G_KEY_FILE_ERROR_KEY_NOT_FOUND,
_("Key file does not have key '%s' in group '%s'"),
key, group->name);
return;
return FALSE;
}
/* Then find all the comments already associated with the
@ -2492,7 +2492,7 @@ g_key_file_set_key_comment (GKeyFile *key_file,
}
if (comment == NULL)
return;
return TRUE;
/* Now we can add our new comment
*/
@ -2501,9 +2501,11 @@ g_key_file_set_key_comment (GKeyFile *key_file,
pair->value = g_key_file_parse_comment_as_value (key_file, comment);
key_node = g_list_insert (key_node, pair, 1);
return TRUE;
}
static void
static gboolean
g_key_file_set_group_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *comment,
@ -2511,7 +2513,7 @@ g_key_file_set_group_comment (GKeyFile *key_file,
{
GKeyFileGroup *group;
g_return_if_fail (g_key_file_is_group_name (group_name));
g_return_val_if_fail (g_key_file_is_group_name (group_name), FALSE);
group = g_key_file_lookup_group (key_file, group_name);
if (!group)
@ -2521,7 +2523,7 @@ g_key_file_set_group_comment (GKeyFile *key_file,
_("Key file does not have group '%s'"),
group_name ? group_name : "(null)");
return;
return FALSE;
}
/* First remove any existing comment
@ -2533,16 +2535,18 @@ g_key_file_set_group_comment (GKeyFile *key_file,
}
if (comment == NULL)
return;
return TRUE;
/* Now we can add our new comment
*/
group->comment = g_slice_new (GKeyFileKeyValuePair);
group->comment->key = NULL;
group->comment->value = g_key_file_parse_comment_as_value (key_file, comment);
return TRUE;
}
static void
static gboolean
g_key_file_set_top_comment (GKeyFile *key_file,
const gchar *comment,
GError **error)
@ -2572,7 +2576,7 @@ g_key_file_set_top_comment (GKeyFile *key_file,
}
if (comment == NULL)
return;
return TRUE;
pair = g_slice_new (GKeyFileKeyValuePair);
pair->key = NULL;
@ -2580,6 +2584,8 @@ g_key_file_set_top_comment (GKeyFile *key_file,
group->key_value_pairs =
g_list_prepend (group->key_value_pairs, pair);
return TRUE;
}
/**
@ -2591,31 +2597,44 @@ g_key_file_set_top_comment (GKeyFile *key_file,
* @error: return location for a #GError
*
* Places a comment above @key from @group_name.
* @group_name. If @key is %NULL then @comment will
* be written above @group_name. If both @key
* and @group_name are NULL, then @comment will
* be written above the first group in the file.
* If @key is %NULL then @comment will be written
* above @group_name. If both @key and @group_name
* are NULL, then @comment will be written above
* the first group in the file.
*
* Returns: %TRUE if the comment was written, %FALSE otherwise
*
* Since: 2.6
**/
void
gboolean
g_key_file_set_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
const gchar *comment,
GError **error)
{
g_return_if_fail (key_file != NULL);
g_return_val_if_fail (key_file != NULL, FALSE);
if (group_name != NULL && key != NULL)
g_key_file_set_key_comment (key_file, group_name, key, comment, error);
{
if (!g_key_file_set_key_comment (key_file, group_name, key, comment, error))
return FALSE;
}
else if (group_name != NULL)
g_key_file_set_group_comment (key_file, group_name, comment, error);
{
if (!g_key_file_set_group_comment (key_file, group_name, comment, error))
return FALSE;
}
else
g_key_file_set_top_comment (key_file, comment, error);
{
if (!g_key_file_set_top_comment (key_file, comment, error))
return FALSE;
}
if (comment != NULL)
key_file->approximate_size += strlen (comment);
return TRUE;
}
static gchar *
@ -2851,23 +2870,25 @@ g_key_file_get_comment (GKeyFile *key_file,
* and @group_name are NULL, then @comment will
* be written above the first group in the file.
*
* Returns: %TRUE if the comment was removed, %FALSE otherwise
*
* Since: 2.6
**/
void
gboolean
g_key_file_remove_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
GError **error)
{
g_return_if_fail (key_file != NULL);
g_return_val_if_fail (key_file != NULL, FALSE);
if (group_name != NULL && key != NULL)
g_key_file_set_key_comment (key_file, group_name, key, NULL, error);
return g_key_file_set_key_comment (key_file, group_name, key, NULL, error);
else if (group_name != NULL)
g_key_file_set_group_comment (key_file, group_name, NULL, error);
return g_key_file_set_group_comment (key_file, group_name, NULL, error);
else
g_key_file_set_top_comment (key_file, NULL, error);
return g_key_file_set_top_comment (key_file, NULL, error);
}
/**
@ -3087,17 +3108,19 @@ g_key_file_remove_group_node (GKeyFile *key_file,
* Removes the specified group, @group_name,
* from the key file.
*
* Returns: %TRUE if the group was removed, %FALSE otherwise
*
* Since: 2.6
**/
void
gboolean
g_key_file_remove_group (GKeyFile *key_file,
const gchar *group_name,
GError **error)
{
GList *group_node;
g_return_if_fail (key_file != NULL);
g_return_if_fail (group_name != NULL);
g_return_val_if_fail (key_file != NULL, FALSE);
g_return_val_if_fail (group_name != NULL, FALSE);
group_node = g_key_file_lookup_group_node (key_file, group_name);
@ -3107,10 +3130,12 @@ g_key_file_remove_group (GKeyFile *key_file,
G_KEY_FILE_ERROR_GROUP_NOT_FOUND,
_("Key file does not have group '%s'"),
group_name);
return;
return FALSE;
}
g_key_file_remove_group_node (key_file, group_node);
return TRUE;
}
static void
@ -3140,9 +3165,11 @@ g_key_file_add_key (GKeyFile *key_file,
*
* Removes @key in @group_name from the key file.
*
* Returns: %TRUE if the key was removed, %FALSE otherwise
*
* Since: 2.6
**/
void
gboolean
g_key_file_remove_key (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
@ -3151,9 +3178,9 @@ g_key_file_remove_key (GKeyFile *key_file,
GKeyFileGroup *group;
GKeyFileKeyValuePair *pair;
g_return_if_fail (key_file != NULL);
g_return_if_fail (group_name != NULL);
g_return_if_fail (key != NULL);
g_return_val_if_fail (key_file != NULL, FALSE);
g_return_val_if_fail (group_name != NULL, FALSE);
g_return_val_if_fail (key != NULL, FALSE);
pair = NULL;
@ -3164,7 +3191,7 @@ g_key_file_remove_key (GKeyFile *key_file,
G_KEY_FILE_ERROR_GROUP_NOT_FOUND,
_("Key file does not have group '%s'"),
group_name ? group_name : "(null)");
return;
return FALSE;
}
pair = g_key_file_lookup_key_value_pair (key_file, group, key);
@ -3175,7 +3202,7 @@ g_key_file_remove_key (GKeyFile *key_file,
G_KEY_FILE_ERROR_KEY_NOT_FOUND,
_("Key file does not have key '%s' in group '%s'"),
key, group->name);
return;
return FALSE;
}
key_file->approximate_size -= strlen (pair->key) + strlen (pair->value) + 2;
@ -3183,6 +3210,8 @@ g_key_file_remove_key (GKeyFile *key_file,
group->key_value_pairs = g_list_remove (group->key_value_pairs, pair);
g_hash_table_remove (group->lookup_map, pair->key);
g_key_file_key_value_pair_free (pair);
return TRUE;
}
static GList *

View File

@ -192,7 +192,7 @@ void g_key_file_set_integer_list (GKeyFile *key_file,
const gchar *key,
gint list[],
gsize length);
void g_key_file_set_comment (GKeyFile *key_file,
gboolean g_key_file_set_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
const gchar *comment,
@ -202,15 +202,15 @@ gchar *g_key_file_get_comment (GKeyFile *key_file,
const gchar *key,
GError **error) G_GNUC_MALLOC;
void g_key_file_remove_comment (GKeyFile *key_file,
gboolean g_key_file_remove_comment (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
GError **error);
void g_key_file_remove_key (GKeyFile *key_file,
gboolean g_key_file_remove_key (GKeyFile *key_file,
const gchar *group_name,
const gchar *key,
GError **error);
void g_key_file_remove_group (GKeyFile *key_file,
gboolean g_key_file_remove_group (GKeyFile *key_file,
const gchar *group_name,
GError **error);