Use a modern spacious idiom on all function local variable
declarations.

Make pkg-config regress tests pass:
- Allow to override PKG_CONFIG and check correct place in logfile.
- Replace "if ...; then false; fi" with a simple ! .
- Remove comments about failing test which do not fail.
- Adapt regress to changes in pkg-config rev 1.64 and 1.72 regarding
  static and missing tests.
OK jasper@

Some extra logging to show why tmux might exit.

Be consistent. "return (e);" -> "return e;"

remove redundant variable declarations in Makefiles, since those are
the default.

okay millert@

Be consistent. "return (e);" -> "return e;"

Some parsing code cleanup: add parse_boolean(); pass literal format
chars to parse_decimal() instead of less obvious *fmt; refactor
to eliminate need for the 'alloc:' and 'bad_flag:' labels and the
invidious backwards goto's to them.

document PROGS, okay jmc@

document more reality.
proper english usage and okay jmc@

use working boilerplate for yacc/lex instead of homemade rules.
okay millert@
(forgot the obvious scanner.l tweak in my diff)

add support to compile several progs at once us
by using alternate setup
PROGS = a b c along with SRCS_a = ... , SRCS_b = ... , SRCS_c = ...

okay millert@

zap empty depends/silence empty installs.
okay millert@

there's no need to fork/exec to execute comments
okay millert@

have bsd.prog.mk/bsd.lib.mk generate .y -> .c/.h rules without intermediate
files. This fixes up parallel builds in the default case.
FreeBSD does something similar.

okay millert@

Whitespace tweaks to assuage auto-indent.

Ignore prefixes with a pltime < ND6_PRIV_MAX_DESYNC_FACTOR seconds
(512 in our case) for privacy addresses as per RFC 4941.

If we lower the pltime for privacy addresses skew it by
ND6_PRIV_MAX_DESYNC_FACTOR like RFC 4941 tells us to.
Toss in a arc4random_uniform for good measure like the kernel always
did.

Remove old junk. From Matthew Martin

ok bentley

vmd/vmctl: Add ability to pause / unpause vms

With help from Ashwin Agrawal

ok reyk@ mlarkin@

this program was infected with lint era casts. i think we're past that now.

remove ARGSUSED, from Scott Cheloha.
and while we're cleaning, switch __progname to getprogname.

use crypt_newhash to protect the password. from Scott Cheloha

clean up some unused variables. also, making a variable global isn't the
best way to zero initialize it if that's all you need.

instead of using time(), refer to getitimer to find out how long the
timeout is.
from Scott Cheloha

can just continue after readpassphrase returns null, the sighandler will
have already printed a message.
from Scott Cheloha

update the little endian processor list to give it a chance of matching
what the reader is using.

Unhook tables.c from Makefile.

Fold tables.c into options.c and stop exporting the one
table (dhcp_options) involved. Provide functions
code_to_name(), name_to_code(), code_to_format() and
replace direct access to dhcp_options with them. Eliminate
unneeded 'struct option'.

Unhook tables.c from Makefile.

Remove duplicate typedesf that are now provided by linux_types.h.

slightly rework previous, to avoid an article issue;

spelling fix from Hiltjo Posthuma

Simplify by creating struct roff_node syntax tree nodes for tbl(7)
right from roff_parseln() rather than delegating to read.c,
similar to what i just did for eqn(7).

The interface function roff_span() becomes obsolete and is deleted,
the former interface function roff_addtbl() becomes static,
the interface functions tbl_read() and tbl_cdata() become void,
and minus twelve linus of code.

No functional change.

Consistently use if ((option & F_FOO) && (option & F_BAR)) instead of
if (option & F_FOO && option & F_BAR).

Prompted by a reverse diff from Klemens Nanni. Both forms are
equivalent due to operator precedence, I consider the later to be
easier on the eyes.

Consistently use if (option & F_FOO); from Klemens Nanni, thanks!

