Re: [PATCH 1/1] Correct indent errors in the function qemuDomainNetsRestart

[Alex Jia <ajia@xxxxxxxxxx>]

On 05/03/2012 01:53 PM, Li Zhang wrote:
> There are three spapces for a indent every line in the
> function qemuDomainNetsRestart.
Yeah, the codes exist indentation issues, in addition, also need to 
align function
arguments in 'qemuDomainNetsRestart'():

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 86e82d6..e2bad02 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -417,8 +417,8 @@ cleanup:


 static void qemuDomainNetsRestart(void *payload,
-                                const void *name ATTRIBUTE_UNUSED,
-                                void *data ATTRIBUTE_UNUSED)
+                                  const void *name ATTRIBUTE_UNUSED,
+                                  void *data ATTRIBUTE_UNUSED)

> This patch is to correct it.
>
> Signed-off-by: Li Zhang<zhlcindy@xxxxxxxxxxxxxxxxxx>
> ---
>   src/qemu/qemu_driver.c |   42 +++++++++++++++++++++---------------------
>   1 file changed, 21 insertions(+), 21 deletions(-)
>
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 86e82d6..bed8edb 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -420,28 +420,28 @@ static void qemuDomainNetsRestart(void *payload,
>                                   const void *name ATTRIBUTE_UNUSED,
>                                   void *data ATTRIBUTE_UNUSED)
>   {
> -   int i;
> -   virDomainObjPtr vm = (virDomainObjPtr)payload;
> -   virDomainDefPtr def = vm->def;
> -
> -   virDomainObjLock(vm);
> -
> -   for (i = 0; i<  def->nnets; i++) {
> -       virDomainNetDefPtr net = def->nets[i];
> -       if (virDomainNetGetActualType(net) == VIR_DOMAIN_NET_TYPE_DIRECT&&
> -           virDomainNetGetActualDirectMode(net) == VIR_NETDEV_MACVLAN_MODE_VEPA) {
> -           VIR_DEBUG("VEPA mode device %s active in domain %s. Reassociating.",
> -                     net->ifname, def->name);
> -           ignore_value(virNetDevMacVLanRestartWithVPortProfile(net->ifname,
> -                                                                net->mac,
> -                                                                virDomainNetGetActualDirectDev(net),
> -                                                                def->uuid,
> -                                                                virDomainNetGetActualVirtPortProfile(net),
> -                                                                VIR_NETDEV_VPORT_PROFILE_OP_CREATE));
> -       }
> -   }
> +    int i;
> +    virDomainObjPtr vm = (virDomainObjPtr)payload;
> +    virDomainDefPtr def = vm->def;
> +
> +    virDomainObjLock(vm);
>
> -   virDomainObjUnlock(vm);
> +    for (i = 0; i<  def->nnets; i++) {
> +        virDomainNetDefPtr net = def->nets[i];
> +        if (virDomainNetGetActualType(net) == VIR_DOMAIN_NET_TYPE_DIRECT&&
> +            virDomainNetGetActualDirectMode(net) == VIR_NETDEV_MACVLAN_MODE_VEPA) {
> +            VIR_DEBUG("VEPA mode device %s active in domain %s. Reassociating.",
> +                      net->ifname, def->name);
> +            ignore_value(virNetDevMacVLanRestartWithVPortProfile(net->ifname,
> +                                                                 net->mac,
> +                                                                 virDomainNetGetActualDirectDev(net),
> +                                                                 def->uuid,
> +                                                                 virDomainNetGetActualVirtPortProfile(net),
> +                                                                 VIR_NETDEV_VPORT_PROFILE_OP_CREATE));
> +        }
> +    }
> +
> +    virDomainObjUnlock(vm);
>   }
>
>   /**

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