Untimely spring cleaning:
authorschwarze <schwarze@openbsd.org>
Mon, 14 Jan 2019 09:06:04 +0000 (09:06 +0000)
committerschwarze <schwarze@openbsd.org>
Mon, 14 Jan 2019 09:06:04 +0000 (09:06 +0000)
commitf6862e3f4763df334e4aee390a46a59cb36fd3a5
tree0404a1146c21618dcc9666319aac8c5698f02641
parentda79231ed25db3e5931fbb8250e8e44f515f76ef
Untimely spring cleaning:
* Garbage collect useless hand-rolled lookup tables.
* Merge one-line helper functions into callers.
* Garbage collect obfuscating macros.
* Fix one format string error (%d used for u_int).
* Garbage collect setlocale(3) and <locale.h>.
* Garbage collect several unused constants.
* Minus 45 LOC, no functional change.

As noticed by tedu@, the lookup table "table" was used uninitialized.
But since it was only used as a pre-matching optimization and the real
string comparison is done further down the line, that bug probably did
not cause wrong results but merely ruined -i performance.

Code review triggered by a naive question from Jan Stary.
OK deraadt@ tedu@
usr.bin/locate/locate/fastfind.c
usr.bin/locate/locate/locate.c
usr.bin/locate/locate/locate.h
usr.bin/locate/locate/util.c