Enable slaacd on the installer and temporarily neuter router
advertisement processing and solicitation sending like we already did
for the non-install kernels.

Next step will be to tedu all that code from the kernel.

OK rpe@ for the installer bits
move forward deraadt@

hook up slaacd to the installer on media where we have INET6

_slaacd user/group for installer
move forward deraadt@

slaacd needs libevent on the ramdisk
move forward deraadt@

Increase size to make slaacd fit.
Size problem noted by me, correct fix from deraadt@

fix an assertion failure triggered by .ce in next-line scope;
found by jsg@ with afl(1)

Always use strcasecmp() when comparing user input to option
names, not just 2 out of 3.

1. Eliminate struct eqn, instead use the existing members
of struct roff_node which is allocated for each equation anyway.
2. Do not keep a list of equation parsers, one parser is enough.
Minus fifty lines of code, no functional change.

- For RTL8188CUS/RTL8192CU, we have to force 8051 reset/enable before waiting
  for firmware to get ready.  It fixes "urtwn0: could not load firmware page"
  while running stsp@'s script: dhclient urtwn0; ifconfig urtwn0 down;
  ifconfig urtwn0 scan in a loop.

- Minor cleanup for rtwn_read_rom()/rtwn_get_txpower(); no need to use
  callbacks.

Tested by stsp@ and me.

ok stsp@

Correctly handle horizontal spans at the beginning of rows,
fixing an assertion failure found by jsg@ with afl(1).

While here, also drop printing of whitespace in tbl_data()
which makes no difference because column positioning code
in term_tbl() already takes care of that.

fix typo in comment

from Hiltjo Posthuma

Revert grabbing the socket lock in kqueue filters.

It is unsafe to sleep while iterating the list of pending events in
kqueue_scan().

Reported by abieber@ and juanfra@

add a hint about subjectAltName. this isn't great, but i'm not sure how
to improve it without muddying up the whole page, and at a minimum it
gives the reader a clue about some concepts to search the web for.
if you can do better, make it so. :)

Always use uintNN_t instead of sometimes u_intNN_t
and sometimes uintNN_t.

Run malloc0test with all possible malloc options.

Remove all references to "make depend" from regress.

Remove useless make depend targets.

garbage collect unused enum member EQN_ROOT

Only access offset if canaries are enabled *and* size > 0, otherwise offset
is not initialized. Problem spotted by Carlin Bingham; ok phessler@ tedu@

add parentheses to the output where required for disambiguation

You win some, you los some.  The Linux 4.4 code is still sub-standard but
triggers different warnings now.

ok naddy@

Handle carp(4) as HTYPE_ETHER.

This fixes "dhcrelay -i carpX" that used to work when dhcrelay didn't care.

Reported and tested by Kapetanakis Giannis bilias at edu.physics.uoc.gr

simplify the eqn_box_makebinary() function by removing the trivial pos
parameter; also minus two lines of code; no functional change

Replace the many occurances of '256' with a new #define
DHO_COUNT.

Disable tests that fail due to known make bugs, fix the others.

clarify which httpd we are talking about;
from Raf Czlonka <rczlonka at gmail dot com>

Fix size of rightmost preview section.

Replace a doubled period and add a blank space in front of a bracket.
From Klemens Nanni

ok rpe

Radically simplify the definitions what the message levels ERROR
and WARNING mean: minus 20 lines of mdoc source.  OK jmc@.

dispatch_imsg() only needs to know name and rdomain.

No more struct interface_info knowledge in privsep.[ch]

assemble_eh_header() needs only to know about hw_addr.

No more struct interface_info knowledge in packet.c

Rename cons_options() to pack_options(), and do_packet() to
unpack_options(). Store the unpacked options in a static
variable. Move remaining raw packet processing from unpack_options()
to packethandler().

No more struct interface_info knowledge in options.c

Add logic for running SD commands. Tested with a few different makes
of MMC/SD memory.

