[asterisk-commits] trunk r16129 - /trunk/pbx.c

[asterisk-commits at lists.digium.com]

Author: rizzo
Date: Wed Mar 29 10:51:51 2006
New Revision: 16129

URL: http://svn.digium.com/view/asterisk?rev=16129&view=rev
Log:
another batch of cli simplifications.


Modified:
    trunk/pbx.c

Modified: trunk/pbx.c
URL: http://svn.digium.com/view/asterisk/trunk/pbx.c?rev=16129&r1=16128&r2=16129&view=diff
==============================================================================
--- trunk/pbx.c (original)
+++ trunk/pbx.c Wed Mar 29 10:51:51 2006
@@ -3092,33 +3092,9 @@
 
 static char *complete_show_applications(const char *line, const char *word, int pos, int state)
 {
-	int wordlen = strlen(word);
-
-	if (pos == 2) {
-		if (ast_strlen_zero(word)) {
-			switch (state) {
-			case 0:
-				return strdup("like");
-			case 1:
-				return strdup("describing");
-			default:
-				return NULL;
-			}
-		} else if (! strncasecmp(word, "like", wordlen)) {
-			if (state == 0) {
-				return strdup("like");
-			} else {
-				return NULL;
-			}
-		} else if (! strncasecmp(word, "describing", wordlen)) {
-			if (state == 0) {
-				return strdup("describing");
-			} else {
-				return NULL;
-			}
-		}
-	}
-	return NULL;
+	static char* choices[] = { "like", "describing", NULL };
+
+	return (pos != 2) ? NULL : ast_cli_complete(word, choices, state);
 }
 
 /*
@@ -3144,16 +3120,11 @@
 
 	wordlen = strlen(word);
 
-	/* ... walk through all contexts ... */
+	/* walk through all contexts and return the n-th match */
 	while ( (c = ast_walk_contexts(c)) ) {
-		/* ... word matches context name? yes? ... */
-		if (!strncasecmp(word, ast_get_context_name(c), wordlen)) {
-			/* ... for serve? ... */
-			if (++which > state) {
-				/* ... yes, serve this context name ... */
-				ret = strdup(ast_get_context_name(c));
-				break;
-			}
+		if (!strncasecmp(word, ast_get_context_name(c), wordlen) && ++which > state) {
+			ret = ast_strdup(ast_get_context_name(c));
+			break;
 		}
 	}