[libvirt-php PATCH 2/2] Add optional flags parameter to domain_create

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Instead of adding another function libvirt_domain_create_with_flags
simply add an optional argument to libvirt_domain_create which defaults
to VIR_DOMAIN_NONE and call virDomainCreateWithFlags internally instead.
---
 src/libvirt-php.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/libvirt-php.c b/src/libvirt-php.c
index 834eff8..c57b107 100644
--- a/src/libvirt-php.c
+++ b/src/libvirt-php.c
@@ -4717,18 +4717,20 @@ PHP_FUNCTION(libvirt_domain_get_info)
 	Since version:	0.4.1(-1)
 	Description:	Function is used to create the domain identified by it's resource
 	Arguments:	@res [resource]: libvirt domain resource, e.g. from libvirt_domain_lookup_by_*()
+			@flags [int]: Optional flags to create the domain. Default: VIR_DOMAIN_NONE (0)
 	Returns:	result of domain creation (startup)
 */
 PHP_FUNCTION(libvirt_domain_create)
 {
 	php_libvirt_domain *domain=NULL;
 	zval *zdomain;
+	long flags=0;
 	int retval;
 
-	GET_DOMAIN_FROM_ARGS("r",&zdomain);
+	GET_DOMAIN_FROM_ARGS("r|l",&zdomain,&flags);
 
-	retval=virDomainCreate(domain->domain);
-	DPRINTF("%s: virDomainCreate(%p) returned %d\n", PHPFUNC, domain->domain, retval);
+	retval=virDomainCreateWithFlags(domain->domain, flags);
+	DPRINTF("%s: virDomainCreateWithFlags(%p, %ld) returned %d\n", PHPFUNC, domain->domain, flags, retval);
 	if (retval != 0) RETURN_FALSE;
 	RETURN_TRUE;
 }
-- 
1.8.2.1

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]