[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