diff options
| author | Thomas Haller <thaller@redhat.com> | 2021-08-05 10:23:38 +0200 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2021-08-05 14:59:15 +0200 |
| commit | 29792975199e2a0dfc1722f310f6bf36e23e589d (patch) | |
| tree | e1f5d648b7e4b477762ff69c46aa3ec3c58ac4e8 | |
| parent | b4e4b8d6146c9c32eb13b88f0114507307d3165c (diff) | |
| download | NetworkManager-29792975199e2a0dfc1722f310f6bf36e23e589d.tar.gz | |
dhcp: drop NM_DHCP_STATE_MAX enum value
These meta flags were not actually used. But when having a switch
statement, the compiler (rightly) asks us to handle them. Drop them.
| -rw-r--r-- | src/core/dhcp/nm-dhcp-client.c | 3 | ||||
| -rw-r--r-- | src/core/dhcp/nm-dhcp-client.h | 2 |
2 files changed, 1 insertions, 4 deletions
diff --git a/src/core/dhcp/nm-dhcp-client.c b/src/core/dhcp/nm-dhcp-client.c index aaefef01ca..9d78887bed 100644 --- a/src/core/dhcp/nm-dhcp-client.c +++ b/src/core/dhcp/nm-dhcp-client.c @@ -352,8 +352,7 @@ NM_UTILS_LOOKUP_STR_DEFINE(nm_dhcp_state_to_string, NM_UTILS_LOOKUP_STR_ITEM(NM_DHCP_STATE_NOOP, "noop"), NM_UTILS_LOOKUP_STR_ITEM(NM_DHCP_STATE_TERMINATED, "terminated"), NM_UTILS_LOOKUP_STR_ITEM(NM_DHCP_STATE_TIMEOUT, "timeout"), - NM_UTILS_LOOKUP_STR_ITEM(NM_DHCP_STATE_UNKNOWN, "unknown"), - NM_UTILS_LOOKUP_ITEM_IGNORE(__NM_DHCP_STATE_MAX), ); + NM_UTILS_LOOKUP_STR_ITEM(NM_DHCP_STATE_UNKNOWN, "unknown"), ); static NMDhcpState reason_to_state(NMDhcpClient *self, const char *iface, const char *reason) diff --git a/src/core/dhcp/nm-dhcp-client.h b/src/core/dhcp/nm-dhcp-client.h index 601225ecd6..525bd1b655 100644 --- a/src/core/dhcp/nm-dhcp-client.h +++ b/src/core/dhcp/nm-dhcp-client.h @@ -57,8 +57,6 @@ typedef enum { NM_DHCP_STATE_FAIL, /* failed for some reason */ NM_DHCP_STATE_TERMINATED, /* client is no longer running */ NM_DHCP_STATE_NOOP, /* state is a non operation for NetworkManager */ - __NM_DHCP_STATE_MAX, - NM_DHCP_STATE_MAX = __NM_DHCP_STATE_MAX - 1, } NMDhcpState; const char *nm_dhcp_state_to_string(NMDhcpState state); |