dhclient(8): fork_privchld, take_charge, propose_release: poll(2) -> ppoll(2)
authorcheloha <cheloha@openbsd.org>
Tue, 2 Feb 2021 15:46:16 +0000 (15:46 +0000)
committercheloha <cheloha@openbsd.org>
Tue, 2 Feb 2021 15:46:16 +0000 (15:46 +0000)
commit32bab3b6920ec1e662677c4467903c68962a5caf
treeaed6886115cfed4019d932d886cefaa40e1d52bf
parent58c7d7ac79dfd59b623e4857c310b266cc721e5e
dhclient(8): fork_privchld, take_charge, propose_release: poll(2) -> ppoll(2)

Switch from poll(2) to ppoll(2) in a few more functions.

Because we're working with ppoll(2) and clock_gettime(2) it is easier
to encode the various timeouts as static const timespecs instead of
preprocessor macros.  This way we aren't packing timespecs in the
middle of the code, which distracts from the (more important) logic of
what the code is doing.

Part of a larger campaign improve "time stuff" in dhclient(8).

Prompted by and discussed with krw@.  Based on a diff by krw@.

ok krw@
sbin/dhclient/dhclient.c