On Mon, 2015-11-02 at 10:49 +0100, Peter Krempa wrote:
On Mon, Nov 02, 2015 at 10:34:35 +0100, Andrea Bolognani wrote:
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]+ +:$$' \
Our labels enforce at least one space before a label so I don't think this will work.
You are of course absolutely right. I was trying to catch the case where two mistakes are present in the same line, eg. myfancylabel : but my implementation was flawed. v2 on its way.
If you will be adding a multi-space match, please make sure that "case" is not matched in that case.
That shouldn't happen as there will always be some other token between the whitespace after 'case' and the colon, right? Cheers. -- Andrea Bolognani Software Engineer - Virtualization Team