Yes, I was going to split it up, but can add that to my diff as well.
Sent from my BlackBerry 10 smartphone on the Rogers network.
Original Message
From: Jeremie Courreges-Anglas
Sent: Thursday, August 3, 2017 11:08 AM
To: Rob Pierce
Cc: tech@openbsd.org
Subject: Re: dhcpd: remove unused structs
On Tue, Aug 01 2017, Rob Pierce <r...@2keys.ca> wrote:
> I can confirm that the following structs have not been used since at least
> OpenBSD 4.9. From Edgar Pettijohn.
>
> Ok?
ok
Looks like youncould trim this file some more (untested):
@@ -315,20 +290,6 @@ struct protocol {
void (*handler)(struct protocol *);
void *local;
};
-
-/* Bitmask of dhcp option codes. */
-typedef unsigned char option_mask[16];
-
-/* DHCP Option mask manipulation macros... */
-#define OPTION_ZERO(mask) (memset (mask, 0, 16))
-#define OPTION_SET(mask, bit) (mask[bit >> 8] |= (1 << (bit & 7)))
-#define OPTION_CLR(mask, bit) (mask[bit >> 8] &= ~(1 << (bit & 7)))
-#define OPTION_ISSET(mask, bit) (mask[bit >> 8] & (1 << (bit & 7)))
-#define OPTION_ISCLR(mask, bit) (!OPTION_ISSET (mask, bit))
-
-/* An option occupies its length plus two header bytes (code and
- length) for every 255 bytes that must be stored. */
-#define OPTION_SPACE(x) ((x) + 2 * ((x) / 255 + 1))
#define _PATH_DHCPD_CONF "/etc/dhcpd.conf"
#define _PATH_DHCPD_DB "/var/db/dhcpd.leases"
--
jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF DDCC 0DFA 74AE 1524 E7EE
