diff options
author | Dan Winship <danw@gnome.org> | 2013-03-07 07:30:23 -0500 |
---|---|---|
committer | Dan Winship <danw@gnome.org> | 2013-03-07 07:32:27 -0500 |
commit | 6f44b7f3c6d406a1bda23056c7cad3ded390c02f (patch) | |
tree | 7261abacc27190136cc5a35983e8c8f4edfce998 /src/nm-netlink-monitor.c | |
parent | 0006c6aa1d6d5375c19d98745c5f3514bf0bfc03 (diff) | |
download | NetworkManager-6f44b7f3c6d406a1bda23056c7cad3ded390c02f.tar.gz |
all: remove redundant return-if-fail checks
NM_IS_FOO(x) returns FALSE if x is NULL, so we don't need a separate
(x != NULL) check before it.
Diffstat (limited to 'src/nm-netlink-monitor.c')
-rw-r--r-- | src/nm-netlink-monitor.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/nm-netlink-monitor.c b/src/nm-netlink-monitor.c index 65d038d481..a8824a9fe6 100644 --- a/src/nm-netlink-monitor.c +++ b/src/nm-netlink-monitor.c @@ -471,7 +471,6 @@ error: static gboolean open_connection (NMNetlinkMonitor *self, GError **error) { - g_return_val_if_fail (self != NULL, FALSE); g_return_val_if_fail (NM_IS_NETLINK_MONITOR (self), FALSE); if (!event_connection_setup (self, error)) @@ -558,7 +557,6 @@ nm_netlink_monitor_request_ip6_info (NMNetlinkMonitor *self, GError **error) { NMNetlinkMonitorPrivate *priv; - g_return_val_if_fail (self != NULL, FALSE); g_return_val_if_fail (NM_IS_NETLINK_MONITOR (self), FALSE); priv = NM_NETLINK_MONITOR_GET_PRIVATE (self); @@ -635,7 +633,6 @@ nm_netlink_monitor_get_flags_sync (NMNetlinkMonitor *self, struct rtnl_link *filter; int err; - g_return_val_if_fail (self != NULL, FALSE); g_return_val_if_fail (NM_IS_NETLINK_MONITOR (self), FALSE); g_return_val_if_fail (ifflags != NULL, FALSE); |