[libvirt] [PATCH] qemu: Remove extraneous space in function prototypes

Martin Kletzander mkletzan at redhat.com
Thu Jul 24 14:18:55 UTC 2014


On Thu, Jul 24, 2014 at 09:31:33AM -0400, John Ferlan wrote:
>During review of the iSCSI hostdev series, eblake noted that the
>prototypes shouldn't have the extranenous space between the "*" and
>the function name:
>
>http://www.redhat.com/archives/libvir-list/2014-July/msg01227.html
>
>Since it was more invasive than 1 or 2 lines - I said I'd send a
>patch covering this once committed.
>
>Signed-off-by: John Ferlan <jferlan at redhat.com>
>---
>NOTE: There are other instances of this in other trees - I figured
>I'd just cover this particular directory.  I can do more if desired...
>

Not that it's required (because there's a LOT of those), but if you
will, I'd have a hint.  If you modify bracket-spacing.pl with diff
that follows, you can then run (because by default bracket-spacing.pl
runs only on .c files):

git ls-files | grep '\.[hc]$' | xargs build-aux/bracket-spacing.pl

And you'll have a rough idea.  I'm sure there are many false positives
and true negatives, so syntax-check cannot be probably created out of
this.

diff --git i/build-aux/bracket-spacing.pl
w/build-aux/bracket-spacing.pl
index e4ae8f0..cda5f2a 100755
--- i/build-aux/bracket-spacing.pl
+++ w/build-aux/bracket-spacing.pl
@@ -155,6 +155,13 @@ foreach my $file (@ARGV) {
             $ret = 1;
             last;
         }
+
+        while ($data =~ /^[a-zA-Z _]*\* [a-zA-Z]/) {
+            print "$file:$.: $line";
+            $ret = 1;
+            last;
+        }
+
     }
     close FILE;
 }
--

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140724/20d39277/attachment-0001.sig>


More information about the libvir-list mailing list