Re: [PATCH v3 3/4] qemu: remove pointless condition

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

 



On Tue, Feb 11, 2014 at 12:13:43PM +0100, Christophe Fergeau wrote:
> 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).
>

Oh, '-w' is the magic one, I could've used that, I also wanted to use
'-C -C' for the next patch, but _somehow_ failed to do that.  Will do
next time, sorry.

Martin

> Christophe
>
> >
> > Signed-off-by: Martin Kletzander <mkletzan@xxxxxxxxxx>
> > ---
> >  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@xxxxxxxxxx
> > https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: Digital signature

--
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]