changeset 15745:8e24d43f529a

Comments
author Mark Doliner <mark@kingant.net>
date Thu, 01 Mar 2007 08:26:07 +0000
parents 12778e2f7282
children d3d9f28fb6d0
files libpurple/server.c pidgin/gtkmain.c
diffstat 2 files changed, 7 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/server.c	Thu Mar 01 08:00:19 2007 +0000
+++ b/libpurple/server.c	Thu Mar 01 08:26:07 2007 +0000
@@ -448,6 +448,7 @@
 	 * been given us by the prpls. So we create temp holders and pass
 	 * those instead. It's basically just to avoid segfaults.
 	 */
+	/* TODO: MAX(message, BUF_LONG) is pretty ugly. */
 	buffy = g_malloc(MAX(strlen(msg) + 1, BUF_LONG));
 	strcpy(buffy, msg);
 	angel = g_strdup(who);
@@ -771,7 +772,7 @@
 	 * data is binary, plugins don't see it. Bitch all you want; i really
 	 * don't want you to be dealing with it.
 	 */
-
+	/* TODO: MAX(message, BUF_LONG) is pretty ugly. */
 	buffy = g_malloc(MAX(strlen(message) + 1, BUF_LONG));
 	strcpy(buffy, message);
 	angel = g_strdup(who);
--- a/pidgin/gtkmain.c	Thu Mar 01 08:00:19 2007 +0000
+++ b/pidgin/gtkmain.c	Thu Mar 01 08:26:07 2007 +0000
@@ -171,6 +171,11 @@
  * function that can be called by applications to disable forking
  * during initialization.  But it's not in 0.10.0, so we shouldn't
  * use it.
+ *
+ * All of this child process reaping stuff is currently only used for
+ * processes that were forked to play sounds.  It's not needed for
+ * forked DNS child, which have their own waitpid() call.  It might
+ * be wise to move this code into gtksound.c.
  */
 static void
 clean_pid()