]> git.ipfire.org Git - thirdparty/dbus.git/commitdiff
sysdeps: remove misleading comments
authorSimon McVittie <simon.mcvittie@collabora.co.uk>
Mon, 26 Sep 2011 11:35:07 +0000 (12:35 +0100)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Wed, 28 Sep 2011 18:01:04 +0000 (19:01 +0100)
The comment claims that _dbus_full_duplex_pipe() is only used for
the debug-pipe server, but in fact the process-spawning code uses it now
(on both Unix and Windows platforms).

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41222

dbus/dbus-sysdeps-unix.c
dbus/dbus-sysdeps-win.c

index 3067be596ba35cc110eb9cbd35e9a1f203d2991f..00283b37c0a0a53d30bad73c7731bd57746c8bf7 100644 (file)
@@ -2930,11 +2930,6 @@ _dbus_print_backtrace (void)
  *
  * Marks both file descriptors as close-on-exec
  *
- * @todo libdbus only uses this for the debug-pipe server, so in
- * principle it could be in dbus-sysdeps-util.c, except that
- * dbus-sysdeps-util.c isn't in libdbus when tests are enabled and the
- * debug-pipe server is used.
- *
  * @param fd1 return location for one end
  * @param fd2 return location for the other end
  * @param blocking #TRUE if pipe should be blocking
index b492b09cf3e8cd11ab0ea8bc8dfb5b79884a8c21..1588174c1ddcbbf6166c713dabdf686bbe9572e1 100644 (file)
@@ -799,11 +799,6 @@ failed:
  * Creates a full-duplex pipe (as in socketpair()).
  * Sets both ends of the pipe nonblocking.
  *
- * @todo libdbus only uses this for the debug-pipe server, so in
- * principle it could be in dbus-sysdeps-util.c, except that
- * dbus-sysdeps-util.c isn't in libdbus when tests are enabled and the
- * debug-pipe server is used.
- * 
  * @param fd1 return location for one end
  * @param fd2 return location for the other end
  * @param blocking #TRUE if pipe should be blocking