Add a pane_pipe format to show if pipe-pane is active, GitHub issue 990.

trigger default .l.o  rule and check the produced file has the right name

COMPILE.c already does -c, so no need to double it.
no functional change

unbreak
this specific rule is only triggered twice in the whole ports tree
(sysutils/xjobs and print/l2a)

This fixes them

we're not shooting yacception

okay millert@

Make libtool regress tests pass:
- There are no NOPIC architectures anymore.
- Add DISABLED targets for tests failing intensionally without
  touching the real targets.
- In execute mode libtool command line must use ./p2 as .  is not
  in my PATH.
OK mpi@ espie@

Make mmap_hint.c compile on i386 by adding includes.  Unfortunately
test is still failing.

Fix function name in panic message.

Make configuration lines match GENERIC files.

This adds amd64 and splits up alpha and i386.

OK deraadt@

Remove unnecessary #ifdefs in telnet. No binary change.

OK deraadt@, tedu@

When working out the current client (for example for switch-client with
no target), prefer clients attached to the current session if there is
one. GitHub issue 995 from Jan Larres.

When generating all hostkeys (ssh-keygen -A), clobber existing keys
if they exist but are zero length. zero-length keys could previously
be made if ssh-keygen failed part way through generating them, so avoid
that case too. bz#2561 reported by Krzysztof Cieplucha; ok dtucker@

allow fetching lists from https:// URLs too

switch user to _spamd before executing ftp(1) to fetch lists.
sprinkle in some closefrom(2); ok deraadt@ beck@

Now that we have the -Wstyle message level, downgrade six warnings
that are not syntax mistakes and that do not cause wrong formatting
or content to style suggestions.
Also upgrade two warnings that may cause information loss to errors.

sync

REGRESS_TARGET has been renamed to TEST_TARGET.
spotted by anton@

Delete variable REGRESS_TARGET, missing plural S is a typo and it
is not used.  Convert tests into a common style.
OK anton@

anton@ has fixed the test script so that it can run as root.  Remove
my workaround that switched to build user.

Fix display of overlong lines containing non-ASCII bytes.
Also fixes a crash reported by Hiltjo Posthuma <hiltjo at codemadness
dot org>, though in a different way than with the patch he sent.
OK florian@ bcallah@

revert previous, requested by jmc@; he says the broken .Xr is intentional

Our website says that socppc was discontinued after 5.8 (thanks to
tobiasu@ for pointing that out), but the manual pages are still
installed.  I have no idea how to properly tedu an architecture, so
deleting the dead .Xr to boot_socppc(8) is all i'm doing in this respect.

Document tls_config_set_crl_file() and tls_config_set_crl_mem().

Based on a diff from Jack Burton <jack at saosce dot com dot au>, thanks!

Delete cross references to boot_landisk(8).
According to tobiasu@, landisk is moribund and writing new manual
pages for it would be a waste of time.

Bump minor due to symbol addition.

Add support for providing CRLs to libtls - once a CRL is provided we
enable CRL checking for the full certificate chain.

Based on a diff from Jack Burton <jack at saosce dot com dot au>, thanks!

Discussed with beck@

fix RCS Id; found with mandoc -Tlint

cons_options() only needs to know a buffer and a length to
pack options into. Not all the gory details of interface_info.

Move some of the raw packet processing out of options.c's
do_packet() and into the more obvious dispatch.c's
packethandler().

Mention that RFC791 is why we use 576-byte UDP packets.

delete duplicate RCS ID and lots of .Tn

delete duplicate RCS IDs; found with mandoc -Tlint

Do not suppress what's going on.

The 0x (or 0X) prefix in base 16 is optional so only skip over the
prefix if the character following it is a valid hex char.  The C99
standard is clear that given the string "0xy" zero should be returned
and endptr set to point to the "x".  OK deraadt@ espie@

fix broken cross references; found with mandoc -Tlint