# HG changeset patch
# User Luke Schierer <lschiere@pidgin.im>
# Date 1112884878 0
# Node ID 19de830330cfdd7e28e1faa623ed4e7e7952aa81
# Parent  c932d96a7c69c987308472ce0c0107af6c8be7f6
[gaim-migrate @ 12429]
" You can't pass NULL as errormsg as the documentation
implies. If you do, it segfaults. Perhaps that should be
allowable, but as the code stands, it's not. The
documentation should be updated in HEAD and oldstatus
to reflect this. Attached is a patch to that effect." --rlaager

committer: Tailor Script <tailor@pidgin.im>

diff -r c932d96a7c69 -r 19de830330cf src/cmds.h
--- a/src/cmds.h	Thu Apr 07 14:35:25 2005 +0000
+++ b/src/cmds.h	Thu Apr 07 14:41:18 2005 +0000
@@ -150,9 +150,8 @@
  * @param markup This is the same as cmd, but is the formatted version. It should be in
  *               HTML, with < > and &, at least, escaped to html entities, and should
  *               include both the default formatting and any extra manual formatting.
- * @param errormsg If the command failed and errormsg is not NULL, it is filled in with
- *                 the appropriate error message. It should be freed by the caller with
- *                 g_free().
+ * @param errormsg If the command failed errormsg is filled in with the appropriate error
+ *                 message. It must be freed by the caller with g_free().
  * @return A GaimCmdStatus indicated if the command succeeded or failed.
  */
 GaimCmdStatus gaim_cmd_do_command(GaimConversation *conv, const gchar *cmdline,