more minor fixes

 -Erik

1 files changed, 69 insertions, 44 deletions

diff --git a/shell/cmdedit.c b/shell/cmdedit.c
index 8a7a5fb03..b3e7fd58c 100644
--- a/shell/cmdedit.c
+++ b/shell/cmdedit.c
@@ -184,19 +184,45 @@ char** username_completion_matches(char* command, int *num_matches)
         fprintf(stderr, "\nin username_completion_matches\n");
         return (matches);
 }
+
+#include <dirent.h>
 char** find_path_executable_n_cwd_matches(char* command, int *num_matches)
 {
+        char *dirName;
         char **matches = (char **) NULL;
-        matches = malloc(sizeof(char*)*100);
-
-        matches[0] = malloc(sizeof(char)*50);
-        matches[1] = malloc(sizeof(char)*50);
-
-        sprintf(matches[0], "Hello");
-        sprintf(matches[1], "Howdy");
-        *num_matches=2;
+        DIR *dir;
+        struct dirent *next;
+                        
+        matches = malloc( sizeof(char*)*50);
+
+        /* Stick a wildcard onto the command, for later use */
+        strcat( command, "*");
+
+        /* Now wall the current directory */
+        dirName = get_current_dir_name();
+        dir = opendir(dirName);
+        if (!dir) {
+                /* Don't print an error, just shut up and return */
+                *num_matches=0;
+                return (matches);
+        }
+        while ((next = readdir(dir)) != NULL) {
+
+                /* Some quick sanity checks */
+                if ((strcmp(next->d_name, "..") == 0)
+                        || (strcmp(next->d_name, ".") == 0)) {
+                        continue;
+                } 
+                /* See if this matches */
+                if (check_wildcard_match(next->d_name, command) == TRUE) {
+                        /* Cool, found a match.  Add it to the list */
+                        matches[*num_matches] = malloc(strlen(next->d_name)+1);
+                        strcpy( matches[*num_matches], next->d_name);
+                        ++*num_matches;
+                        //matches = realloc( matches, sizeof(char*)*(*num_matches));
+                }
+        }
 
-//      fprintf(stderr, "\nin find_path_executable_n_cwd_matches\n");
         return (matches);
 }
 
@@ -319,11 +345,11 @@ extern int cmdedit_read_input(char* prompt, int inputFd, int outputFd,
                                         /* For now, we will not bother with trying to distinguish
                                          * whether the cursor is in/at a command extression -- we
                                          * will always try all possable matches.  If you don't like
-                                         * that, feel free to fix it.
+                                         * that then feel free to fix it.
                                          */
                                         
                                         /* Make a local copy of the string -- up 
-                                         * to the the position of the cursor */
+                                         * to the position of the cursor */
                                         matchBuf = (char *) calloc(BUFSIZ, sizeof(char));
                                         strncpy(matchBuf, parsenextc, cursor);
                                         tmp=matchBuf;
@@ -346,59 +372,58 @@ extern int cmdedit_read_input(char* prompt, int inputFd, int outputFd,
                                                 matches = (char **) NULL;
                                         }
 
-                                        /* If the word starts in `~', and there is no slash in the word, 
+                                        /* If the word starts with `~' and there is no slash in the word, 
                                          * then try completing this word as a username. */
+
+                                        /* FIXME -- this check is broken! */
                                         if (*tmp == '~' && !strchr(tmp, '/'))
                                                 matches = username_completion_matches(tmp, &num_matches);
 
-                                        /* Try to match any executable in our patch, and everything 
-                                         * in the current working directory that matches.
-                                         */
+                                        /* Try to match any executable in our path and everything 
+                                         * in the current working directory that matches.  */
                                         if (!matches)
                                                 matches = find_path_executable_n_cwd_matches(tmp, &num_matches);
+
+                                        /* Don't leak memory */
+                                        free( matchBuf);
+
+                                        /* Did we find exactly one match? */
+                                        if (matches && num_matches==1) {
+                                                /* write out the matched command */
+                                                strncpy(parsenextc+pos, matches[0]+pos, strlen(matches[0])-pos);
+                                                len=strlen(parsenextc);
+                                                cursor=len;
+                                                xwrite(outputFd, matches[0]+pos, strlen(matches[0])-pos);
+                                                break;
+                                        }
                                 } else {
+                                        /* Ok -- the last char was a TAB.  Since they
+                                         * just hit TAB again, print a list of all the
+                                         * available choices... */
                                         if ( matches && num_matches>0 ) {
                                                 int i, col;
                                                 
-                                                fprintf(stderr, "\nTabbing...\n");
-                                                
-                                                /* Make a list of the matches */
-                                                col += xwrite(outputFd, "\n", 1);
+                                                /* Go to the next line */
+                                                xwrite(outputFd, "\n", 1);
+                                                /* Print the list of matches */
                                                 for (i=0,col=0; i<num_matches; i++) {
-                                                        col += xwrite(outputFd, prompt, strlen(matches[i]));
+                                                        char foo[17];
+                                                        sprintf(foo, "%-14s  ", matches[i]);
+                                                        col += xwrite(outputFd, foo, strlen(foo));
                                                         if (col > 60 && matches[i+1] != NULL) {
                                                                 xwrite(outputFd, "\n", 1);
                                                                 col = 0;
                                                         }
                                                 }
+                                                /* Go to the next line */
                                                 xwrite(outputFd, "\n", 1);
-
-                                                len+=strlen(prompt);
-                                                fprintf(stderr, "len=%d\n", len);
-                                                
-                                                /* Move to the beginning of the line */
-                                                input_home(outputFd, &len);
-
-                                                /* erase everything */
-                                                for (j = 0; j < len; j++)
-                                                        xwrite(outputFd, " ", 1);
-
-                                                /* return to begining of line */
-                                                input_home(outputFd, &cursor);
-
-                                                /* Rewrite the prompt) */
+                                                /* Rewrite the prompt */
                                                 xwrite(outputFd, prompt, strlen(prompt));
-
                                                 /* Rewrite the command */
-                                                len = strlen(parsenextc);
                                                 xwrite(outputFd, parsenextc, len);
-
-                                                /* Move back to where the cursor used to be */
-                                                for (cursor=pos; cursor > 0; cursor--)
+                                                /* Put the cursor back to where it used to be */
+                                                for (cursor=len; cursor > pos; cursor--)
                                                         xwrite(outputFd, "\b", 1);
-                                                cursor = pos;
-
-                                                //fprintf(stderr, "\nprompt='%s'\n", prompt);
                                         }
                                 }
                                 break;
@@ -451,7 +476,6 @@ extern int cmdedit_read_input(char* prompt, int inputFd, int outputFd,
                                                 /* return to begining of line */
                                                 for (; cursor > 0; cursor--)
                                                         xwrite(outputFd, "\b", 1);
-                                                xwrite(outputFd, parsenextc, len);
 
                                                 /* erase old command */
                                                 for (j = 0; j < len; j++)
@@ -462,6 +486,7 @@ extern int cmdedit_read_input(char* prompt, int inputFd, int outputFd,
                                                         xwrite(outputFd, "\b", 1);
 
                                                 memset(parsenextc, 0, BUFSIZ);
+                                                len = strlen(parsenextc);
                                                 /* write new command */
                                                 strcpy(parsenextc, hp->s);
                                                 len = strlen(hp->s);