[libvirt] [PATCH 4/6] maint: use consistent if-else braces in lxc, vbox, phyp

Peter Krempa pkrempa at redhat.com
Thu Sep 4 12:16:18 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 drivers that had several issues.
> 
> * src/lxc/lxc_fuse.c (lxcProcGetattr, lxcProcReadMeminfo): Correct
> use of {}.
> * src/lxc/lxc_driver.c (lxcDomainMergeBlkioDevice): Likewise.
> * src/phyp/phyp_driver.c (phypConnectNumOfDomainsGeneric)
> (phypUUIDTable_Init, openSSHSession, phypStoragePoolListVolumes)
> (phypConnectListStoragePools, phypDomainSetVcpusFlags)
> (phypStorageVolGetXMLDesc, phypStoragePoolGetXMLDesc)
> (phypConnectListDefinedDomains): Likewise.
> * src/vbox/vbox_common.c (vboxAttachSound, vboxDumpDisplay)
> (vboxDomainRevertToSnapshot, vboxDomainSnapshotDelete): Likewise.
> * src/vbox/vbox_tmpl.c (vboxStorageVolGetXMLDesc): Likewise.
> 
> Signed-off-by: Eric Blake <eblake at redhat.com>
> ---
>  src/lxc/lxc_driver.c   | 24 ++++++++++++------------
>  src/lxc/lxc_fuse.c     | 34 ++++++++++++++++++----------------
>  src/phyp/phyp_driver.c | 45 ++++++++++++++++++++++++++-------------------
>  src/vbox/vbox_common.c | 12 ++++++------
>  src/vbox/vbox_tmpl.c   |  3 ++-
>  5 files changed, 64 insertions(+), 54 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/36ecde87/attachment-0001.sig>


More information about the libvir-list mailing list