[libvirt] [PATCH v3 3/4] qemu: remove pointless condition
Christophe Fergeau
cfergeau at redhat.com
Tue Feb 11 11:13:43 UTC 2014
On Mon, Feb 10, 2014 at 03:03:06PM +0100, Martin Kletzander wrote:
> This patch is here just to ease the code review and make related
> changes look more sensible.
Hehe this description actually made things not as easy as they could have
;)
I would have specifically mentioned that if (def->nserials) is removed and
that the rest of the patch is just whitespace changes (output of git show
-w could even have been added to the log).
Christophe
>
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
> src/qemu/qemu_command.c | 44 +++++++++++++++++++++-----------------------
> 1 file changed, 21 insertions(+), 23 deletions(-)
>
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index d8a7951..6ff9fd5 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -8803,32 +8803,30 @@ qemuBuildCommandLine(virConnectPtr conn,
> virCommandAddArgBuffer(cmd, &opt);
> }
>
> - if (def->nserials) {
> - for (i = 0; i < def->nserials; i++) {
> - virDomainChrDefPtr serial = def->serials[i];
> - char *devstr;
> + for (i = 0; i < def->nserials; i++) {
> + virDomainChrDefPtr serial = def->serials[i];
> + char *devstr;
>
> - /* Use -chardev with -device if they are available */
> - if (virQEMUCapsSupportsChardev(def, qemuCaps, serial)) {
> - virCommandAddArg(cmd, "-chardev");
> - if (!(devstr = qemuBuildChrChardevStr(&serial->source,
> - serial->info.alias,
> - qemuCaps)))
> - goto error;
> - virCommandAddArg(cmd, devstr);
> - VIR_FREE(devstr);
> + /* Use -chardev with -device if they are available */
> + if (virQEMUCapsSupportsChardev(def, qemuCaps, serial)) {
> + virCommandAddArg(cmd, "-chardev");
> + if (!(devstr = qemuBuildChrChardevStr(&serial->source,
> + serial->info.alias,
> + qemuCaps)))
> + goto error;
> + virCommandAddArg(cmd, devstr);
> + VIR_FREE(devstr);
>
> - if (qemuBuildChrDeviceCommandLine(cmd, def, serial, qemuCaps) < 0)
> - goto error;
> - } else {
> - virCommandAddArg(cmd, "-serial");
> - if (!(devstr = qemuBuildChrArgStr(&serial->source, NULL)))
> - goto error;
> - virCommandAddArg(cmd, devstr);
> - VIR_FREE(devstr);
> - }
> - actualSerials++;
> + if (qemuBuildChrDeviceCommandLine(cmd, def, serial, qemuCaps) < 0)
> + goto error;
> + } else {
> + virCommandAddArg(cmd, "-serial");
> + if (!(devstr = qemuBuildChrArgStr(&serial->source, NULL)))
> + goto error;
> + virCommandAddArg(cmd, devstr);
> + VIR_FREE(devstr);
> }
> + actualSerials++;
> }
>
> /* If we have -device, then we set -nodefault already */
> --
> 1.8.5.4
>
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140211/4da5710e/attachment-0001.sig>
More information about the libvir-list
mailing list