redis.c:2394:7: warning: variable 'nb_keys' set but not used

remicollet · remicollet · commit 297d83e10706 · 2012-08-30T07:43:32.000+02:00
diff --git a/redis.c b/redis.c
@@ -2391,12 +2391,7 @@ PHPAPI int generic_multiple_args_cmd(INTERNAL_FUNCTION_PARAMETERS, char *keyword
 			real_argc++;
 		}
     } else {
-		int nb_keys;
-		if(!has_timeout) {
-			nb_keys = argc;
-		} else if(has_timeout && Z_TYPE_P(z_args[argc - 1]) == IS_LONG) {
-			nb_keys = argc - 1;
-		} else {
+		if(has_timeout && Z_TYPE_P(z_args[argc - 1]) != IS_LONG) {
 			php_error_docref(NULL TSRMLS_CC, E_ERROR, "Syntax error on timeout");
 		}
 			

Original file line number	Diff line number	Diff line change
`@@ -2391,12 +2391,7 @@ PHPAPI int generic_multiple_args_cmd(INTERNAL_FUNCTION_PARAMETERS, char *keyword`
`2391`	`2391`	`real_argc++;`
`2392`	`2392`	`}`
`2393`	`2393`	`} else {`
`2394`		`- int nb_keys;`
`2395`		`- if(!has_timeout) {`
`2396`		`- nb_keys = argc;`
`2397`		`- } else if(has_timeout && Z_TYPE_P(z_args[argc - 1]) == IS_LONG) {`
`2398`		`- nb_keys = argc - 1;`
`2399`		`- } else {`
	`2394`	`+ if(has_timeout && Z_TYPE_P(z_args[argc - 1]) != IS_LONG) {`
`2400`	`2395`	`php_error_docref(NULL TSRMLS_CC, E_ERROR, "Syntax error on timeout");`
`2401`	`2396`	`}`
`2402`	`2397`