Commit Graph

97 Commits

Author SHA1 Message Date
cperciva
55239b4ba7 Setting a variable to the same value twice doesn't actually make it
more likely to have the right value.  Remove superfluous assignments.

Found by:	LLVM/Clang Static Checker
2008-08-04 06:39:52 +00:00
cperciva
45f0d08992 Mark functions as __dead2 in order to help the LLVM static checker
understand which code paths aren't possible.

This commit eliminates 117 false positive bug reports of the form
"allocate memory; error out if pointer is NULL; use pointer".
2008-08-04 01:25:48 +00:00
delphij
ec5327b1cd Indent the else path, reduce diff against OpenBSD. 2008-07-18 23:35:31 +00:00
kevlo
08fb11c891 Fix a longstanding bug, from Otto Moerbeck:
if we're reducing a rule that has an empty
right hand side and the yacc stackpointer is pointing at the very
end of the allocated stack, we end up accessing the stack out of
bounds by the implicit $$ = $1 action

Obtained from:	OpenBSD
2008-07-18 15:05:14 +00:00
ru
7f3c7f0d46 Sort sections. 2005-01-18 13:43:56 +00:00
ru
fb1d8b3724 Mechanically kill hard sentence breaks. 2004-07-02 22:22:35 +00:00
wes
c1543e1757 Use getopt instead of hand-rolled argument parsing. Usage remains
the same, no man page changes required.

