chiark / gitweb /
libsystemd-bus: true/false instead of yes/no in msg dump
authorLukasz Skalski <l.skalski@partner.samsung.com>
Tue, 17 Dec 2013 09:55:28 +0000 (10:55 +0100)
committerLennart Poettering <lennart@poettering.net>
Tue, 17 Dec 2013 20:45:43 +0000 (21:45 +0100)
commit5232c42ec43e86f90a850d965a33bb413b9e5a00
tree4080b20787a3b6d1857f8c1988ad87416a31b8f4
parent0d647f7f73fa17074a1df81e2bdd00877554e436
libsystemd-bus: true/false instead of yes/no in msg dump

Due to this patch, message dump (for message which includes boolean
type) is more consistent with dbus-send (which display true/false
instead of yes/no for boolean). It's only simple 'cosmetics change'.

** For dbus-send **

dbus-send --system --dest=org.freedesktop.DBus --type=method_call --print-reply / org.freedesktop.DBus.NameHasOwner string:org.freedesktop.login1

method return sender=org.freedesktop.DBus -> dest=:1.97 reply_serial=2
   boolean true

** For libsystemd-bus (without this patch) **

‣ Type=method_call  Endian=l  Flags=0  Version=2 Serial=8
  Destination=org.freedesktop.DBus  Path=/org/freedesktop/DBus  Interface=org.freedesktop.DBus  Member=NameHasOwner
  MESSAGE "s" {
   STRING "org.freedesktop.login1";
  };

‣ Type=method_return  Endian=l  Flags=1  Version=2 Serial=51  ReplySerial=8
  Sender=:1.59  Destination=:1.67
  UniqueName=:1.59  WellKnownNames={org.freedesktop.DBus}
  MESSAGE "b" {
   BOOLEAN yes;
  };

For me true/false seems to be better readable than yes/no for BOOLEAN.
src/libsystemd-bus/bus-dump.c
src/shared/util.h