[libvirt] [PATCH] syntax-check: Add sc_prohibit_space_in_label rule

Andrea Bolognani abologna at redhat.com
Mon Nov 2 09:34:35 UTC 2015


This guards against code such as

 cleanup :

which is happily accepted by the compiler but does not conform
to our style guidelines.
---
 cfg.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/cfg.mk b/cfg.mk
index a9bba38..8462051 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -919,6 +919,12 @@ sc_require_space_before_label:
 	halt="Top-level labels should be indented by one space"        \
 	  $(_sc_search_regexp)
 
+sc_prohibit_space_in_label:
+	@prohibit='^[_a-zA-Z0-9]+ +:$$'                                \
+	in_vc_files='\.[ch]$$'                                         \
+	halt="There should be no space between label name and colon"   \
+	  $(_sc_search_regexp)
+
 # Doesn't catch all cases of mismatched braces across if-else, but it helps
 sc_require_if_else_matching_braces:
 	@prohibit='(  else( if .*\))? {|} else( if .*\))?$$)'		\
-- 
2.4.3




More information about the libvir-list mailing list