diff options
author | Dan Winship <danw@gnome.org> | 2014-10-01 13:41:49 -0400 |
---|---|---|
committer | Dan Winship <danw@gnome.org> | 2014-10-03 09:36:27 -0400 |
commit | 5c2d2eed30610b24dc1aaaf48548696d8ef7fa80 (patch) | |
tree | 524dd263b87f96d6975bc86e41962ddb84c2415d | |
parent | 3b1528417aa7072b62596344184ee0ce0519aaa4 (diff) | |
download | NetworkManager-5c2d2eed30610b24dc1aaaf48548696d8ef7fa80.tar.gz |
libnm: fix properties-changed debug logging
The code was mistakenly still using G_VALUE_TYPE_NAME() instead of
g_variant_get_type_string().
Also, refer to the properties with standard gtk-doc punctuation:
"TypeName:property-name".
-rw-r--r-- | libnm/nm-object.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/libnm/nm-object.c b/libnm/nm-object.c index f3d758e422..026cb24aef 100644 --- a/libnm/nm-object.c +++ b/libnm/nm-object.c @@ -1198,11 +1198,11 @@ handle_property_changed (NMObject *self, const char *dbus_name, if (G_UNLIKELY (debug)) { char *s; s = g_variant_print (value, FALSE); - dbgmsg ("PC: (%p) %s::%s => '%s' (%s%s%s)", + dbgmsg ("PC: (%p) %s:%s => '%s' (%s%s%s)", self, G_OBJECT_TYPE_NAME (self), prop_name, s, - G_VALUE_TYPE_NAME (value), + g_variant_get_type_string (value), pi->object_type ? " / " : "", pi->object_type ? g_type_name (pi->object_type) : ""); g_free (s); @@ -1327,7 +1327,7 @@ demarshal_generic (NMObject *object, HANDLE_TYPE (G_TYPE_LONG, G_VARIANT_TYPE_INT64, glong, g_variant_get_int64) HANDLE_TYPE (G_TYPE_ULONG, G_VARIANT_TYPE_UINT64, gulong, g_variant_get_uint64) } else { - dbgmsg ("%s: %s/%s unhandled type %s.", + dbgmsg ("%s: %s:%s unhandled type %s.", __func__, G_OBJECT_TYPE_NAME (object), pspec->name, @@ -1339,9 +1339,9 @@ done: if (success) { _nm_object_queue_notify (object, pspec->name); } else { - dbgmsg ("%s: %s/%s (type %s) couldn't be set with type %s.", + dbgmsg ("%s: %s:%s (type %s) couldn't be set from D-Bus type %s.", __func__, G_OBJECT_TYPE_NAME (object), pspec->name, - g_type_name (pspec->value_type), G_VALUE_TYPE_NAME (value)); + g_type_name (pspec->value_type), g_variant_get_type_string (value)); } return success; } |