[libvirt] [PATCH] Adjust backslash alignment syntax-check
Michal Privoznik
mprivozn at redhat.com
Thu Nov 23 05:46:02 UTC 2017
On 11/22/2017 11:17 PM, Martin Kletzander wrote:
> We have a check for backslash alignment that checks for two blanks preceding a
> backslash. However there can be alignment done using a tabulator and in some
> cases one might be enough. There are none currently, but I found out that was
> the case before. at some point. so let's check for \t preceding the final
> backslash as well.
>
> Signed-off-by: Martin Kletzander <mkletzan at redhat.com>
> ---
> cfg.mk | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cfg.mk b/cfg.mk
> index 5321a52b7ee2..ea10ca19f512 100644
> --- a/cfg.mk
> +++ b/cfg.mk
> @@ -1038,7 +1038,7 @@ sc_prohibit_http_urls:
> $(_sc_search_regexp)
>
> sc_prohibit_backslash_alignment:
> - @prohibit='[[:blank:]][[:blank:]]\\$$' \
> + @prohibit='([[:blank:]][[:blank:]]| )\\$$' \
> in_vc_files='*\.([chx]|am|mk)$$' \
> halt='Do not attempt to right-align backslashes' \
> $(_sc_search_regexp)
>
ACK
Michal
More information about the libvir-list
mailing list