Re: [libvirt] [PATCH v2 2/2] Document bracket whitespace rules & add syntax-check rule

From: "Daniel P. Berrange" <berrange(a)redhat.com&gt;

--- build-aux/bracket-spacing.pl | 116 +++++++++++++++++++++++++++++++++++++++++++ cfg.mk | 7 ++- docs/hacking.html.in | 49 ++++++++++++++++++ 3 files changed, 171 insertions(+), 1 deletion(-) create mode 100755 build-aux/bracket-spacing.pl

diff --git a/build-aux/bracket-spacing.pl b/build-aux/bracket-spacing.pl new file mode 100755 index 0000000..d3a916f --- /dev/null +++ b/build-aux/bracket-spacing.pl @@ -0,0 +1,116 @@ +#!/usr/bin/perl +# +# bracket-spacing.pl: Report any usage of 'function (..args..)' +# +# This library is free software; you can redistribute it and/or +# modify it under the terms of the GNU Lesser General Public +# License as published by the Free Software Foundation; either +# version 2.1 of the License, or (at your option) any later version. +# +# This library is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +# Lesser General Public License for more details. +# +# You should have received a copy of the GNU Lesser General Public +# License along with this library. If not, see +# <http://www.gnu.org/licenses/>;. +# +# Authors: +# Daniel P. Berrange <berrange(a)redhat.com&gt; + +use strict; +use warnings; + +my $ret = 0; +my $incomment = 0; + +foreach my $file (@ARGV) { + open FILE, $file; + + while (defined (my $line = <FILE>)) { + my $data = $line; + + # Kill any quoted strongs + $data =~ s,".*?","XXX",g;

diff --git a/docs/hacking.html.in b/docs/hacking.html.in index d41b39c..37ed00b 100644 --- a/docs/hacking.html.in +++ b/docs/hacking.html.in @@ -212,6 +212,55 @@ </p> + <h2><a name="bracket_spacing">Bracket spacing</a></h2> + + <p> + The keywords <code>if</code>, <code>for</code>, <code>while</code>, + and <code>switch</code> must have a single space following them + before the opening bracket. eg