diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2017-01-17 20:42:31 +0000 |
|---|---|---|
| committer | Simon McVittie <smcv@collabora.com> | 2017-04-07 12:07:21 +0100 |
| commit | 67c3ad422cffdd59a8e1706bcc7f875c54f03314 (patch) | |
| tree | 06df28e5e940b62cd38a4fec5d68076e89270e54 /tools | |
| parent | 40560fa0c7ec921c8144296c72a80d20f403b8bf (diff) | |
| download | dbus-67c3ad422cffdd59a8e1706bcc7f875c54f03314.tar.gz | |
dbus-monitor: handle default case for binary mode header
Also comment why it's OK to not do anything for the modes that don't
have a header. We are effectively treating the default case as one
of those, on the assumption that future modes are more likely to
lack a header than to have one.
Based on part of a patch from Thomas Zimmermann.
Signed-off-by: Simon McVittie <smcv@collabora.com>
Reviewed-by: Philip Withnall <withnall@endlessm.com>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=98191
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/dbus-monitor.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/dbus-monitor.c b/tools/dbus-monitor.c index 4edee29e..c6f36d59 100644 --- a/tools/dbus-monitor.c +++ b/tools/dbus-monitor.c @@ -555,6 +555,8 @@ main (int argc, char *argv[]) { case BINARY_MODE_NOT: case BINARY_MODE_RAW: + default: + /* no special header needed */ break; case BINARY_MODE_PCAP: |