diff options
Diffstat (limited to 'dbus')
-rw-r--r-- | dbus/dbus-address.c | 7 | ||||
-rw-r--r-- | dbus/dbus-server.c | 8 | ||||
-rw-r--r-- | dbus/dbus-transport.c | 3 |
3 files changed, 15 insertions, 3 deletions
diff --git a/dbus/dbus-address.c b/dbus/dbus-address.c index 5b65f2fd..4c584bc0 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 51774318..028144f5 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 8c923982..88a78dbf 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. |