summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorDan Williams <dcbw@redhat.com>2014-09-05 13:15:44 -0500
committerDan Williams <dcbw@redhat.com>2014-09-11 12:40:08 -0500
commit56e2915536f6958f9103e080539240f9e258ea20 (patch)
tree246a6aa078b46054f41401faeec4c7b34449d7c0 /src
parent3382d3e1a2d5bf893ad5721eedad3a554e04f995 (diff)
downloadNetworkManager-56e2915536f6958f9103e080539240f9e258ea20.tar.gz
trivial: spacing and code cleanups in nm-manager.h
Remove some dead code and a redundant function.
Diffstat (limited to 'src')
-rw-r--r--src/nm-manager.c39
-rw-r--r--src/nm-manager.h57
2 files changed, 29 insertions, 67 deletions
diff --git a/src/nm-manager.c b/src/nm-manager.c
index bee2c55f11..53cf4c26cb 100644
--- a/src/nm-manager.c
+++ b/src/nm-manager.c
@@ -497,24 +497,6 @@ nm_manager_get_device_by_path (NMManager *manager, const char *path)
}
NMDevice *
-nm_manager_get_device_by_master (NMManager *manager, const char *master, const char *driver)
-{
- GSList *iter;
-
- g_return_val_if_fail (master != NULL, NULL);
-
- for (iter = NM_MANAGER_GET_PRIVATE (manager)->devices; iter; iter = iter->next) {
- NMDevice *device = NM_DEVICE (iter->data);
-
- if (!strcmp (nm_device_get_iface (device), master) &&
- (!driver || !strcmp (nm_device_get_driver (device), driver)))
- return device;
- }
-
- return NULL;
-}
-
-NMDevice *
nm_manager_get_device_by_ifindex (NMManager *manager, int ifindex)
{
GSList *iter;
@@ -1864,20 +1846,7 @@ find_device_by_ip_iface (NMManager *self, const gchar *iface)
return NULL;
}
-static NMDevice *
-find_device_by_ifindex (NMManager *self, guint32 ifindex)
-{
- NMManagerPrivate *priv = NM_MANAGER_GET_PRIVATE (self);
- GSList *iter;
-
- for (iter = priv->devices; iter; iter = g_slist_next (iter)) {
- NMDevice *candidate = NM_DEVICE (iter->data);
-
- if (ifindex == nm_device_get_ifindex (candidate))
- return candidate;
- }
- return NULL;
-}
+/*******************************************************************/
static void
factory_device_added_cb (NMDeviceFactory *factory,
@@ -2105,7 +2074,7 @@ platform_link_added (NMManager *self,
if (priv->ignore_link_added_cb > 0)
return;
- if (find_device_by_ifindex (self, ifindex))
+ if (nm_manager_get_device_by_ifindex (self, ifindex))
return;
/* Try registered device factories */
@@ -2154,7 +2123,7 @@ platform_link_added (NMManager *self,
case NM_LINK_TYPE_VLAN:
/* Have to find the parent device */
if (nm_platform_vlan_get_info (ifindex, &parent_ifindex, NULL)) {
- parent = find_device_by_ifindex (self, parent_ifindex);
+ parent = nm_manager_get_device_by_ifindex (self, parent_ifindex);
if (parent)
device = nm_device_vlan_new (plink, parent);
else {
@@ -2228,7 +2197,7 @@ platform_link_cb (NMPlatform *platform,
NMManager *self = NM_MANAGER (user_data);
NMDevice *device;
- device = find_device_by_ifindex (self, ifindex);
+ device = nm_manager_get_device_by_ifindex (self, ifindex);
if (device)
remove_device (self, device, FALSE);
break;
diff --git a/src/nm-manager.h b/src/nm-manager.h
index 2c6c9a04d3..55b0b3fc9f 100644
--- a/src/nm-manager.h
+++ b/src/nm-manager.h
@@ -92,45 +92,38 @@ typedef struct {
GType nm_manager_get_type (void);
/* nm_manager_new() should only be used by main.c */
-NMManager *nm_manager_new (NMSettings *settings,
- const char *state_file,
- gboolean initial_net_enabled,
- gboolean initial_wifi_enabled,
- gboolean initial_wwan_enabled,
- gboolean initial_wimax_enabled,
- GError **error);
+NMManager * nm_manager_new (NMSettings *settings,
+ const char *state_file,
+ gboolean initial_net_enabled,
+ gboolean initial_wifi_enabled,
+ gboolean initial_wwan_enabled,
+ gboolean initial_wimax_enabled,
+ GError **error);
-NMManager *nm_manager_get (void);
+NMManager * nm_manager_get (void);
-void nm_manager_start (NMManager *manager);
-
-const GSList *nm_manager_get_active_connections (NMManager *manager);
-GSList *nm_manager_get_activatable_connections (NMManager *manager);
+void nm_manager_start (NMManager *manager);
+NMState nm_manager_get_state (NMManager *manager);
+const GSList *nm_manager_get_active_connections (NMManager *manager);
+GSList * nm_manager_get_activatable_connections (NMManager *manager);
/* Device handling */
-const GSList *nm_manager_get_devices (NMManager *manager);
-
-NMDevice *nm_manager_get_device_by_master (NMManager *manager,
- const char *master,
- const char *driver);
-NMDevice *nm_manager_get_device_by_ifindex (NMManager *manager,
- int ifindex);
-
-NMActiveConnection *nm_manager_activate_connection (NMManager *manager,
- NMConnection *connection,
- const char *specific_object,
- NMDevice *device,
- NMAuthSubject *subject,
- GError **error);
+const GSList * nm_manager_get_devices (NMManager *manager);
-gboolean nm_manager_deactivate_connection (NMManager *manager,
- const char *connection_path,
- NMDeviceStateReason reason,
- GError **error);
+NMDevice * nm_manager_get_device_by_ifindex (NMManager *manager,
+ int ifindex);
-/* State handling */
+NMActiveConnection *nm_manager_activate_connection (NMManager *manager,
+ NMConnection *connection,
+ const char *specific_object,
+ NMDevice *device,
+ NMAuthSubject *subject,
+ GError **error);
-NMState nm_manager_get_state (NMManager *manager);
+gboolean nm_manager_deactivate_connection (NMManager *manager,
+ const char *connection_path,
+ NMDeviceStateReason reason,
+ GError **error);
#endif /* __NETWORKMANAGER_MANAGER_H__ */