[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 03/10] checkpatch.pl: Check .cpp files
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH v8 03/10] checkpatch.pl: Check .cpp files |
Date: |
Thu, 25 Jul 2013 18:16:25 -0500 |
User-agent: |
alot/0.3.4 |
Quoting Tomoki Sekiyama (2013-07-23 17:45:39)
> Enable checkpatch.pl to apply the same checks as C source files for
> C++ files with .cpp extensions. It also adds some exceptions for C++
> sources to suppress errors for:
> - <> used in C++ template arguments (e.g. template <class T>)
> - :: used to represent namespaces (e.g. SomeClass::method())
> - : used in class declaration (e.g. class T : public Super)
> - ~ used in destructor method name (e.g. T::~T())
> - spacing around 'catch' (e.g. catch (...))
>
> Signed-off-by: Tomoki Sekiyama <address@hidden>
Reviewed-by: Michael Roth <address@hidden>
> ---
> scripts/checkpatch.pl | 34 ++++++++++++++++++++++++++++------
> 1 file changed, 28 insertions(+), 6 deletions(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index ec0aa4c..9d46e5a 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -1363,7 +1363,7 @@ sub process {
> # Check for incorrect file permissions
> if ($line =~ /^new (file )?mode.*[7531]\d{0,2}$/) {
> my $permhere = $here . "FILE: $realfile\n";
> - if ($realfile =~
> /(Makefile|Kconfig|\.c|\.h|\.S|\.tmpl)$/) {
> + if ($realfile =~
> /(Makefile|Kconfig|\.c|\.cpp|\.h|\.S|\.tmpl)$/) {
> ERROR("do not set execute permissions for
> source files\n" . $permhere);
> }
> }
> @@ -1460,7 +1460,7 @@ sub process {
> }
>
> # check we are in a valid source file if not then ignore this hunk
> - next if ($realfile !~ /\.(h|c|s|S|pl|sh)$/);
> + next if ($realfile !~ /\.(h|c|cpp|s|S|pl|sh)$/);
>
> #80 column limit
> if ($line =~ /^\+/ && $prevrawline !~ /\/\*\*/ &&
> @@ -1495,7 +1495,7 @@ sub process {
> }
>
> # check we are in a valid source file C or perl if not then ignore this hunk
> - next if ($realfile !~ /\.(h|c|pl)$/);
> + next if ($realfile !~ /\.(h|c|cpp|pl)$/);
>
> # in QEMU, no tabs are allowed
> if ($rawline =~ /^\+.* /) {
> @@ -1505,7 +1505,7 @@ sub process {
> }
>
> # check we are in a valid C source file if not then ignore this hunk
> - next if ($realfile !~ /\.(h|c)$/);
> + next if ($realfile !~ /\.(h|c|cpp)$/);
>
> # check for RCS/CVS revision markers
> if ($rawline =~ /^\+.*\$(Revision|Log|Id)(?:\$|)/) {
> @@ -1969,6 +1969,9 @@ sub process {
> asm|__asm__)$/x)
> {
>
> + # Ignore 'catch (...)' in C++
> + } elsif ($name =~ /^catch$/ && $realfile =~
> /(\.cpp|\.h)$/) {
> +
> # cpp #define statements have non-optional spaces, ie
> # if there is a space between the name and the open
> # parenthesis it is simply not a parameter group.
> @@ -1992,7 +1995,7 @@ sub process {
> \+=|-=|\*=|\/=|%=|\^=|\|=|&=|
> =>|->|<<|>>|<|>|=|!|~|
> &&|\|\||,|\^|\+\+|--|&|\||\+|-|\*|\/|%|
> - \?|:
> + \?|::|:
> }x;
> my @elements = split(/($ops|;)/, $opline);
> my $off = 0;
> @@ -2062,6 +2065,10 @@ sub process {
> # // is a comment
> } elsif ($op eq '//') {
>
> + # Ignore : used in class declaration in C++
> + } elsif ($opv eq ':B' && $ctx =~ /Wx[WE]/ &&
> + $line =~ /class/ &&
> $realfile =~ /(\.cpp|\.h)$/) {
> +
> # No spaces for:
> # ->
> # : when part of a bitfield
> @@ -2088,7 +2095,10 @@ sub process {
> } elsif ($op eq '!' || $op eq '~' ||
> $opv eq '*U' || $opv eq '-U' ||
> $opv eq '&U' || $opv eq '&&U') {
> - if ($ctx !~ /[WEBC]x./ && $ca !~
> /(?:\)|!|~|\*|-|\&|\||\+\+|\-\-|\{)$/) {
> + if ($op eq '~' && $ca =~ /::$/ &&
> $realfile =~ /(\.cpp|\.h)$/) {
> + # '~' used as a name of
> Destructor
> +
> + } elsif ($ctx !~ /[WEBC]x./ && $ca !~
> /(?:\)|!|~|\*|-|\&|\||\+\+|\-\-|\{)$/) {
> ERROR("space required before
> that '$op' $at\n" . $hereptr);
> }
> if ($op eq '*' && $cc
> =~/\s*$Modifier\b/) {
> @@ -2135,6 +2145,18 @@ sub process {
> } elsif ($ctx !~ /[EWC]x[CWE]/) {
> my $ok = 0;
>
> + if ($realfile =~ /\.cpp|\.h$/) {
> + # Ignore template arguments
> <...> in C++
> + if (($op eq '<' || $op eq
> '>') && $line =~ /<.*>/) {
> + $ok = 1;
> + }
> +
> + # Ignore :: in C++
> + if ($op eq '::') {
> + $ok = 1;
> + }
> + }
> +
> # Ignore email addresses
> <address@hidden>
> if (($op eq '<' &&
> $cc =~ /address@hidden>/) ||
- [Qemu-devel] [PATCH v8 00/10] qemu-ga: fsfreeze on Windows using VSS, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 01/10] configure: Support configuring C++ compiler, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 02/10] Add c++ keywords to QAPI helper script, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 03/10] checkpatch.pl: Check .cpp files, Tomoki Sekiyama, 2013/07/23
- Re: [Qemu-devel] [PATCH v8 03/10] checkpatch.pl: Check .cpp files,
Michael Roth <=
- [Qemu-devel] [PATCH v8 04/10] Add a script to extract VSS SDK headers on POSIX system, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 05/10] qemu-ga: Add configure options to specify path to Windows/VSS SDK, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 06/10] error: Add error_set_win32 and error_setg_win32, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 09/10] qemu-ga: Install Windows VSS provider on `qemu-ga -s install', Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 08/10] qemu-ga: Call Windows VSS requester in fsfreeze command handler, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 10/10] QMP/qemu-ga-client: Make timeout longer for guest-fsfreeze-freeze command, Tomoki Sekiyama, 2013/07/23
- [Qemu-devel] [PATCH v8 07/10] qemu-ga: Add Windows VSS provider and requester as DLL, Tomoki Sekiyama, 2013/07/23