remove erroneous return from setup_user_args

This fixes setup_user_args to drop a useless and confusing "return".

	* cli/cli-script.c (setup_user_args): Don't return after error.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2345ff8..eaa47a0 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,9 @@
 2013-05-30  Tom Tromey  <tromey@redhat.com>
 
+       * cli/cli-script.c (setup_user_args): Don't return after error.
+
+2013-05-30  Tom Tromey  <tromey@redhat.com>
+
        * somread.c (som_symtab_read): Call do_cleanups.
 
 2013-05-30  Tom Tromey  <tromey@redhat.com>

diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c
index 3b24799..43fd479 100644 (file)
--- a/gdb/cli/cli-script.c
+++ b/gdb/cli/cli-script.c
@@ -689,11 +689,8 @@ setup_user_args (char *p)
       int bsquote = 0;
 
       if (arg_count >= MAXUSERARGS)
-       {
-         error (_("user defined function may only have %d arguments."),
-                MAXUSERARGS);
-         return old_chain;
-       }
+       error (_("user defined function may only have %d arguments."),
+              MAXUSERARGS);
 
       /* Strip whitespace.  */
       while (*p == ' ' || *p == '\t')