From d7ada2457e35d92edfe6bc9946f9c91bd6aeac9b Mon Sep 17 00:00:00 2001
From: zwelch <zwelch@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Date: Thu, 11 Jun 2009 11:42:40 +0000
Subject: [PATCH] Simplify and fix handle_reset_command: - Return syntax error
 if more than one argument is given. - Move variables to location of first
 use.

git-svn-id: svn://svn.berlios.de/openocd/trunk@2192 b42882b7-edfa-0310-969c-e2dbd0fdcd60
---
 src/target/target.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/target/target.c b/src/target/target.c
index 260f9cec7..b207ac95a 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -1912,11 +1912,13 @@ static int handle_soft_reset_halt_command(struct command_context_s *cmd_ctx, cha
 
 static int handle_reset_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
 {
-	const Jim_Nvp *n;
-	enum target_reset_mode reset_mode = RESET_RUN;
+	if (argc > 1)
+		return ERROR_COMMAND_SYNTAX_ERROR;
 
-	if (argc >= 1)
+	enum target_reset_mode reset_mode = RESET_RUN;
+	if (argc == 1)
 	{
+		const Jim_Nvp *n;
 		n = Jim_Nvp_name2value_simple( nvp_reset_modes, args[0] );
 		if( (n->name == NULL) || (n->value == RESET_UNKNOWN) ){
 			return ERROR_COMMAND_SYNTAX_ERROR;
-- 
GitLab