summaryrefslogtreecommitdiffstats
path: root/dbus
diff options
context:
space:
mode:
authorHavoc Pennington <hp@redhat.com>2003-01-26 16:13:57 +0000
committerHavoc Pennington <hp@redhat.com>2003-01-26 16:13:57 +0000
commit1d2478ae4f2da8869eab94ca455a1329230c179e (patch)
treef20bab58fa3d4d8c0230a7a164ac05745742f2d2 /dbus
parent9a0b07f225be10c545a8b1de8555bca2dc3a9cab (diff)
add a couple @todo
Diffstat (limited to 'dbus')
-rw-r--r--dbus/dbus-string.c11
1 files changed, 11 insertions, 0 deletions
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