[libvirt] [PATCH 2/2] syntax-check: mandate space after mid-line semicolon

2013-05-24 Thread Eric Blake
Enforce the style cleanup in the previous patch. * build-aux/bracket-spacing.pl: Enforce trailing spacing. * cfg.mk (bracket-spacing-check): Tweak error wording. * docs/hacking.html.in: Document the rule. * HACKING: Regenerate. Signed-off-by: Eric Blake --- HACKING | 23 +++

Re: [libvirt] [PATCH 2/2] syntax-check: mandate space after mid-line semicolon

2013-05-25 Thread Martin Kletzander
On 05/24/2013 07:19 PM, Eric Blake wrote: > Enforce the style cleanup in the previous patch. > > * build-aux/bracket-spacing.pl: Enforce trailing spacing. > * cfg.mk (bracket-spacing-check): Tweak error wording. > * docs/hacking.html.in: Document the rule. > * HACKING: Regenerate. > > Signed-off-

Re: [libvirt] [PATCH 2/2] syntax-check: mandate space after mid-line semicolon

2013-05-28 Thread Eric Blake
On 05/25/2013 03:13 PM, Martin Kletzander wrote: >> @@ -31,6 +32,9 @@ foreach my $file (@ARGV) { >> while (defined (my $line = )) { >> my $data = $line; >> >> +# Kill any quoted ; or " >> +$data =~ s,'[";]','X',g; >> + > > Good catch, I didn't know this was missing.

Re: [libvirt] [PATCH 2/2] syntax-check: mandate space after mid-line semicolon

2013-05-28 Thread Osier Yang
On 25/05/13 01:19, Eric Blake wrote: Enforce the style cleanup in the previous patch. * build-aux/bracket-spacing.pl: Enforce trailing spacing. * cfg.mk (bracket-spacing-check): Tweak error wording. * docs/hacking.html.in: Document the rule. * HACKING: Regenerate. Signed-off-by: Eric Blake ---

Re: [libvirt] [PATCH 2/2] syntax-check: mandate space after mid-line semicolon

2013-05-28 Thread Eric Blake
On 05/28/2013 09:28 AM, Osier Yang wrote: > On 25/05/13 01:19, Eric Blake wrote: >> Enforce the style cleanup in the previous patch. >> >> * build-aux/bracket-spacing.pl: Enforce trailing spacing. >> * cfg.mk (bracket-spacing-check): Tweak error wording. >> * docs/hacking.html.in: Document the rule