From b459906a7a6519c034f5882c472f0c945371dfc6 Mon Sep 17 00:00:00 2001 From: "John (J5) Palmieri" Date: Mon, 17 Jul 2006 17:44:07 +0000 Subject: * 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 --- ChangeLog | 8 ++++++++ dbus/dbus-bus.c | 6 ------ dbus/dbus-errors.c | 4 ---- dbus/dbus-marshal-validate.c | 4 ---- doc/TODO | 12 ------------ 5 files changed, 8 insertions(+), 26 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4122b164..027dbb30 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2006-07-17 John (J5) Palmieri + + * 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 * dbus-qt4-1.pc.in, dbus-sharp.pc.in: diff --git a/dbus/dbus-bus.c b/dbus/dbus-bus.c index 273fe5fe..2c14ea81 100644 --- 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 d37c0688..a53041e7 100644 --- 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 e9aa26a8..11131060 100644 --- 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 6ae67d1c..227c06c5 100644 --- 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 === -- cgit