[libvirt] [PATCH] Fix virsh.c compilation warning

A call to vshError() containing 'doexit' parameter sneaked in after I removed said parameter. Regards, Jim

On Mon, Oct 19, 2009 at 02:03:21PM -0600, Jim Fehlig wrote:
A call to vshError() containing 'doexit' parameter sneaked in after I removed said parameter.
Regards, Jim
commit edea3dfdd861d5eee4712da43781908f0fa2a6d5 Author: Jim Fehlig <jfehlig@novell.com> Date: Mon Oct 19 13:53:40 2009 -0600
Remove extra arg in call to vshError()
diff --git a/tools/virsh.c b/tools/virsh.c index 47122d5..6b93405 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -2518,7 +2518,7 @@ cmdMigrate (vshControl *ctl, const vshCmd *cmd) * a libvirt URI, or a hypervisor specific URI. */
if (migrateuri != NULL) { - vshError(ctl, FALSE, "%s", _("migrate: Unexpected migrateuri for peer2peer/direct migration")); + vshError(ctl, "%s", _("migrate: Unexpected migrateuri for peer2peer/direct migration")); goto done; }
ACK, i missed this when i rebased Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

On Mon, Oct 19, 2009 at 02:03:21PM -0600, Jim Fehlig wrote:
A call to vshError() containing 'doexit' parameter sneaked in after I removed said parameter.
Regards, Jim
commit edea3dfdd861d5eee4712da43781908f0fa2a6d5 Author: Jim Fehlig <jfehlig@novell.com> Date: Mon Oct 19 13:53:40 2009 -0600
Remove extra arg in call to vshError()
diff --git a/tools/virsh.c b/tools/virsh.c index 47122d5..6b93405 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -2518,7 +2518,7 @@ cmdMigrate (vshControl *ctl, const vshCmd *cmd) * a libvirt URI, or a hypervisor specific URI. */
if (migrateuri != NULL) { - vshError(ctl, FALSE, "%s", _("migrate: Unexpected migrateuri for peer2peer/direct migration")); + vshError(ctl, "%s", _("migrate: Unexpected migrateuri for peer2peer/direct migration")); goto done; }
Okidoc, pushed ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/
participants (3)
-
Daniel P. Berrange
-
Daniel Veillard
-
Jim Fehlig