From: Havoc Pennington <hp@redhat.com>
Date: Thu, 21 Jun 2007 21:28:59 +0000 (+0000)
Subject: 2007-06-21  Havoc Pennington  <hp@redhat.com>
X-Git-Tag: dbus-1.1.2~46
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ee721d13c455b5780af5989662ff7fca68374327;p=platform%2Fupstream%2Fdbus.git

2007-06-21  Havoc Pennington  <hp@redhat.com>

	* dbus/dbus-watch.c (dbus_watch_get_fd): 1) its behavior should
	not be the same as before, the ABI has never been declared stable
	on Windows and 2) do not commit to cross-platform files without
	posting the exact patch to the mailing list

	* dbus/dbus-sysdeps-util.c (_dbus_sysdeps_test): as I have pointed
	out before, either 0xff needs to work on both platforms, or none
	of the dbus code can rely on it working. That means the options
	are 1) audit the code for anywhere that relies on 0xff working, if
	none found ideally add a test that it *doesn't* work and make unix
	reject it explicitly, but in any case this test would go away or
	2) make it work on Windows also, then we don't have to figure out
	whether we rely on it.

	And in either case, post the exact patch to the mailing list and
	don't just commit.
---

diff --git a/ChangeLog b/ChangeLog
index 9efb0ac..0fc6e9e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,22 @@
+2007-06-21  Havoc Pennington  <hp@redhat.com>
+
+	* dbus/dbus-watch.c (dbus_watch_get_fd): 1) its behavior should
+	not be the same as before, the ABI has never been declared stable
+	on Windows and 2) do not commit to cross-platform files without
+	posting the exact patch to the mailing list
+	
+	* dbus/dbus-sysdeps-util.c (_dbus_sysdeps_test): as I have pointed
+	out before, either 0xff needs to work on both platforms, or none
+	of the dbus code can rely on it working. That means the options
+	are 1) audit the code for anywhere that relies on 0xff working, if
+	none found ideally add a test that it *doesn't* work and make unix
+	reject it explicitly, but in any case this test would go away or
+	2) make it work on Windows also, then we don't have to figure out 
+	whether we rely on it.
+
+	And in either case, post the exact patch to the mailing list and
+	don't just commit.
+
 2007-06-21 Ralf Habacker  <ralf.habacker@freenet.de>
 
 	* cmake/CMakeLists.txt: added VERSION_PATCH to be able to 
diff --git a/dbus/dbus-sysdeps-util.c b/dbus/dbus-sysdeps-util.c
index c5aecb9..1f6ceb9 100644
--- a/dbus/dbus-sysdeps-util.c
+++ b/dbus/dbus-sysdeps-util.c
@@ -141,7 +141,6 @@ _dbus_sysdeps_test (void)
       exit (1);
     }
 
-#ifndef DBUS_WIN
   _dbus_string_init_const (&str, "0xff");
   if (!_dbus_string_parse_double (&str,
 				  0, &val, &pos))
@@ -159,7 +158,7 @@ _dbus_sysdeps_test (void)
       _dbus_warn ("_dbus_string_parse_double of \"0xff\" returned wrong position %d", pos);
       exit (1);
     }
-#endif
+
 #ifdef DBUS_WIN
   check_path_absolute ("c:/", TRUE);
   check_path_absolute ("c:/foo", TRUE);
diff --git a/dbus/dbus-watch.c b/dbus/dbus-watch.c
index 634ab24..eed9a10 100644
--- a/dbus/dbus-watch.c
+++ b/dbus/dbus-watch.c
@@ -489,11 +489,7 @@ _dbus_watch_set_handler (DBusWatch        *watch,
 int
 dbus_watch_get_fd (DBusWatch *watch)
 {
-#ifdef DBUS_WIN_FIXME
-  return watch->fd;
-#else
   return dbus_watch_get_unix_fd(watch);
-#endif
 }
 
 /**