fdo-mirrors/NetworkManager
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git synced 2026-02-28 07:30:37 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

dhcp: minor cleanup of accept/decline functions in "nm-dhcp-client.c"

Browse source 
- assign the result of NM_DHCP_CLIENT_GET_CLASS() to a local variable.
  It feels nicer to only call the macro once. Of course, the macro
  expands to plain pointer dereferences, so there is little difference
  in terms of executed code.

- handle the default case with no virtual function first.

(cherry picked from commit 0f6df633fa)
This commit is contained in:
Thomas Haller 2022-05-24 09:08:07 +02:00 • committed by Beniamino Galvani
parent c9bd7cfc5d
commit 6a2c459d86
1 changed files with 18 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
src/core/dhcp/nm-dhcp-client.c
View file

@ -493,32 +493,34 @@ nm_dhcp_client_stop_watch_child(NMDhcpClient *self, pid_t pid)
static gboolean
_dhcp_client_accept(NMDhcpClient *self, const NML3ConfigData *l3cd, GError **error)
{
NMDhcpClientPrivate *priv;
NMDhcpClientClass *klass;
g_return_val_if_fail(NM_IS_DHCP_CLIENT(self), FALSE);
nm_assert(l3cd);
priv = NM_DHCP_CLIENT_GET_PRIVATE(self);
klass = NM_DHCP_CLIENT_GET_CLASS(self);
g_return_val_if_fail(priv->l3cd, FALSE);
g_return_val_if_fail(NM_DHCP_CLIENT_GET_PRIVATE(self)->l3cd, FALSE);
if (NM_DHCP_CLIENT_GET_CLASS(self)->accept) {
return NM_DHCP_CLIENT_GET_CLASS(self)->accept(self, l3cd, error);
}
if (!klass->accept)
return TRUE;
return TRUE;
return klass->accept(self, l3cd, error);
}
static gboolean
_dhcp_client_can_accept(NMDhcpClient *self)
{
gboolean can_accept;
NMDhcpClientClass *klass;
g_return_val_if_fail(NM_IS_DHCP_CLIENT(self), FALSE);
nm_assert(NM_IS_DHCP_CLIENT(self));
can_accept = !!(NM_DHCP_CLIENT_GET_CLASS(self)->accept);
klass = NM_DHCP_CLIENT_GET_CLASS(self);
nm_assert(can_accept == (!!(NM_DHCP_CLIENT_GET_CLASS(self)->decline)));
can_accept = !!klass->accept;
nm_assert(can_accept == (!!klass->decline));
return can_accept;
}
@ -529,20 +531,19 @@ _dhcp_client_decline(NMDhcpClient *self,
const char *error_message,
GError **error)
{
NMDhcpClientPrivate *priv;
NMDhcpClientClass *klass;
g_return_val_if_fail(NM_IS_DHCP_CLIENT(self), FALSE);
nm_assert(l3cd);
priv = NM_DHCP_CLIENT_GET_PRIVATE(self);
klass = NM_DHCP_CLIENT_GET_CLASS(self);
g_return_val_if_fail(priv->l3cd, FALSE);
g_return_val_if_fail(NM_DHCP_CLIENT_GET_PRIVATE(self)->l3cd, FALSE);
if (NM_DHCP_CLIENT_GET_CLASS(self)->decline) {
return NM_DHCP_CLIENT_GET_CLASS(self)->decline(self, l3cd, error_message, error);
}
if (!klass->decline)
return TRUE;
return TRUE;
return klass->decline(self, l3cd, error_message, error);
}
static GBytes *