* doc/TODO:

* dbus/dbus-bus.c:
* dbus-errors.c:
* dbus/dbus-marshal-validate.c:
  Removed TODO items which were fixed or are no longer relevent

diff --git a/ChangeLog b/ChangeLog
index 4122b1641dae7d2f842e9fdad49cf229755159d8..027dbb30eea41b909b677ed85ae13b0bc0fe5afd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2006-07-17  John (J5) Palmieri  <johnp@redhat.com>
+
+       * doc/TODO:
+       * dbus/dbus-bus.c:
+       * dbus-errors.c:
+       * dbus/dbus-marshal-validate.c:
+         Removed TODO items which were fixed or are no longer relevent
+
 2006-07-17  John (J5) Palmieri  <johnp@redhat.com>
 
        * dbus-qt4-1.pc.in, dbus-sharp.pc.in:

diff --git a/dbus/dbus-bus.c b/dbus/dbus-bus.c
index 273fe5fe9c975dae9fc3bfa8d94d321049732920..2c14ea81ee49148bdf1cf1dc0456992545a4848e 100644 (file)
--- a/dbus/dbus-bus.c
+++ b/dbus/dbus-bus.c
@@ -430,8 +430,6 @@ internal_bus_get (DBusBusType  type,
  * connection to the bus already exists, then that connection is
  * returned.  Caller owns a reference to the bus.
  *
- * @todo alex thinks we should nullify the connection when we get a disconnect-message.
- *
  * @param type bus type
  * @param error address where an error can be returned.
  * @returns a DBusConnection with new ref
@@ -722,10 +720,6 @@ dbus_bus_get_unix_user (DBusConnection *connection,
  * (last-started-wins) or be queued up (first-started-wins) according
  * to whether #DBUS_NAME_FLAG_ALLOW_REPLACEMENT was given.
  * 
- * @todo this all seems sort of broken. Shouldn't the flags be a property
- * of the name, not the app requesting the name? What are the use-cases
- * other than the "text editor" thing and how are we supporting them?
- * 
  * @param connection the connection
  * @param name the name to request
  * @param flags flags

diff --git a/dbus/dbus-errors.c b/dbus/dbus-errors.c
index d37c068868fd6a49bc72af8f5688c80a7e2e28d1..a53041e71d10edd4a926cdbda2390b94c1664be1 100644 (file)
--- a/dbus/dbus-errors.c
+++ b/dbus/dbus-errors.c
@@ -194,8 +194,6 @@ dbus_error_free (DBusError *error)
  * message will be deduced from the name. If the error name is unknown
  * to D-BUS the default message will be totally useless, though.
  *
- * @todo should be called dbus_error_set_const() 
- * 
  * @param error the error.
  * @param name the error name (not copied!!!)
  * @param message the error message (not copied!!!)
@@ -306,8 +304,6 @@ dbus_error_is_set (const DBusError *error)
  * If no memory can be allocated for the error message, 
  * an out-of-memory error message will be set instead.
  *
- * @todo should be called dbus_error_set()
- *
  * @param error the error.
  * @param name the error name
  * @param format printf-style format string.

diff --git a/dbus/dbus-marshal-validate.c b/dbus/dbus-marshal-validate.c
index e9aa26a8bd2db844247116290782edd86bc6f8ea..1113106034a6272e2ac2f2245a2ebdf585d53039 100644 (file)
--- a/dbus/dbus-marshal-validate.c
+++ b/dbus/dbus-marshal-validate.c
@@ -41,10 +41,6 @@
  * The range passed in should NOT include the terminating
  * nul/DBUS_TYPE_INVALID.
  *
- * @todo verify that dict entries have exactly two fields
- * 
- * @todo require that dict entries are in an array
- *
  * @param type_str the string
  * @param type_pos where the typecodes start
  * @param len length of typecodes

diff --git a/doc/TODO b/doc/TODO
index 6ae67d1cbbebb0e39823e97f7b57b8e60eacb150..227c06c51a4f130437fbe8bf418e6e3715cff4a8 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -1,15 +1,3 @@
-Important for 0.90 freeze
-===
-
-- Audit @todo and FIXME for security issues that require API/ABI changes
-
-- dbus-pending-call.c has some API and thread safety issues to review.
-   DBusPendingCall is used from multiple threads with no locks.
-   Either DBusConnection's lock has to protect all associated pending
-   call (means pending->connection can't ever be set to null) or
-   or DBusPendingCall needs its own lock
-   http://lists.freedesktop.org/archives/dbus/2006-June/004945.html
-
 Important for 1.0
 ===