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;