From f89a50e24d2088061eb7d975db7e6f39beaf853d Mon Sep 17 00:00:00 2001 From: Emmanuele Bassi Date: Mon, 10 Jan 2011 11:16:17 +0000 Subject: object: Do some more validation in set_member() Check if we're setting the same node, to avoid a needless replace. --- json-glib/json-object.c | 10 ++++++++++ 1 file changed, 10 insertions(+) (limited to 'json-glib/json-object.c') diff --git a/json-glib/json-object.c b/json-glib/json-object.c index de91ebb..0b5875f 100644 --- a/json-glib/json-object.c +++ b/json-glib/json-object.c @@ -188,10 +188,20 @@ json_object_set_member (JsonObject *object, const gchar *member_name, JsonNode *node) { + JsonNode *old_node; + g_return_if_fail (object != NULL); g_return_if_fail (member_name != NULL); g_return_if_fail (node != NULL); + old_node = g_hash_table_lookup (object->members, member_name); + if (old_node == NULL) + goto set_member; + + if (old_node == node) + return; + +set_member: object_set_member_internal (object, member_name, node); } -- cgit v1.2.1