makedefs build on non-C99 compilers follow up

Pat Rankin wrote:
>      That patch looks incorrect.  `CONSUME' increments argv,
> so now a different value is being passed to the function when
> initializing that variable than was passed before.
This commit is contained in:
nethack.allison
2008-05-21 02:15:09 +00:00
parent 3a75bf5ab7
commit 91016cff99
+4 -2
View File
@@ -422,8 +422,9 @@ do_ext_makedefs(int argc, char **argv){
CONTINUE; CONTINUE;
} }
IS_OPTION("grep-define"){ IS_OPTION("grep-define"){
struct grep_var *p = grepsearch(argv[0]); struct grep_var *p;
CONSUME; CONSUME;
p = grepsearch(argv[0]);
if(p){ if(p){
p->is_defined = 1; p->is_defined = 1;
} else { } else {
@@ -433,8 +434,9 @@ do_ext_makedefs(int argc, char **argv){
CONTINUE; CONTINUE;
} }
IS_OPTION("grep-undef"){ IS_OPTION("grep-undef"){
struct grep_var *p = grepsearch(argv[0]); struct grep_var *p;
CONSUME; CONSUME;
p = grepsearch(argv[0]);
if(p){ if(p){
p->is_defined = 0; p->is_defined = 0;
} else { } else {