Re: [RP] Small bug

[Rupert Levene]

On Sun, Jul 27, 2003 at 02:13:21PM +0200, Florian Cramer wrote:
> Noted that tab expansion doesn't work for the "newwm" and "tmpwm"
> commands. 

Here ya go... this adds bash-style completions (which doesn't
distinguish between WMs and non-WMs).

Index: actions.c
===================================================================
RCS file: /cvsroot/ratpoison/ratpoison/src/actions.c,v
retrieving revision 1.177
diff -u -r1.177 actions.c
--- actions.c   17 Jul 2003 05:41:41 -0000      1.177
+++ actions.c   27 Jul 2003 13:05:49 -0000
@@ -1190,7 +1190,7 @@
   char *prog;
 
   if (data == NULL)
-    prog = get_input (MESSAGE_PROMPT_SWITCH_WM, trivial_completions);
+    prog = get_input (MESSAGE_PROMPT_SWITCH_WM, exec_completions);
   else
     prog = xstrdup (data);
 
@@ -3098,12 +3098,16 @@
   int status;
   int pid;
   int i;
+  char *prog;
 
   if (data == NULL)
-    {
-      message (" tmpwm: one argument required ");
-      return NULL;
-    }
+    prog = get_input (MESSAGE_PROMPT_SWITCH_WM, exec_completions);
+  else
+    prog = xstrdup (data);
+
+
+  if (prog == NULL)
+    return NULL;
 
   /* Release event selection on the root windows, so the new WM can
      have it. */
@@ -3123,7 +3127,7 @@
   XSync (dpy, False);
 
   /* Launch the new WM and wait for it to terminate. */
-  pid = spawn (data);
+  pid = spawn (prog);
   do
     {
       child = waitpid (pid, &status, 0);