From b81b921f794dc3ce6112e784f39511004c7cb542 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Thu, 30 May 2013 16:31:33 +0000 Subject: [PATCH] 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. --- gdb/ChangeLog | 4 ++++ gdb/cli/cli-script.c | 7 ++----- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2345ff8003a..eaa47a08910 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,7 @@ +2013-05-30 Tom Tromey + + * cli/cli-script.c (setup_user_args): Don't return after error. + 2013-05-30 Tom Tromey * somread.c (som_symtab_read): Call do_cleanups. diff --git a/gdb/cli/cli-script.c b/gdb/cli/cli-script.c index 3b247997552..43fd4796d6f 100644 --- 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')