[libvirt] [PATCH 0/2] syntax check: hanging braces

Eric Blake eblake at redhat.com
Wed Sep 3 23:17:15 UTC 2014


Another case where it is worth enforcing a common existing
style, and documenting it in HACKING.  Here, there were
fewer offenders, so the series is not split in as many pieces.
Obviously depends on my earlier series for if-else {} checks,
as otherwise this will detect several 'else' as lacking a
hanging brace.

Eric Blake (2):
  maint: use hanging curly braces
  maint: tighten curly brace syntax checking

 HACKING                                 | 27 +++++++++++++++++++++++++++
 cfg.mk                                  | 18 ++++++++++++------
 docs/hacking.html.in                    | 31 +++++++++++++++++++++++++++++++
 src/conf/domain_conf.c                  |  6 ++----
 src/interface/interface_backend_netcf.c |  9 +++------
 src/network/bridge_driver.c             |  3 +--
 src/util/virfile.c                      |  3 +--
 src/util/virnetdev.c                    |  3 +--
 src/util/virnetdevmacvlan.c             |  5 ++---
 src/util/virtypedparam.c                |  3 +--
 src/util/virutil.c                      |  6 ++----
 src/vbox/vbox_common.c                  |  4 ++--
 tests/seclabeltest.c                    |  6 ++----
 13 files changed, 87 insertions(+), 37 deletions(-)

-- 
1.9.3




More information about the libvir-list mailing list