X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=blobdiff_plain;f=src%2Fhelper%2Fcommand.c;h=f474756ad692d976af89ad620f01f50d6c59e5d3;hp=5dbc047e6758fbe082ac7079ce516b65d3b9a964;hb=53d605e12c3765aeedabf2bfe0c5cc338dc95d5a;hpb=c0f4495d5e9736d77cb5142a6f5112764b5ca3d2

diff --git a/src/helper/command.c b/src/helper/command.c
index 5dbc047e67..f474756ad6 100644
--- a/src/helper/command.c
+++ b/src/helper/command.c
@@ -226,7 +226,7 @@ command_t* register_command(command_context_t *context, command_t *parent, char
 	const char *t2="";
 	const char *t3="";
 	/* maximum of two levels :-) */
-	if (c->parent!=NULL)
+	if (c->parent != NULL)
 	{
 		t1=c->parent->name;
 		t2="_";
@@ -250,7 +250,7 @@ command_t* register_command(command_context_t *context, command_t *parent, char
 	Jim_Obj *cmd_list=Jim_NewListObj(interp, NULL, 0);
 
 	/* maximum of two levels :-) */
-	if (c->parent!=NULL)
+	if (c->parent != NULL)
 	{
 		Jim_ListAppendElement(interp, cmd_list, Jim_NewStringObj(interp, c->parent->name, -1));
 	}
@@ -269,11 +269,11 @@ int unregister_all_commands(command_context_t *context)
 	if (context == NULL)
 		return ERROR_OK;
 
-	while(NULL != context->commands)
+	while (NULL != context->commands)
 	{
 		c = context->commands;
 
-		while(NULL != c->children)
+		while (NULL != c->children)
 		{
 			c2 = c->children;
 			c->children = c->children->next;
@@ -304,7 +304,7 @@ int unregister_command(command_context_t *context, char *name)
 	/* find command */
 	c = context->commands;
 
-	while(NULL != c)
+	while (NULL != c)
 	{
 		if (strcmp(name, c->name) == 0)
 		{
@@ -320,7 +320,7 @@ int unregister_command(command_context_t *context, char *name)
 			}
 
 			/* unregister children */
-			while(NULL != c->children)
+			while (NULL != c->children)
 			{
 				c2 = c->children;
 				c->children = c->children->next;
@@ -348,7 +348,7 @@ int unregister_command(command_context_t *context, char *name)
 
 void command_output_text(command_context_t *context, const char *data)
 {
-	if( context && context->output_handler && data  ){
+	if ( context && context->output_handler && data  ){
 		context->output_handler( context, data );
 	}
 }
@@ -420,7 +420,7 @@ int run_command(command_context_t *context, command_t *c, char *words[], int num
 		const char *t2="";
 		const char *t3="";
 		/* maximum of two levels :-) */
-		if (c->parent!=NULL)
+		if (c->parent != NULL)
 		{
 			t1=c->parent->name;
 			t2=" ";
@@ -473,7 +473,7 @@ int command_run_line(command_context_t *context, char *line)
 		Jim_DeleteAssocData(interp, "context");
 	}
 	if (retcode == JIM_ERR) {
-		if (retval!=ERROR_COMMAND_CLOSE_CONNECTION)
+		if (retval != ERROR_COMMAND_CLOSE_CONNECTION)
 		{
 			/* We do not print the connection closed error message */
 			Jim_PrintErrorMessage(interp);
@@ -520,7 +520,7 @@ int command_run_linef(command_context_t *context, const char *format, ...)
 	va_list ap;
 	va_start(ap, format);
 	string = alloc_vprintf(format, ap);
-	if (string!=NULL)
+	if (string != NULL)
 	{
 		retval=command_run_line(context, string);
 	}
@@ -809,7 +809,7 @@ int handle_sleep_command(struct command_context_s *cmd_ctx,
 
 int handle_fast_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc)
 {
-	if (argc!=1)
+	if (argc != 1)
 		return ERROR_COMMAND_SYNTAX_ERROR;
 
 	fast_and_dangerous = strcmp("enable", args[0])==0;
@@ -872,9 +872,9 @@ long jim_global_long(const char *variable)
 		*ul = func(str, &end, 0); \
 		if (*end) \
 			return ERROR_COMMAND_ARGUMENT_INVALID; \
-		if (*ul == max) \
+		if ((max == *ul) && (ERANGE == errno)) \
 			return ERROR_COMMAND_ARGUMENT_OVERFLOW; \
-		if (min && min == *ul) \
+		if (min && (min == *ul) && (ERANGE == errno)) \
 			return ERROR_COMMAND_ARGUMENT_UNDERFLOW; \
 		return ERROR_OK; \
 	}