Module Name: src Committed By: lukem Date: Tue Apr 14 09:41:31 UTC 2009
Modified Files: src/usr.bin/yacc: defs.h error.c lalr.c lr0.c main.c mkpar.c output.c reader.c symtab.c verbose.c Log Message: Fix WARNS=4 issues (-Wshadow -Wcast-qual) To generate a diff of this commit: cvs rdiff -u -r1.16 -r1.17 src/usr.bin/yacc/defs.h cvs rdiff -u -r1.10 -r1.11 src/usr.bin/yacc/error.c src/usr.bin/yacc/lalr.c cvs rdiff -u -r1.9 -r1.10 src/usr.bin/yacc/lr0.c src/usr.bin/yacc/verbose.c cvs rdiff -u -r1.20 -r1.21 src/usr.bin/yacc/main.c cvs rdiff -u -r1.11 -r1.12 src/usr.bin/yacc/mkpar.c src/usr.bin/yacc/symtab.c cvs rdiff -u -r1.14 -r1.15 src/usr.bin/yacc/output.c cvs rdiff -u -r1.17 -r1.18 src/usr.bin/yacc/reader.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/usr.bin/yacc/defs.h diff -u src/usr.bin/yacc/defs.h:1.16 src/usr.bin/yacc/defs.h:1.17 --- src/usr.bin/yacc/defs.h:1.16 Sat Dec 15 22:01:25 2007 +++ src/usr.bin/yacc/defs.h Tue Apr 14 09:41:30 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: defs.h,v 1.16 2007/12/15 22:01:25 perry Exp $ */ +/* $NetBSD: defs.h,v 1.17 2009/04/14 09:41:30 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -226,9 +226,9 @@ extern char rflag; extern char tflag; extern char vflag; -extern char *symbol_prefix; +extern const char *symbol_prefix; -extern char *myname; +extern const char *myname; extern char *cptr; extern char *line; extern int lineno; @@ -243,7 +243,7 @@ extern char *action_file_name; extern char *code_file_name; extern char *defines_file_name; -extern char *input_file_name; +extern const char *input_file_name; extern char *output_file_name; extern char *text_file_name; extern char *union_file_name; @@ -314,19 +314,19 @@ extern char *allocate(unsigned); extern bucket *lookup(char *); -extern bucket *make_bucket(char *); +extern bucket *make_bucket(const char *); extern void set_first_derives(void); extern void closure(short *, int); extern void finalize_closure(void); -extern __dead void fatal(char *); +extern __dead void fatal(const char *); extern void reflexive_transitive_closure(unsigned *, int); extern __dead void done(int); extern __dead void no_space(void); -extern __dead void open_error(char *); +extern __dead void open_error(const char *); extern __dead void unexpected_EOF(void); extern void print_pos(char *, char *); extern __dead void syntax_error(int, char *, char *); Index: src/usr.bin/yacc/error.c diff -u src/usr.bin/yacc/error.c:1.10 src/usr.bin/yacc/error.c:1.11 --- src/usr.bin/yacc/error.c:1.10 Wed May 24 18:01:43 2006 +++ src/usr.bin/yacc/error.c Tue Apr 14 09:41:30 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: error.c,v 1.10 2006/05/24 18:01:43 christos Exp $ */ +/* $NetBSD: error.c,v 1.11 2009/04/14 09:41:30 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)error.c 5.3 (Berkeley) 6/1/90"; #else -__RCSID("$NetBSD: error.c,v 1.10 2006/05/24 18:01:43 christos Exp $"); +__RCSID("$NetBSD: error.c,v 1.11 2009/04/14 09:41:30 lukem Exp $"); #endif #endif /* not lint */ @@ -46,7 +46,7 @@ #include "defs.h" __dead void -fatal(char *msg) +fatal(const char *msg) { fprintf(stderr, "%s: f - %s\n", myname, msg); done(2); @@ -61,7 +61,7 @@ } __dead void -open_error(char *filename) +open_error(const char *filename) { fprintf(stderr, "%s: f - cannot open \"%s\"\n", myname, filename); done(2); Index: src/usr.bin/yacc/lalr.c diff -u src/usr.bin/yacc/lalr.c:1.10 src/usr.bin/yacc/lalr.c:1.11 --- src/usr.bin/yacc/lalr.c:1.10 Wed May 24 18:06:58 2006 +++ src/usr.bin/yacc/lalr.c Tue Apr 14 09:41:30 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: lalr.c,v 1.10 2006/05/24 18:06:58 christos Exp $ */ +/* $NetBSD: lalr.c,v 1.11 2009/04/14 09:41:30 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)lalr.c 5.3 (Berkeley) 6/1/90"; #else -__RCSID("$NetBSD: lalr.c,v 1.10 2006/05/24 18:06:58 christos Exp $"); +__RCSID("$NetBSD: lalr.c,v 1.11 2009/04/14 09:41:30 lukem Exp $"); #endif #endif /* not lint */ @@ -407,7 +407,7 @@ shifts *sp; int length; int nedges; - int done; + int isdone; int state1; int stateno; int symbol1; @@ -451,16 +451,16 @@ add_lookback_edge(stateno, *rulep, i); length--; - done = 0; - while (!done) + isdone = 0; + while (!isdone) { - done = 1; + isdone = 1; rp--; if (ISVAR(*rp)) { stateno = states[--length]; edge[nedges++] = map_goto(stateno, *rp); - if (nullable[*rp] && length > 0) done = 0; + if (nullable[*rp] && length > 0) isdone = 0; } } } @@ -517,7 +517,7 @@ static short ** -transpose(short **R, int n) +transpose(short **tR, int n) { short **new_R; short **temp_R; @@ -530,7 +530,7 @@ for (i = 0; i < n; i++) { - sp = R[i]; + sp = tR[i]; if (sp) { while (*sp >= 0) @@ -557,7 +557,7 @@ for (i = 0; i < n; i++) { - sp = R[i]; + sp = tR[i]; if (sp) { while (*sp >= 0) Index: src/usr.bin/yacc/lr0.c diff -u src/usr.bin/yacc/lr0.c:1.9 src/usr.bin/yacc/lr0.c:1.10 --- src/usr.bin/yacc/lr0.c:1.9 Wed May 24 18:01:43 2006 +++ src/usr.bin/yacc/lr0.c Tue Apr 14 09:41:30 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: lr0.c,v 1.9 2006/05/24 18:01:43 christos Exp $ */ +/* $NetBSD: lr0.c,v 1.10 2009/04/14 09:41:30 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)lr0.c 5.3 (Berkeley) 1/20/91"; #else -__RCSID("$NetBSD: lr0.c,v 1.9 2006/05/24 18:01:43 christos Exp $"); +__RCSID("$NetBSD: lr0.c,v 1.10 2009/04/14 09:41:30 lukem Exp $"); #endif #endif /* not lint */ @@ -615,7 +615,7 @@ { int i, j; int empty; - int done; + int isdone; nullable = MALLOC(nsyms); if (nullable == 0) no_space(); @@ -623,10 +623,10 @@ for (i = 0; i < nsyms; ++i) nullable[i] = 0; - done = 0; - while (!done) + isdone = 0; + while (!isdone) { - done = 1; + isdone = 1; for (i = 1; i < nitems; i++) { empty = 1; @@ -642,7 +642,7 @@ if (!nullable[j]) { nullable[j] = 1; - done = 0; + isdone = 0; } } } Index: src/usr.bin/yacc/verbose.c diff -u src/usr.bin/yacc/verbose.c:1.9 src/usr.bin/yacc/verbose.c:1.10 --- src/usr.bin/yacc/verbose.c:1.9 Wed May 24 18:01:43 2006 +++ src/usr.bin/yacc/verbose.c Tue Apr 14 09:41:31 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: verbose.c,v 1.9 2006/05/24 18:01:43 christos Exp $ */ +/* $NetBSD: verbose.c,v 1.10 2009/04/14 09:41:31 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)verbose.c 5.3 (Berkeley) 1/20/91"; #else -__RCSID("$NetBSD: verbose.c,v 1.9 2006/05/24 18:01:43 christos Exp $"); +__RCSID("$NetBSD: verbose.c,v 1.10 2009/04/14 09:41:31 lukem Exp $"); #endif #endif /* not lint */ @@ -328,7 +328,7 @@ static void -print_reductions(action *p, int defred) +print_reductions(action *p, int prdefred) { int k, anyreds; action *q; @@ -349,7 +349,7 @@ { for (; p; p = p->next) { - if (p->action_code == REDUCE && p->number != defred) + if (p->action_code == REDUCE && p->number != prdefred) { k = p->number - 2; if (p->suppressed == 0) @@ -358,8 +358,8 @@ } } - if (defred > 0) - fprintf(verbose_file, "\t. reduce %d\n", defred - 2); + if (prdefred > 0) + fprintf(verbose_file, "\t. reduce %d\n", prdefred - 2); } } @@ -369,15 +369,15 @@ { int i, k; int as; - short *to_state; + short *state; shifts *sp; putc('\n', verbose_file); sp = shift_table[stateno]; - to_state = sp->shift; + state = sp->shift; for (i = 0; i < sp->nshifts; ++i) { - k = to_state[i]; + k = state[i]; as = accessing_symbol[k]; if (ISVAR(as)) fprintf(verbose_file, "\t%s goto %d\n", symbol_name[as], k); Index: src/usr.bin/yacc/main.c diff -u src/usr.bin/yacc/main.c:1.20 src/usr.bin/yacc/main.c:1.21 --- src/usr.bin/yacc/main.c:1.20 Mon Jul 21 14:19:28 2008 +++ src/usr.bin/yacc/main.c Tue Apr 14 09:41:30 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: main.c,v 1.20 2008/07/21 14:19:28 lukem Exp $ */ +/* $NetBSD: main.c,v 1.21 2009/04/14 09:41:30 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -46,7 +46,7 @@ #if 0 static char sccsid[] = "@(#)main.c 5.5 (Berkeley) 5/24/93"; #else -__RCSID("$NetBSD: main.c,v 1.20 2008/07/21 14:19:28 lukem Exp $"); +__RCSID("$NetBSD: main.c,v 1.21 2009/04/14 09:41:30 lukem Exp $"); #endif #endif /* not lint */ @@ -61,8 +61,8 @@ char tflag; char vflag; -char *symbol_prefix; -char *myname = "yacc"; +const char *symbol_prefix; +const char *myname = "yacc"; int lineno; @@ -71,7 +71,7 @@ char *action_file_name; char *code_file_name; char *defines_file_name; -char *input_file_name = ""; +const char *input_file_name = ""; char *output_file_name; char *text_file_name; char *union_file_name; @@ -110,8 +110,8 @@ short **derives; char *nullable; -static char *file_prefix = "y"; -static char *temp_form = "yacc.XXXXXXX"; +static const char *file_prefix = "y"; +static const char *temp_form = "yacc.XXXXXXX"; static int explicit_file_name; @@ -300,7 +300,7 @@ create_file_names(void) { int i, len; - char *tmpdir; + const char *tmpdir; tmpdir = getenv("TMPDIR"); if (tmpdir == 0) tmpdir = "/tmp"; Index: src/usr.bin/yacc/mkpar.c diff -u src/usr.bin/yacc/mkpar.c:1.11 src/usr.bin/yacc/mkpar.c:1.12 --- src/usr.bin/yacc/mkpar.c:1.11 Wed May 24 18:01:43 2006 +++ src/usr.bin/yacc/mkpar.c Tue Apr 14 09:41:31 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: mkpar.c,v 1.11 2006/05/24 18:01:43 christos Exp $ */ +/* $NetBSD: mkpar.c,v 1.12 2009/04/14 09:41:31 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)mkpar.c 5.3 (Berkeley) 1/20/91"; #else -__RCSID("$NetBSD: mkpar.c,v 1.11 2006/05/24 18:01:43 christos Exp $"); +__RCSID("$NetBSD: mkpar.c,v 1.12 2009/04/14 09:41:31 lukem Exp $"); #endif #endif /* not lint */ @@ -104,7 +104,7 @@ { action *actions, *temp; shifts *sp; - short *to_state; + short *state; int i, k; int symbol; @@ -112,10 +112,10 @@ sp = shift_table[stateno]; if (sp) { - to_state = sp->shift; + state = sp->shift; for (i = sp->nshifts - 1; i >= 0; i--) { - k = to_state[i]; + k = state[i]; symbol = accessing_symbol[k]; if (ISTOKEN(symbol)) { @@ -200,15 +200,15 @@ find_final_state(void) { int goal, i; - short *to_state; + short *state; shifts *p; p = shift_table[0]; - to_state = p->shift; + state = p->shift; goal = ritem[1]; for (i = p->nshifts - 1; i >= 0; --i) { - final_state = to_state[i]; + final_state = state[i]; if (accessing_symbol[final_state] == goal) break; } } Index: src/usr.bin/yacc/symtab.c diff -u src/usr.bin/yacc/symtab.c:1.11 src/usr.bin/yacc/symtab.c:1.12 --- src/usr.bin/yacc/symtab.c:1.11 Wed May 24 18:06:58 2006 +++ src/usr.bin/yacc/symtab.c Tue Apr 14 09:41:31 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: symtab.c,v 1.11 2006/05/24 18:06:58 christos Exp $ */ +/* $NetBSD: symtab.c,v 1.12 2009/04/14 09:41:31 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)symtab.c 5.3 (Berkeley) 6/1/90"; #else -__RCSID("$NetBSD: symtab.c,v 1.11 2006/05/24 18:06:58 christos Exp $"); +__RCSID("$NetBSD: symtab.c,v 1.12 2009/04/14 09:41:31 lukem Exp $"); #endif #endif /* not lint */ @@ -54,13 +54,13 @@ static bucket **symbol_table; -static int hash(char *); +static int hash(const char *); static int -hash(char *name) +hash(const char *name) { - char *s; + const char *s; int c, k; assert(name && *name); @@ -74,7 +74,7 @@ bucket * -make_bucket(char *name) +make_bucket(const char *name) { bucket *bp; Index: src/usr.bin/yacc/output.c diff -u src/usr.bin/yacc/output.c:1.14 src/usr.bin/yacc/output.c:1.15 --- src/usr.bin/yacc/output.c:1.14 Wed May 24 18:01:43 2006 +++ src/usr.bin/yacc/output.c Tue Apr 14 09:41:31 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: output.c,v 1.14 2006/05/24 18:01:43 christos Exp $ */ +/* $NetBSD: output.c,v 1.15 2009/04/14 09:41:31 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)output.c 5.7 (Berkeley) 5/24/93"; #else -__RCSID("$NetBSD: output.c,v 1.14 2006/05/24 18:01:43 christos Exp $"); +__RCSID("$NetBSD: output.c,v 1.15 2009/04/14 09:41:31 lukem Exp $"); #endif #endif /* not lint */ @@ -931,7 +931,7 @@ output_debug(void) { int i, j, k, max; - char **symnam, *s; + const char **symnam, *s; ++outline; fprintf(code_file, "#define YYFINAL %d\n", final_state); @@ -949,7 +949,7 @@ ++outline; fprintf(code_file, "#define YYMAXTOKEN %d\n", max); - symnam = (char **) MALLOC((max+1)*sizeof(char *)); + symnam = (const char **) MALLOC((max+1)*sizeof(char *)); if (symnam == 0) no_space(); /* Note that it is not necessary to initialize the element */ Index: src/usr.bin/yacc/reader.c diff -u src/usr.bin/yacc/reader.c:1.17 src/usr.bin/yacc/reader.c:1.18 --- src/usr.bin/yacc/reader.c:1.17 Fri Nov 24 19:47:00 2006 +++ src/usr.bin/yacc/reader.c Tue Apr 14 09:41:31 2009 @@ -1,4 +1,4 @@ -/* $NetBSD: reader.c,v 1.17 2006/11/24 19:47:00 christos Exp $ */ +/* $NetBSD: reader.c,v 1.18 2009/04/14 09:41:31 lukem Exp $ */ /* * Copyright (c) 1989 The Regents of the University of California. @@ -37,7 +37,7 @@ #if 0 static char sccsid[] = "@(#)reader.c 5.7 (Berkeley) 1/20/91"; #else -__RCSID("$NetBSD: reader.c,v 1.17 2006/11/24 19:47:00 christos Exp $"); +__RCSID("$NetBSD: reader.c,v 1.18 2009/04/14 09:41:31 lukem Exp $"); #endif #endif /* not lint */ @@ -64,7 +64,6 @@ static int linesize; static bucket *goal; -static int prec; static int gensym; static char last_was_action; @@ -920,6 +919,7 @@ bucket *bp; int value; char *tag = 0; + static int prec; if (assoc != TOKEN) ++prec;