Attachment 79025 Details for Bug 64362 – Fix compiler warnings when X11 autolaunch and launchd are both disabled

[patch] Fix compiler warnings when X11 autolaunch and launchd are both disabled

0001-Fix-compiler-warnings-when-X11-autolaunch-and-launch.patch (text/plain), 2.32 KB, created by Simon McVittie on 2013-05-08 14:38:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2013-05-08 14:38:34 UTC

Size: 2.32 KB

patch

obsolete

>From 74cb08ee8dd6826946a2ba17e7b821f8e74125ca Mon Sep 17 00:00:00 2001
>From: Simon McVittie <simon.mcvittie@collabora.co.uk>
>Date: Wed, 8 May 2013 15:26:14 +0100
>Subject: [PATCH] Fix compiler warnings when X11 autolaunch and launchd are
> both disabled
>
>From the department of "if it isn't tested, it doesn't work". I tried
>compiling dbus without an assortment of optional features:
>
>    in_builddir ~/build/dbus/legacy ${MR_REPO}/configure \
>        --enable-developer --enable-maintainer-mode --enable-tests \
>        dbus_cv_sync_sub_and_fetch=no \
>        --disable-selinux \
>        --disable-inotify \
>        --disable-dnotify \
>        --disable-epoll \
>        --disable-kqueue \
>        --disable-launchd \
>        --disable-systemd \
>        --disable-libaudit \
>        --without-valgrind \
>        --disable-x11-autolaunch \
>        && ...
>
>and it resulted in -Wunused warnings.
>
>The tests also fail with dbus_cv_sync_sub_and_fetch=no, but that's
>a separate issue.
>
>Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
>---
> dbus/dbus-sysdeps-unix.c |    2 ++
> tools/dbus-launch.c      |    3 ++-
> 2 files changed, 4 insertions(+), 1 deletion(-)
>
>diff --git a/dbus/dbus-sysdeps-unix.c b/dbus/dbus-sysdeps-unix.c
>index 88db415..ebeb4e5 100644
>--- a/dbus/dbus-sysdeps-unix.c
>+++ b/dbus/dbus-sysdeps-unix.c
>@@ -3207,6 +3207,7 @@ _dbus_get_tmpdir(void)
>   return tmpdir;
> }
> 
>+#if defined(DBUS_ENABLE_X11_AUTOLAUNCH) || defined(DBUS_ENABLE_LAUNCHD)
> /**
>  * Execute a subprocess, returning up to 1024 bytes of output
>  * into @p result.
>@@ -3408,6 +3409,7 @@ _read_subprocess_line_argv (const char *progpath,
> 
>   return retval;
> }
>+#endif
> 
> /**
>  * Returns the address of a new session bus.
>diff --git a/tools/dbus-launch.c b/tools/dbus-launch.c
>index 2a9dabf..b2ffe41 100644
>--- a/tools/dbus-launch.c
>+++ b/tools/dbus-launch.c
>@@ -1162,7 +1162,6 @@ main (int argc, char **argv)
>       char *end;
>       long wid = 0;
>       long val;
>-      int ret2;
> 
>       verbose ("=== Parent dbus-launch continues\n");
>       
>@@ -1236,6 +1235,8 @@ main (int argc, char **argv)
> #ifdef DBUS_ENABLE_X11_AUTOLAUNCH
>       if (xdisplay != NULL)
>         {
>+          int ret2;
>+
>           verbose("Saving x11 address\n");
>           ret2 = x11_save_address (bus_address, bus_pid, &wid);
>           /* Only get an existing dbus session when autolaunching */
>-- 
>1.7.10.4
>

Actions: View | Splinter Review

Attachments on bug 64362: 79025 | 79045 | 79046

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.