X-Git-Url: https://git.cworth.org/git?p=ttt;a=blobdiff_plain;f=src%2Fttt-client.c;h=1dd755f83651e3d7c6f80f176e0befa05f8af8dc;hp=81f6e13e14268227e23a640f8288d933e7e66e15;hb=f2187ebd49f78b84bd2dca6172abc81e54dda199;hpb=b18c78a24034922e7b9152a03683cf930d703dd0;ds=sidebyside

diff --git a/src/ttt-client.c b/src/ttt-client.c
index 81f6e13..1dd755f 100644
--- a/src/ttt-client.c
+++ b/src/ttt-client.c
@@ -84,15 +84,28 @@ typedef struct _ttt_command_description {
     int                args_min;
     int                args_max;
     ttt_command_func_t execute;
+    const char         *usage;
+    const char         *description;
 } ttt_command_description_t;
 
 ttt_command_description_t command_descriptions[] = {
-    {"HELO",    1, 1, _ttt_client_execute_helo   },
-    {"WHO",     0, 0, _ttt_client_execute_who    },
-    {"MESSAGE", 1, 1, _ttt_client_execute_message},
-    {"HELP",    0, 1, _ttt_client_execute_help   },
-    {"VERSION", 1, 1, _ttt_client_execute_version},
-    {"QUIT",    0, 0, _ttt_client_execute_quit   }
+    {"HELO",    1, 1, _ttt_client_execute_helo,
+     "HELO <username>         ", "Register."},
+
+    {"HELP",    0, 1, _ttt_client_execute_help,
+     "HELP <command>          ", "Display help for a command."},
+
+    {"MESSAGE", 1, 1, _ttt_client_execute_message,
+     "MESSAGE <message>       ", "Send a message to everyone."},
+
+    {"QUIT",    0, 0, _ttt_client_execute_quit,
+     "QUIT                    ", "Quit session."},
+
+    {"VERSION", 1, 1, _ttt_client_execute_version,
+     "VERSION <client-version>", "Negotiate protocol version."},
+
+    {"WHO",     0, 0, _ttt_client_execute_who,
+     "WHO                     ", "List registered users."}
 };
 
 #define ARRAY_SIZE(arr) (sizeof(arr)/sizeof(arr[0]))
@@ -164,10 +177,10 @@ _ttt_client_execute_message (ttt_client_t  *client,
     if (!client->registered)
 	return TTT_ERROR_NO_NAME_SET;
 
-    xasprintf(&response, "MESSAGE\r\n");
+    xasprintf (&response, "MESSAGE\r\n");
     ttt_client_send (client, response);
 
-    xasprintf(&notice, "NOTICE MESSAGE %s \"%s\"\r\n",
+    xasprintf (&notice, "NOTICE MESSAGE %s \"%s\"\r\n",
 	      client->name,
 	      args[0]);
     ttt_server_broadcast (client->server, notice);
@@ -184,10 +197,47 @@ _ttt_client_execute_help (ttt_client_t  *client,
 			  int	   num_args)
 {
     char *response;
+    char *command;
+    ttt_command_description_t *desc;
+    int i;
+    ttt_bool_t is_command = FALSE;
+    
+    if (num_args == 0) {
+	xasprintf (&response, "HELP \"\r\n"
+		   "Available Commands:\r\n");
+	for (i = 0; i < ARRAY_SIZE(command_descriptions); i++) {
+	    desc = &command_descriptions[i];
+	    xasprintf (&response, "%s\r\n  %s - %s\r\n",
+		       response,
+		       desc->usage,
+		       desc->description);
+	}
+	xasprintf (&response, "%s\"\r\n", response);
+    } else {
+	command = args[0];
+	for (i = 0; i < strlen (command); i++)
+	    command[i] = toupper (command[i]);
+	for (i = 0; i < ARRAY_SIZE(command_descriptions); i++) {
+	    desc = &command_descriptions[i];
+	    if (strcmp (desc->command, command) == 0) {
+		is_command = TRUE;
+		xasprintf (&response, "HELP %s \"\r\n"
+			   "%s\r\n"
+			   "\r\n"
+			   "Usage:\r\n"
+			   "  %s\r\n"
+			   "\"\r\n",
+			   desc->command,
+			   desc->description,
+			   desc->usage);
+		/* XXX: Add detailed help. */
+	    }
+	}
+    }
+    
+    if ((num_args == 1) && (!is_command))
+	return TTT_ERROR_SYNTAX;
 
-    /* XXX: NYI */
-
-    xasprintf(&response, "HELP - NYI\r\n");
     ttt_client_send (client, response);
 
     free (response);
@@ -347,15 +397,17 @@ _execute_request (ttt_client_t *client)
     int num_args = client->num_request_strings-1;
     ttt_command_description_t *desc;
 
-    for (i=0; i < strlen (command); i++)
+    for (i = 0; i < strlen (command); i++)
 	command[i] = toupper (command[i]);
 
-    for (i=0; i < ARRAY_SIZE(command_descriptions); i++) {
+    for (i = 0; i < ARRAY_SIZE(command_descriptions); i++) {
 	desc = &command_descriptions[i];
-	if (strcmp(command, desc->command) == 0) {
+	if (strcmp (command, desc->command) == 0) {
 	    if ((num_args < desc->args_min) || (num_args > desc->args_max))
 		return TTT_ERROR_SYNTAX;
-	    return (desc->execute) (client, &client->request_strings[1], num_args);
+	    return (desc->execute) (client,
+				    &client->request_strings[1],
+				    num_args);
 	}
     }