From 6fc53eb13073722f6f44d0af869c7ff1eb3f206e Mon Sep 17 00:00:00 2001 From: Havoc Pennington Date: Fri, 31 Jan 2003 00:56:01 +0000 Subject: [PATCH] 2003-01-30 Havoc Pennington * dbus/dbus-server.c (dbus_server_listen): @todo about how we need a better way to report errors here. e.g. "unix address lacks path" or something. also "no such file" when the path doesn't exist, etc. * dbus/dbus-address.c (dbus_address_entries_free): add @todo about leaking list nodes (dbus_parse_address): add @todo about documenting address format, and allowing , and ; to be escaped --- ChangeLog | 12 ++++++++++++ dbus/dbus-address.c | 7 +++++++ dbus/dbus-server.c | 8 ++++++-- dbus/dbus-transport.c | 3 ++- 4 files changed, 27 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 32f47bf..d12e14d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,15 @@ +2003-01-30 Havoc Pennington + + * dbus/dbus-server.c (dbus_server_listen): @todo about how we need + a better way to report errors here. e.g. "unix address lacks + path" or something. also "no such file" when the path doesn't + exist, etc. + + * dbus/dbus-address.c (dbus_address_entries_free): add @todo about + leaking list nodes + (dbus_parse_address): add @todo about documenting address format, + and allowing , and ; to be escaped + 2003-01-30 Anders Carlsson * dbus/Makefile.am: diff --git a/dbus/dbus-address.c b/dbus/dbus-address.c index 5b65f2f..4c584bc 100644 --- a/dbus/dbus-address.c +++ b/dbus/dbus-address.c @@ -73,6 +73,8 @@ dbus_address_entry_free (DBusAddressEntry *entry) /** * Frees a #NULL-terminated array of address entries. * + * @todo dbus_address_entry_free() seems to leak list nodes + * * @param entries the array. */ void @@ -157,6 +159,11 @@ dbus_address_entry_get_value (DBusAddressEntry *entry, * * method:key=value,key=value;method:key=value * + * @todo document address format in the specification + * + * @todo need to be able to escape ';' and ',' in the + * key values, and the parsing needs to handle that. + * * @param address the address. * @param entry return location to an array of entries. * @param array_len return location for array length. diff --git a/dbus/dbus-server.c b/dbus/dbus-server.c index 5177431..028144f 100644 --- a/dbus/dbus-server.c +++ b/dbus/dbus-server.c @@ -191,6 +191,9 @@ _dbus_server_remove_timeout (DBusServer *server, * dbus_server_set_watch_functions() should be called * immediately to render the server fully functional. * + * @todo error messages on bad address could really be better. + * DBusResultCode is a bit limiting here. + * * @param address the address of this server. * @param result location to store rationale for failure. * @returns a new DBusServer, or #NULL on failure. @@ -235,9 +238,10 @@ dbus_server_listen (const char *address, server = _dbus_server_debug_new (name, result); if (server) - break; + break; } - else goto bad_address; + else + goto bad_address; } dbus_address_entries_free (entries); diff --git a/dbus/dbus-transport.c b/dbus/dbus-transport.c index 8c92398..88a78db 100644 --- a/dbus/dbus-transport.c +++ b/dbus/dbus-transport.c @@ -184,7 +184,8 @@ _dbus_transport_finalize_base (DBusTransport *transport) * Opens a new transport for the given address. (This opens a * client-side-of-the-connection transport.) * - * @todo right now the address is just a Unix domain socket path. + * @todo error messages on bad address could really be better. + * DBusResultCode is a bit limiting here. * * @param address the address. * @param result location to store reason for failure. -- 2.7.4