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

Re: [libvirt] [PATCH 3/6] maint: use consistent if-else braces in xen and friends



On 09/03/14 23:25, Eric Blake wrote:
> I'm about to add a syntax check that enforces our documented
> HACKING style of always using matching {} on if-else statements.
> 
> This patch focuses on code related to xen.
> 
> * src/libxl/libxl_conf.c (libxlCapsInitGuests)
> (libxlMakeDomBuildInfo): Correct use of {}.
> * src/xen/xen_hypervisor.c (virXen_getvcpusinfo)
> (xenHypervisorMakeCapabilitiesInternal): Likewise.
> * src/xen/xend_internal.c (xenDaemonOpen)
> (xenDaemonDomainMigratePerform, xend_detect_config_version)
> (xenDaemonDetachDeviceFlags, xenDaemonDomainMigratePerform)
> (xenDaemonDomainBlockPeek): Likewise.
> * src/xenapi/xenapi_driver.c (xenapiConnectListDomains)
> (xenapiDomainLookupByUUID, xenapiDomainGetOSType): Likewise.
> * src/xenconfig/xen_common.c (xenParseCPUFeatures, xenFormatNet):
> Likewise.
> * src/xenconfig/xen_sxpr.c (xenParseSxpr, xenFormatSxprNet)
> (xenFormatSxpr): Likewise.
> 
> Signed-off-by: Eric Blake <eblake redhat com>
> ---
>  src/libxl/libxl_conf.c     | 12 ++++--------
>  src/xen/xen_hypervisor.c   | 18 +++++++++---------
>  src/xen/xend_internal.c    | 20 ++++++++------------
>  src/xenapi/xenapi_driver.c | 13 ++++++++-----
>  src/xenconfig/xen_common.c |  8 +++-----
>  src/xenconfig/xen_sxpr.c   | 18 +++++++++---------
>  6 files changed, 41 insertions(+), 48 deletions(-)
> 

ACK,

Peter


Attachment: signature.asc
Description: OpenPGP digital signature


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