[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[irssi-xmpp] "in-code documentation" patch
Hello list,
this one had me a bit confused at first, but trying to tabcomplete
turned out that there actually is a way to set the status to "online"
(not "available") and still have a status message. I think it is more
accurate if this is mentioned in the specific comment. I put it in first
in the list, because "online" is the most "active" status. Maybe the
others need reordering, maybe nobody cares, but I wanted to point it out
still.
I think real documentation would be really useful. I know it's annoying
to write, but I for one do really appreciate it. It doesn't need to be
long, but there is almost none except the code... This is the best XMPP
client I've come across so far, so thanks, and keep up the good work,
but with more documentation, please ;)
Also, the ML seems relatively dead, except for occasional spam and some
messages without replies. This is a bit sad. I come to wonder how many
people other than me use irssi-xmpp as their main XMPP client. Do we
have any statistics?
Please let me know if I can be of any help. I would love to participate.
Best regards,
Moritz
Index: src/core/xmpp-commands.c
===================================================================
RCS file: /cvs/irssi-xmpp/irssi-xmpp/src/core/xmpp-commands.c,v
retrieving revision 1.61
diff -u -r1.61 xmpp-commands.c
--- src/core/xmpp-commands.c 14 Jul 2010 16:07:13 -0000 1.61
+++ src/core/xmpp-commands.c 26 Aug 2011 20:14:43 -0000
@@ -165,7 +165,7 @@
g_strfreev(tmp);
}
-/* SYNTAX: AWAY [-one | -all] [away|dnd|xa|chat] [<reason>] */
+/* SYNTAX: AWAY [-one | -all] [online|away|dnd|xa|chat] [<reason>] */
static void
cmd_away(const char *data, XMPP_SERVER_REC *server)
{