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

Peter Krempa pkrempa at redhat.com
Thu Sep 4 12:10:16 UTC 2014


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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140904/c99e99e5/attachment-0001.sig>


More information about the libvir-list mailing list