[PATCH 2/2] maint: syntax check for double semicolons

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Commit 28de556 fixed some doubled semicolons, but failed to
add a syntax check, and thereby missed another instance. This
finishes the job (and also found an instance in gnulib, prior
to the previous patch's submodule update).

* cfg.mk (sc_prohibit_double_semicolon): New rule.
* src/xenconfig/xen_common.c (xenFormatConfigCommon): Fix offender.

Signed-off-by: Eric Blake <eblake@xxxxxxxxxx>
---
 cfg.mk                     | 8 ++++++++
 src/xenconfig/xen_common.c | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/cfg.mk b/cfg.mk
index 77f1868..0a4395b 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -803,6 +803,14 @@ sc_prohibit_semicolon_at_eol_in_python:
 	halt="Don't use semicolon at eol in python files"		\
 	  $(_sc_search_regexp)

+# Except for shell files and for loops, double semicolon is probably a mistake
+sc_prohibit_double_semicolon:
+	@prohibit=';;'							\
+	in_files='\.[chx]$$'						\
+	exclude='for \(.*\)'						\
+	halt="Double semicolon detected"				\
+	  $(_sc_search_regexp)
+
 # mymain() in test files should use return, not exit, for nicer output
 sc_prohibit_exit_in_tests:
 	@prohibit='\<exit *\('						\
diff --git a/src/xenconfig/xen_common.c b/src/xenconfig/xen_common.c
index accd25f..bab67de 100644
--- a/src/xenconfig/xen_common.c
+++ b/src/xenconfig/xen_common.c
@@ -2034,7 +2034,7 @@ xenFormatConfigCommon(virConfPtr conf,
         return -1;

     if (xenFormatCPUFeatures(conf, def, xendConfigVersion) < 0)
-        return -1;;
+        return -1;

     if (xenFormatCDROM(conf, def, xendConfigVersion) < 0)
         return -1;
-- 
1.9.3

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]