[libvirt] [PATCH 2/6] maint: use consistent if-else braces in qemu

Peter Krempa pkrempa at redhat.com
Thu Sep 4 12:01:38 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 commit focuses on the qemu driver.
> 
> * src/qemu/qemu_command.c (qemuParseISCSIString)
> (qemuParseCommandLineDisk, qemuParseCommandLine)
> (qemuBuildSmpArgStr, qemuBuildCommandLine)
> (qemuParseCommandLineDisk, qemuParseCommandLineSmp): Correct use
> of {}.
> * src/qemu/qemu_capabilities.c (virQEMUCapsProbeCPUModels):
> Likewise.
> * src/qemu/qemu_driver.c (qemuDomainCoreDumpWithFormat)
> (qemuDomainRestoreFlags, qemuDomainGetInfo)
> (qemuDomainMergeBlkioDevice): Likewise.
> * src/qemu/qemu_hotplug.c (qemuDomainAttachNetDevice): Likewise.
> * src/qemu/qemu_monitor_text.c (qemuMonitorTextCreateSnapshot)
> (qemuMonitorTextLoadSnapshot, qemuMonitorTextDeleteSnapshot):
> Likewise.
> * src/qemu/qemu_process.c (qemuProcessStop): Likewise.
> 
> Signed-off-by: Eric Blake <eblake at redhat.com>
> ---
>  src/qemu/qemu_capabilities.c |  6 ++---
>  src/qemu/qemu_command.c      | 56 ++++++++++++++++++++++++--------------------
>  src/qemu/qemu_driver.c       | 36 ++++++++++++++--------------
>  src/qemu/qemu_hotplug.c      |  9 +++----
>  src/qemu/qemu_monitor_text.c | 30 ++++++++----------------
>  src/qemu/qemu_process.c      |  3 +--
>  6 files changed, 67 insertions(+), 73 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/7af3fbda/attachment-0001.sig>


More information about the libvir-list mailing list