Re: [libvirt] [PATCH] Adjust backslash alignment syntax-check

2017-12-04 Thread Andrea Bolognani
On Thu, 2017-11-23 at 06:46 +0100, Michal Privoznik wrote: > > @@ -1038,7 +1038,7 @@ sc_prohibit_http_urls: > > $(_sc_search_regexp) > > > > sc_prohibit_backslash_alignment: > > - @prohibit='[[:blank:]][[:blank:]]\\$$' \ > > + @prohibit='([[:blank:]][[:blank:]]| )\\$$' \ > >

Re: [libvirt] [PATCH] Adjust backslash alignment syntax-check

2017-11-22 Thread Michal Privoznik
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

[libvirt] [PATCH] Adjust backslash alignment syntax-check

2017-11-22 Thread Martin Kletzander
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