Re: [PATCH] virsh: Define multi line macros properly

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

 



On Tue, 2017-10-31 at 12:39 +0100, Michal Privoznik wrote:
> @@ -781,7 +781,7 @@ static const vshCmdOptDef opts_attach_interface[] = {
>   * 'average,peak,burst,floor', in which peak and burst are optional,
>   * thus 'average,,burst' and 'average,peak' are also legal. */
>  
> -#define VIRSH_PARSE_RATE_FIELD(index, name)                                      \
> +#define VIRSH_PARSE_RATE_FIELD(index, name)                                    \
>      do {                                                                       \
>          if (index < ntok &&                                                    \
>              *tok[index] != '\0' &&                                             \
[...]
> @@ -9096,7 +9096,7 @@ cmdMemtune(vshControl *ctl, const vshCmd *cmd)
>          return false;
>  
>  #define PARSE_MEMTUNE_PARAM(NAME, FIELD)                                    \
> -    if ((rc = virshMemtuneGetSize(ctl, cmd, NAME, &tmpVal)) < 0) {            \
> +    if ((rc = virshMemtuneGetSize(ctl, cmd, NAME, &tmpVal)) < 0) {          \
>          vshError(ctl, _("Unable to parse integer parameter %s"), NAME);     \
>          goto cleanup;                                                       \
>      }                                                                       \

As an aside, I'd really like if we could stop trying to be cute by
right-aligning backslashes, thus making silly adjustments like the
above wholly unnecessary.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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]
  Powered by Linux