diff options
author | Will Thompson <will.thompson@collabora.co.uk> | 2009-09-09 20:35:13 +0100 |
---|---|---|
committer | Will Thompson <will.thompson@collabora.co.uk> | 2009-11-01 13:46:45 +0000 |
commit | 1e14e409a48a96e0599c8f675604735ca132fc0f (patch) | |
tree | 931ce9a0b36d05ec59e8f63881ed9c0c18c6fa43 /tools/dbus-print-message.c | |
parent | 03cc20707a3e7b2d8629e84d7a766f41edb8b444 (diff) | |
download | dbus-1e14e409a48a96e0599c8f675604735ca132fc0f.tar.gz |
Make array-printing code easier to follow
Previously dbus_message_iter_get_arg_type() was called twice: once in
the loop condition to update 'current_type', and once to check if the
loop will run again. This patch moves updating current_type to the end
of the loop body.
Diffstat (limited to 'tools/dbus-print-message.c')
-rw-r--r-- | tools/dbus-print-message.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tools/dbus-print-message.c b/tools/dbus-print-message.c index 335aa3dc..749fca68 100644 --- a/tools/dbus-print-message.c +++ b/tools/dbus-print-message.c @@ -186,12 +186,17 @@ print_iter (DBusMessageIter *iter, dbus_bool_t literal, int depth) dbus_message_iter_recurse (iter, &subiter); + current_type = dbus_message_iter_get_arg_type (&subiter); + printf("array [\n"); - while ((current_type = dbus_message_iter_get_arg_type (&subiter)) != DBUS_TYPE_INVALID) + while (current_type != DBUS_TYPE_INVALID) { print_iter (&subiter, literal, depth+1); + dbus_message_iter_next (&subiter); - if (dbus_message_iter_get_arg_type (&subiter) != DBUS_TYPE_INVALID) + current_type = dbus_message_iter_get_arg_type (&subiter); + + if (current_type != DBUS_TYPE_INVALID) printf (","); } indent(depth); |