conf: networkportdef: Provide only virNetworkPortDefParse

Replace the two helpers virNetworkPortDefParseString/File with the
common helper.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index b3202c9f7671f1dc723e153aa55a20e14f3052c5..adbd97e632751023e2864afdef9b806e8ab3ab55 100644 (file)
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -29537,7 +29537,7 @@ virDomainNetCreatePort(virConnectPtr conn,
     VIR_FREE(portxml);
 
     if (!(portxml = virNetworkPortGetXMLDesc(port, 0)) ||
-        !(portdef = virNetworkPortDefParseString(portxml, 0)) ||
+        !(portdef = virNetworkPortDefParse(portxml, NULL, 0)) ||
         virDomainNetDefActualFromNetworkPort(iface, portdef) < 0) {
         virErrorPreserveLast(&save_err);
         virNetworkPortDelete(port, 0);

diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
index b4c61fff857dea4ee78c0d34bddc955502e17546..7621fa338080280f474c1e5eaae145168503b901 100644 (file)
--- a/src/conf/virnetworkobj.c
+++ b/src/conf/virnetworkobj.c
@@ -1833,7 +1833,7 @@ virNetworkObjLoadAllPorts(virNetworkObj *net,
 
         file = g_strdup_printf("%s/%s.xml", dir, de->d_name);
 
-        portdef = virNetworkPortDefParseFile(file);
+        portdef = virNetworkPortDefParse(NULL, file, 0);
         if (!portdef) {
             VIR_WARN("Cannot parse port %s", file);
             continue;

diff --git a/src/conf/virnetworkportdef.c b/src/conf/virnetworkportdef.c
index 651e4c33295e65911e3f2307078db6930f549a53..035e3fe7586520e4ac76e57921dcea4e9143bff8 100644 (file)
--- a/src/conf/virnetworkportdef.c
+++ b/src/conf/virnetworkportdef.c
@@ -251,7 +251,7 @@ virNetworkPortDefParseXML(xmlXPathContextPtr ctxt)
 }
 
 
-static virNetworkPortDef *
+virNetworkPortDef *
 virNetworkPortDefParse(const char *xmlStr,
                        const char *filename,
                        unsigned int flags)
@@ -268,21 +268,6 @@ virNetworkPortDefParse(const char *xmlStr,
 }
 
 
-virNetworkPortDef *
-virNetworkPortDefParseString(const char *xmlStr,
-                             unsigned int flags)
-{
-    return virNetworkPortDefParse(xmlStr, NULL, flags);
-}
-
-
-virNetworkPortDef *
-virNetworkPortDefParseFile(const char *filename)
-{
-    return virNetworkPortDefParse(NULL, filename, 0);
-}
-
-
 char *
 virNetworkPortDefFormat(const virNetworkPortDef *def)
 {

diff --git a/src/conf/virnetworkportdef.h b/src/conf/virnetworkportdef.h
index 908a7c87950f52c08401b2ba3a19e6ab576706d3..48e73dbefde9b9ab8b1befc476e8cb7377424726 100644 (file)
--- a/src/conf/virnetworkportdef.h
+++ b/src/conf/virnetworkportdef.h
@@ -81,11 +81,9 @@ virNetworkPortDefFree(virNetworkPortDef *port);
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(virNetworkPortDef, virNetworkPortDefFree);
 
 virNetworkPortDef *
-virNetworkPortDefParseString(const char *xml,
-                             unsigned int flags);
-
-virNetworkPortDef *
-virNetworkPortDefParseFile(const char *filename);
+virNetworkPortDefParse(const char *xmlStr,
+                       const char *filename,
+                       unsigned int flags);
 
 char *
 virNetworkPortDefFormat(const virNetworkPortDef *def);

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 903c4196df89bc008282468e566133aa0fc8c3db..3ef32101845886f0b11062d14f4b4fba1b1e99cb 100644 (file)
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -1326,8 +1326,7 @@ virNetworkObjUpdateAssignDef;
 virNetworkPortDefFormat;
 virNetworkPortDefFormatBuf;
 virNetworkPortDefFree;
-virNetworkPortDefParseFile;
-virNetworkPortDefParseString;
+virNetworkPortDefParse;
 virNetworkPortDefSaveStatus;
 
 

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 7c6430b4e3e373a69ff56e81069d8e4fccdedca4..e4f5e93779abd4ffd05dc3ed656818e702197dc7 100644 (file)
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -4844,7 +4844,7 @@ networkPortCreateXML(virNetworkPtr net,
 
     def = virNetworkObjGetDef(obj);
 
-    if (!(portdef = virNetworkPortDefParseString(xmldesc, flags)))
+    if (!(portdef = virNetworkPortDefParse(xmldesc, NULL, flags)))
         goto cleanup;
 
     if (virNetworkPortCreateXMLEnsureACL(net->conn, def, portdef) < 0)

diff --git a/tests/virnetworkportxml2xmltest.c b/tests/virnetworkportxml2xmltest.c
index 237624a7521546fef73145e030b48188341ed725..21e3c6f69781db0bdac2b3ecb871f75f32ce7211 100644 (file)
--- a/tests/virnetworkportxml2xmltest.c
+++ b/tests/virnetworkportxml2xmltest.c
@@ -38,7 +38,7 @@ testCompareXMLToXMLFiles(const char *expected)
     g_autofree char *actual = NULL;
     g_autoptr(virNetworkPortDef) dev = NULL;
 
-    if (!(dev = virNetworkPortDefParseFile(expected)))
+    if (!(dev = virNetworkPortDefParse(NULL, expected, 0)))
         return -1;
 
     if (!(actual = virNetworkPortDefFormat(dev)))