summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2014-11-11 10:14:30 +0100
committerThomas Haller <thaller@redhat.com>2014-11-19 22:55:32 +0100
commit393f213c3bad9935e25652dade02237578562f17 (patch)
tree5075e3c7a5b50e1e6a2dd92f3adce74f32ca89b3
parent825885d5b12300ada94826e75de254702b730e30 (diff)
downloadNetworkManager-393f213c3bad9935e25652dade02237578562f17.tar.gz
policy: remove unused @do_sync argument from NMDefaultRouteManager functions
Signed-off-by: Thomas Haller <thaller@redhat.com>
-rw-r--r--src/nm-default-route-manager.c30
1 files changed, 14 insertions, 16 deletions
diff --git a/src/nm-default-route-manager.c b/src/nm-default-route-manager.c
index 9ebf138e92..cc2c6daee7 100644
--- a/src/nm-default-route-manager.c
+++ b/src/nm-default-route-manager.c
@@ -293,7 +293,7 @@ _sort_entries_cmp (gconstpointer a, gconstpointer b, gpointer user_data)
}
static void
-_resync_all (const VTableIP *vtable, NMDefaultRouteManager *self, const Entry *changed_entry, const Entry *old_entry, gboolean do_sync)
+_resync_all (const VTableIP *vtable, NMDefaultRouteManager *self, const Entry *changed_entry, const Entry *old_entry)
{
NMDefaultRouteManagerPrivate *priv = NM_DEFAULT_ROUTE_MANAGER_GET_PRIVATE (self);
Entry *entry;
@@ -354,18 +354,16 @@ _resync_all (const VTableIP *vtable, NMDefaultRouteManager *self, const Entry *c
last_metric = expected_metric;
}
- if (do_sync) {
- g_hash_table_iter_init (&iter, changed_metrics);
- while (g_hash_table_iter_next (&iter, &ptr, NULL))
- _platform_route_sync_add (vtable, self, GPOINTER_TO_UINT (ptr));
- _platform_route_sync_flush (vtable, self);
- }
+ g_hash_table_iter_init (&iter, changed_metrics);
+ while (g_hash_table_iter_next (&iter, &ptr, NULL))
+ _platform_route_sync_add (vtable, self, GPOINTER_TO_UINT (ptr));
+ _platform_route_sync_flush (vtable, self);
g_hash_table_unref (changed_metrics);
}
static void
-_entry_at_idx_update (const VTableIP *vtable, NMDefaultRouteManager *self, guint entry_idx, const Entry *old_entry, gboolean do_sync)
+_entry_at_idx_update (const VTableIP *vtable, NMDefaultRouteManager *self, guint entry_idx, const Entry *old_entry)
{
NMDefaultRouteManagerPrivate *priv = NM_DEFAULT_ROUTE_MANAGER_GET_PRIVATE (self);
Entry *entry;
@@ -390,11 +388,11 @@ _entry_at_idx_update (const VTableIP *vtable, NMDefaultRouteManager *self, guint
g_ptr_array_sort_with_data (entries, _sort_entries_cmp, NULL);
- _resync_all (vtable, self, entry, old_entry, do_sync);
+ _resync_all (vtable, self, entry, old_entry);
}
static void
-_entry_at_idx_remove (const VTableIP *vtable, NMDefaultRouteManager *self, guint entry_idx, gboolean do_sync)
+_entry_at_idx_remove (const VTableIP *vtable, NMDefaultRouteManager *self, guint entry_idx)
{
NMDefaultRouteManagerPrivate *priv = NM_DEFAULT_ROUTE_MANAGER_GET_PRIVATE (self);
Entry *entry;
@@ -414,7 +412,7 @@ _entry_at_idx_remove (const VTableIP *vtable, NMDefaultRouteManager *self, guint
g_ptr_array_index (entries, entry_idx) = NULL;
g_ptr_array_remove_index (entries, entry_idx);
- _resync_all (vtable, self, NULL, entry, do_sync);
+ _resync_all (vtable, self, NULL, entry);
_entry_free (entry);
}
@@ -470,7 +468,7 @@ _ipx_update_default_route (const VTableIP *vtable, NMDefaultRouteManager *self,
entry->route.rx.ifindex, ip_ifindex);
g_object_freeze_notify (G_OBJECT (self));
- _entry_at_idx_remove (vtable, self, entry_idx, FALSE);
+ _entry_at_idx_remove (vtable, self, entry_idx);
g_assert (!_entry_find_by_source (entries, source, NULL));
_ipx_update_default_route (vtable, self, source);
g_object_thaw_notify (G_OBJECT (self));
@@ -549,7 +547,7 @@ _ipx_update_default_route (const VTableIP *vtable, NMDefaultRouteManager *self,
entry->synced = synced;
g_ptr_array_add (entries, entry);
- _entry_at_idx_update (vtable, self, entries->len - 1, NULL, TRUE);
+ _entry_at_idx_update (vtable, self, entries->len - 1, NULL);
} else if (default_route) {
/* update */
Entry old_entry, new_entry;
@@ -570,10 +568,10 @@ _ipx_update_default_route (const VTableIP *vtable, NMDefaultRouteManager *self,
old_entry = *entry;
*entry = new_entry;
- _entry_at_idx_update (vtable, self, entry_idx, &old_entry, TRUE);
+ _entry_at_idx_update (vtable, self, entry_idx, &old_entry);
} else {
/* delete */
- _entry_at_idx_remove (vtable, self, entry_idx, TRUE);
+ _entry_at_idx_remove (vtable, self, entry_idx);
}
}
@@ -603,7 +601,7 @@ _ipx_remove_default_route (const VTableIP *vtable, NMDefaultRouteManager *self,
priv = NM_DEFAULT_ROUTE_MANAGER_GET_PRIVATE (self);
if (_entry_find_by_source (vtable->get_entries (priv), source, &entry_idx))
- _entry_at_idx_remove (vtable, self, entry_idx, TRUE);
+ _entry_at_idx_remove (vtable, self, entry_idx);
}
void