PR:		bin/48313
Submitted by:	Stefan Farfeleder <stefan@fafoe.dyndns.org>
Reviewed by:	joe@
2004-03-05 01:52:09 +00:00
davidc
7df8d20337 Add __unused to the declaration of yyrcsid in a more portable way.
Discussed with: alfred, bde, jmallett, obrien
2003-02-12 18:03:55 +00:00
davidc
ff5d2916db Backout my previous commit as requested. This solution generates
parsers that are non-portable.
2003-02-08 17:23:22 +00:00
davidc
583bf78cf8 Add __unused to the declaration of yyrcsid. 2003-02-07 03:39:27 +00:00
ru
e7bd2cb66e mdoc(7) police: back out unproved changes from previous revision.
Approved by:	re
2002-11-26 15:34:18 +00:00
charnier
bdf2708e48 Add section number to .Xr directive. 2002-10-16 15:46:43 +00:00
charnier
7dd9d47059 Replace various spelling with FALLTHROUGH which is lint()able 2002-08-25 13:23:09 +00:00
robert
f85d0f0060 Use the constants from <limits.h> for the sizes of integral C types
rather than defining them ourselves.
2002-06-11 11:27:20 +00:00
jmallett
241a475e6f Note that this appeared at least as early as PWB UNIX.
Use the literal string 'PWB UNIX', as we still have no .At macro for it.
2002-06-10 22:59:59 +00:00
ru
67d93076d6 mdoc(7) police: tidy up the markup. 2002-05-30 06:56:43 +00:00
obrien
3311fa2aba Restore some of the original variable names, but fix the warnings their
renaming addressed by removing the global variants instead.
2002-04-23 20:30:30 +00:00
charnier
ad8a79e6a5 Use `The .Nm utility' 2002-04-20 12:18:28 +00:00
fenner
798d62af10 mdoc police: use .El to end the list in the ENVIRONMENT section. 2002-04-17 21:34:47 +00:00
obrien
8c178db49d Quiet uninitialized warnings. 2002-04-10 03:00:50 +00:00
ru
6d38d7dc3c No longer needed to #ifdef __FBSDID, this is now handled by Makefile.inc1. 2002-04-09 11:39:05 +00:00
obrien
1f93d739ef Revert the "tell the program name in diagnostics". What was I thinking??
warnx() already does this for us.
2002-04-05 03:21:41 +00:00
obrien
ef28acc1be Minor reordering of declarations to reduce diffs to OpenBSD/NetBSD. 2002-04-04 22:25:04 +00:00
obrien
30affec10e Allow to compile errorlessly with GCC 3.1. 2002-04-04 22:15:56 +00:00
obrien
6d04a5e00d Fix signal race.
Use strlen() rather than magic number.
Use _PATH_TMP.

Obtained from:	OpenBSD rev 1.13 (sig race) & rev 1.9 (strlen)
2002-04-04 22:15:13 +00:00
obrien
e14f194d59 Revert revision 1.3. This is WARNS=4 clean w/o it and it only increases
the diffs to Open/NetBSD.
2002-04-04 22:10:23 +00:00
obrien
6cdcffea17 Fix style regression introduced in rev 1.11 during a WARNS cleanup. 2002-04-04 21:59:10 +00:00
obrien
cc9830e3c2 Add STANDARDS, ENVIRONMENT, and TABLES sections.
Obtained from:	OpenBSD
2002-04-04 21:31:03 +00:00
obrien
efdbbdd9ce Minor word smithing. 2002-04-04 21:24:19 +00:00
obrien
430aedfb79 Follow suit with OpenBSD and NetBSD and tell the program name in diagnostics. 2002-04-04 21:09:06 +00:00
obrien
5b964d2945 Follow suit with OpenBSD and NetBSD and tell the program name in diagnostics. 2002-04-04 20:58:27 +00:00
imp
0b20191705 remove __P 2002-03-22 01:42:45 +00:00
markm
31d81e4fbc Remove NO_WERRORs and WARNS=n's. To be revisited after GCC3. 2002-02-08 23:07:37 +00:00
markm
9c9a045dca WARNS=2 fix, but do not set this im Makefile, because this ill be the
default for src/usr.bin/
2001-12-11 22:13:02 +00:00
dwmalone
d9613ea383 Style improvements recommended by Bruce as a follow up to some
of the recent WARNS commits. The idea is:

1) FreeBSD id tags should follow vendor tags.
2) Vendor tags should not be compiled (though copyrights probably should).
3) There should be no blank line between including cdefs and __FBSDIF.
2001-12-10 21:13:08 +00:00
imp
8f226dc222 Only use __FBSDID if it is defined. This fixes the 4.4-release (but
not stable) -> current upgrade path.

Reviewed by: markm
2001-12-08 02:28:16 +00:00
markm
c73844e02d Use __FBSDID(). 2001-12-02 21:24:03 +00:00
markm
4a974575ba WARNS=2 and s/register//.
Set NO_WERROR=true because there are some flex(1) issues that need
to be dealt with.
2001-12-01 17:34:42 +00:00
obrien
be3ae37c00 Properly static'ize increase_maxtable().
Submitted by:	db@db.net
2001-10-05 03:02:04 +00:00
obrien
1547f1d07f Fix the output so it really does dynamically resize the table.
Submitted by:	Diane Bruce <db@db.net>
2001-10-05 02:09:43 +00:00
obrien
7d2463e9d1 Increase the table size by an order of magnitude.
The previous version was too small and YACC core dumped on the during the
hybrid-7 build.

Submitted by:	Diane Bruce <db@db.net>
2001-10-05 00:46:22 +00:00
ru
36f138439b mdoc(7) police: removed HISTORY info from the .Os call. 2001-07-10 14:16:33 +00:00
ru
ef2dbc1fb0 beforeinstall -> SCRIPTS. 2001-04-07 11:21:35 +00:00
ru
89596e4538 MAN[1-9] -> MAN. 2001-03-27 10:52:19 +00:00
peter
589564befb Have yacc use a prototype for yygrowstack(void) on ANSI compilers.
This fixes a gcc warning with -Wmissing-prototypes.
2001-02-19 01:10:01 +00:00
kris
b64f677da6 Constify 2000-11-26 11:07:45 +00:00
n_hibma
5240cf8e35 Fix breakage introduced in rev.1.7. yystacksize and yysslim are global and
should be prefixed by YYPREFIX as well.

Only relevant if -p <yyprefix> is used. Used in cases where multiple parsers
are used in executable.
2000-09-07 15:01:18 +00:00
bde
668e3ec014 Fixed breakage of K&R support in rev.1.26. yyparse() was defined as
`int yyparse(;) ; { ... }' in K&R mode.  Getting rid of the second
unwanted semicolon in this made the ifdef tangle more tangled than
before.  Fixed a backwards comment in the tangle.
2000-01-17 02:04:06 +00:00
bde
6b3da1fe21 Fixed breakage of K&R support in rev.1.8 of output.c: don't generate
#elif.  Cleaned up rev.1.8 a bit more: generate the #include of
<stdio.h> closer to the code that needs it.
2000-01-17 01:51:29 +00:00
kris
98f787026a malloc more space for temp file name
Noticed by:	marcel
2000-01-10 20:26:24 +00:00