From 1d2478ae4f2da8869eab94ca455a1329230c179e Mon Sep 17 00:00:00 2001 From: Havoc Pennington Date: Sun, 26 Jan 2003 16:13:57 +0000 Subject: add a couple @todo --- dbus/dbus-string.c | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'dbus/dbus-string.c') diff --git a/dbus/dbus-string.c b/dbus/dbus-string.c index e1707bcc..0e9a94d2 100644 --- a/dbus/dbus-string.c +++ b/dbus/dbus-string.c @@ -139,6 +139,17 @@ typedef struct * no maximum. The string starts life with zero length. * The string must eventually be freed with _dbus_string_free(). * + * @todo the max length feature is useless, because it looks + * to the app like out of memory, and the app might try + * to "recover" - but recovery in this case is impossible, + * as we can't ever "get more memory" - so should delete the + * max length feature I think. + * + * @todo we could make this init routine not alloc any memory and + * return void, would simplify a lot of code, however it might + * complexify things elsewhere because _dbus_string_get_data() + * etc. could suddenly fail as they'd need to alloc new memory. + * * @param str memory to hold the string * @param max_length the maximum size of the string * @returns #TRUE on success -- cgit