(act_getline): Change return type to

unsigned. Return actual number of input lines processed or 0 on
failure (either I/O, or memory allocation or EOF).
(action_echo): Rewritten to use shell-like syntax
(run_user_action): Rewritten main loop to make it cleaner. In
particular, expand agruments after word splitting. Use new echo
semantics.

@@ -39,12 +39,13 @@
                  number of characters and line in the expansion.
                  number of characters and line in the expansion.
 		 When omitted, they default to 400, 5. */
 		 When omitted, they default to 400, 5. */
 
 
-static int
+static unsigned
 act_getline (FILE *fp, char **sptr, size_t *size)
 act_getline (FILE *fp, char **sptr, size_t *size)
 {
 {
   char buf[256];
   char buf[256];
   int cont = 1;
   int cont = 1;
   size_t used = 0;
   size_t used = 0;
+  unsigned lines = 0;
   
   
   while (cont && fgets (buf, sizeof buf, fp))
   while (cont && fgets (buf, sizeof buf, fp))
     {
     {
@@ -56,6 +57,7 @@ act_getline (FILE *fp, char **sptr, size_t *size)
 	    {
 	    {
 	      buf[--len] = 0;
 	      buf[--len] = 0;
 	      cont = 1;
 	      cont = 1;
+	      lines++;
 	    }
 	    }
 	  else
 	  else
 	    cont = 0;
 	    cont = 0;
@@ -67,7 +69,7 @@ act_getline (FILE *fp, char **sptr, size_t *size)
 	{
 	{
 	  *sptr = realloc (*sptr, len + used + 1);
 	  *sptr = realloc (*sptr, len + used + 1);
 	  if (!*sptr)
 	  if (!*sptr)
-	    return -1;
+	    return 0;
 	  *size = len + used + 1;
 	  *size = len + used + 1;
 	}
 	}
       memcpy (*sptr + used, buf, len);
       memcpy (*sptr + used, buf, len);
@@ -77,7 +79,9 @@ act_getline (FILE *fp, char **sptr, size_t *size)
   if (*sptr)
   if (*sptr)
     (*sptr)[used] = 0;
     (*sptr)[used] = 0;
 
 
-  return used;
+  if (used && !feof (fp))
+    lines++;
+  return lines;
 }
 }
 
 
 static int
 static int
@@ -237,7 +241,7 @@ action_beep (FILE *tty)
 }
 }
 
 
 static void
 static void
-action_echo (FILE *tty, const char *cr, char *str)
+echo_string (FILE *tty, const char *cr, char *str)
 {
 {
   if (!str)
   if (!str)
     return;
     return;
@@ -255,6 +259,32 @@ action_echo (FILE *tty, const char *cr, char *str)
 }
 }
 
 
 static void
 static void
+action_echo (FILE *tty, const char *cr, int omit_newline,
+	     int argc, char **argv)
+{
+  int i;
+
+  if (omit_newline)
+    {
+      argc--;
+      argv++;
+    }
+  
+  for (i = 0;;)
+    {
+      echo_string (tty, cr, argv[i]);
+      if (++i < argc)
+	echo_string (tty, cr, " ");
+      else
+	{
+	  if (!omit_newline)
+	    echo_string (tty, cr, "\n");
+	  break;
+	}
+    }
+}
+
+static void
 action_exec (FILE *tty, int line, int argc, char **argv)
 action_exec (FILE *tty, int line, int argc, char **argv)
 {
 {
   pid_t pid;
   pid_t pid;
@@ -341,40 +371,46 @@ run_user_action (FILE *tty, const char *cr, mu_message_t msg)
   fp = open_rc (BIFF_RC, tty);
   fp = open_rc (BIFF_RC, tty);
   if (fp)
   if (fp)
     {
     {
-      int line = 0;
+      unsigned line = 1, n;
 
 
-      while (act_getline (fp, &stmt, &size))
+      while ((n = act_getline (fp, &stmt, &size)))
 	{
 	{
-	  char *str;
 	  int argc;
 	  int argc;
 	  char **argv;
 	  char **argv;
 
 
-	  line++;
-	  str = expand_line (stmt, msg);
-	  if (!str)
-	    continue;
-	  if (mu_argcv_get (str, "", NULL, &argc, &argv)
-	      || argc == 0
-	      || argv[0][0] == '#')
+	  if (mu_argcv_get (stmt, "", NULL, &argc, &argv) == 0
+	      && argc
+	      && argv[0][0] != '#')
 	    {
 	    {
-	      free (str);
-	      mu_argcv_free (argc, argv);
-	      continue;
-	    }
-
 	      if (strcmp (argv[0], "beep") == 0)
 	      if (strcmp (argv[0], "beep") == 0)
 		{
 		{
+		  /* FIXME: excess arguments are ignored */
 		  action_beep (tty);
 		  action_beep (tty);
 		  nact++;
 		  nact++;
 		}
 		}
-	  else if (strcmp (argv[0], "echo") == 0)
+	      else
+		{
+		  /* Rest of actions require keyword expansion */
+		  int i;
+		  int n_option = argc > 1 && strcmp (argv[1], "-n") == 0;
+		  
+		  for (i = 1; i < argc; i++)
+		    {
+		      char *oldarg = argv[i];
+		      argv[i] = expand_line (argv[i], msg);
+		      free (oldarg);
+		      if (!argv[i])
+			break;
+		    }
+		  
+		  if (strcmp (argv[0], "echo") == 0)
 		    {
 		    {
-	      action_echo (tty, cr, argv[1]);
+		      action_echo (tty, cr, n_option, argc - 1, argv + 1);
 		      nact++;
 		      nact++;
 		    }
 		    }
 		  else if (strcmp (argv[0], "exec") == 0)
 		  else if (strcmp (argv[0], "exec") == 0)
 		    {
 		    {
-	      action_exec (tty, line, argc-1, argv+1);
+		      action_exec (tty, line, argc - 1, argv + 1);
 		      nact++;
 		      nact++;
 		    }
 		    }
 		  else
 		  else
@@ -386,9 +422,13 @@ run_user_action (FILE *tty, const char *cr, mu_message_t msg)
 		      break;
 		      break;
 		    }
 		    }
 		} 
 		} 
+	    }
+	  mu_argcv_free (argc, argv);
+	  line += n;
+	}
       fclose (fp);
       fclose (fp);
     }
     }
 
 
   if (nact == 0)
   if (nact == 0)
-    action_echo (tty, cr, expand_line (default_action, msg));
+    echo_string (tty, cr, expand_line (default_action, msg));
 }
 }