From: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: Tue, 17 Jan 2017 20:42:31 +0000 (+0000)
Subject: dbus-monitor: handle default case for binary mode header
X-Git-Tag: dbus-1.12.0~198
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=67c3ad422cffdd59a8e1706bcc7f875c54f03314;p=platform%2Fupstream%2Fdbus.git

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
---

diff --git a/tools/dbus-monitor.c b/tools/dbus-monitor.c
index 4edee29..c6f36d5 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: