[libvirt] [PATCH 2/4] nwfilter: Alter @aclfilter to just @filter

John Ferlan jferlan at redhat.com
Wed Aug 16 13:38:58 UTC 2017


Rename the variable, recent review requested just use of @filter,
so be consistent throughout.

NB: Also change the virNWFilterPtr to be @nwfilter to not conflict
with the renamed variable.

Signed-off-by: John Ferlan <jferlan at redhat.com>
---
 src/conf/virnwfilterobj.c | 18 +++++++++---------
 src/conf/virnwfilterobj.h |  6 +++---
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/src/conf/virnwfilterobj.c b/src/conf/virnwfilterobj.c
index 93072be..408e575 100644
--- a/src/conf/virnwfilterobj.c
+++ b/src/conf/virnwfilterobj.c
@@ -398,7 +398,7 @@ virNWFilterObjListAssignDef(virNWFilterObjListPtr nwfilters,
 int
 virNWFilterObjListNumOfNWFilters(virNWFilterObjListPtr nwfilters,
                                  virConnectPtr conn,
-                                 virNWFilterObjListFilter aclfilter)
+                                 virNWFilterObjListFilter filter)
 {
     size_t i;
     int nfilters = 0;
@@ -406,7 +406,7 @@ virNWFilterObjListNumOfNWFilters(virNWFilterObjListPtr nwfilters,
     for (i = 0; i < nwfilters->count; i++) {
         virNWFilterObjPtr obj = nwfilters->objs[i];
         virNWFilterObjLock(obj);
-        if (!aclfilter || aclfilter(conn, obj->def))
+        if (!filter || filter(conn, obj->def))
             nfilters++;
         virNWFilterObjUnlock(obj);
     }
@@ -418,7 +418,7 @@ virNWFilterObjListNumOfNWFilters(virNWFilterObjListPtr nwfilters,
 int
 virNWFilterObjListGetNames(virNWFilterObjListPtr nwfilters,
                            virConnectPtr conn,
-                           virNWFilterObjListFilter aclfilter,
+                           virNWFilterObjListFilter filter,
                            char **const names,
                            int maxnames)
 {
@@ -430,7 +430,7 @@ virNWFilterObjListGetNames(virNWFilterObjListPtr nwfilters,
         virNWFilterObjPtr obj = nwfilters->objs[i];
         virNWFilterObjLock(obj);
         def = obj->def;
-        if (!aclfilter || aclfilter(conn, def)) {
+        if (!filter || filter(conn, def)) {
             if (VIR_STRDUP(names[nnames], def->name) < 0) {
                 virNWFilterObjUnlock(obj);
                 goto failure;
@@ -454,11 +454,11 @@ int
 virNWFilterObjListExport(virConnectPtr conn,
                          virNWFilterObjListPtr nwfilters,
                          virNWFilterPtr **filters,
-                         virNWFilterObjListFilter aclfilter)
+                         virNWFilterObjListFilter filter)
 {
     virNWFilterPtr *tmp_filters = NULL;
     int nfilters = 0;
-    virNWFilterPtr filter = NULL;
+    virNWFilterPtr nwfilter = NULL;
     virNWFilterObjPtr obj = NULL;
     virNWFilterDefPtr def;
     size_t i;
@@ -476,12 +476,12 @@ virNWFilterObjListExport(virConnectPtr conn,
         obj = nwfilters->objs[i];
         virNWFilterObjLock(obj);
         def = obj->def;
-        if (!aclfilter || aclfilter(conn, def)) {
-            if (!(filter = virGetNWFilter(conn, def->name, def->uuid))) {
+        if (!filter || filter(conn, def)) {
+            if (!(nwfilter = virGetNWFilter(conn, def->name, def->uuid))) {
                 virNWFilterObjUnlock(obj);
                 goto cleanup;
             }
-            tmp_filters[nfilters++] = filter;
+            tmp_filters[nfilters++] = nwfilter;
         }
         virNWFilterObjUnlock(obj);
     }
diff --git a/src/conf/virnwfilterobj.h b/src/conf/virnwfilterobj.h
index 509e8dc..8e79518 100644
--- a/src/conf/virnwfilterobj.h
+++ b/src/conf/virnwfilterobj.h
@@ -86,12 +86,12 @@ typedef bool
 int
 virNWFilterObjListNumOfNWFilters(virNWFilterObjListPtr nwfilters,
                                  virConnectPtr conn,
-                                 virNWFilterObjListFilter aclfilter);
+                                 virNWFilterObjListFilter filter);
 
 int
 virNWFilterObjListGetNames(virNWFilterObjListPtr nwfilters,
                            virConnectPtr conn,
-                           virNWFilterObjListFilter aclfilter,
+                           virNWFilterObjListFilter filter,
                            char **const names,
                            int maxnames);
 
@@ -99,7 +99,7 @@ int
 virNWFilterObjListExport(virConnectPtr conn,
                          virNWFilterObjListPtr nwfilters,
                          virNWFilterPtr **filters,
-                         virNWFilterObjListFilter aclfilter);
+                         virNWFilterObjListFilter filter);
 
 int
 virNWFilterObjListLoadAllConfigs(virNWFilterObjListPtr nwfilters,
-- 
2.9.4




More information about the libvir-list mailing list