blob: d212fde95368adb65e36604e377ae4f3981835fe (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
|
#!/usr/bin/perl
use strict;
my $date = `date`;
print "# $date";
sub escape {
my $text = shift;
$text =~ s/\\/\\\\/g;
$text =~ s/\n/\\n/g;
$text =~ s/\t/\\t/g;
return $text;
}
sub checkLine {
my $line = shift;
my $file = shift;
my $row = shift;
$file = escape($file);
$row = escape($row);
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /(FIXME|TODO)(.*)$/) {
print "$file\t$row\tWARNING\t$1$2\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /(qDebug|QDebug>|QtDebug>|<debug>)/) {
print "$file\t$row\tWARNING\tRemove debug code.\n";
}
if ($line =~ /^(<<<<|====|>>>>)/) {
print "$file\n$row\tERROR\tResolved conflict.\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /(Q_SIGNALS|Q_SLOTS)/) {
print "$file\t$row\tWARNING\tPrefer signals and slots over Q_SIGNALS and Q_SLOTS.\n";
}
if ($line =~ /^#\s*(warning|error) (.*)$/) {
print "$file\t$row\t$1\tClean up preprocessor $1:$2.\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /\s+$/) {
print "$file\t$row\tWARNING\tTrailing whitespaces found.\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /^ *\t */) {
print "$file\t$row\tWARNING\tTabs used to indent.\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /^\s+{$/) {
print "$file\t$row\tWARNING\tOpening bracket found all alone on a line.\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /{.*[^\s].*$/) {
if ($line !~ /\{\s+\/\*/ && $line !~ /^\s*\{ \}$/ && $line !~ /^namespace .* \}$/)
{ print "$file\t$row\tWARNING\tText found after opening bracket.\n"; }
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /[a-zA-Z0-9_]\*[^\/]/) {
next if $line =~ /SIGNAL\(/;
next if $line =~ /SLOT\(/;
print "$file\t$row\tWARNING\tUse \"TYPE *NAME\" (currently the space between TYPE and * is missing).\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /[a-zA-Z0-9_]&[^&]/) {
print "$file\t$row\tWARNING\tUse \"TYPE &NAME\" (currently the space between TYPE and & is missing).\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /\*\s+/) {
next if ($line =~ /^\s*\*\*?\s+/);
next if ($line =~ /\/\*/);
print "$file\t$row\tWARNING\tUse \"TYPE *NAME\" (currently there is a space between * and NAME).\n";
}
if ($file =~ /(.cpp|.c|.cxx|.h)$/ && $line =~ /[^&]&\s+/) {
print "$file\t$row\tWARNING\tUse \"TYPE &NAME\" (currently there is a space between & and NAME).\n";
}
}
my $filename = "";
my $pos = 0;
sub getDiffOrigin {
my $currentBranch = `git branch | grep "^* "`;
chop $currentBranch;
$currentBranch =~ s/^\s*\* //;
my $remoteRepo = `git config --local --get branch.$currentBranch.remote`;
chop $remoteRepo;
return "HEAD" if (!$remoteRepo);
my $remoteBranch = `git config --local --get branch.$currentBranch.merge`;
chop $remoteBranch;
$remoteBranch =~ s!^refs/heads/!!;
return "HEAD" if (!$remoteBranch);
return "$remoteRepo/$remoteBranch";
}
my $origin = shift;
$origin = getDiffOrigin unless $origin;
print "# running: git diff $origin ...\n";
open(PIPE, "git diff $origin|");
while (<PIPE>) {
chomp;
my $line = $_;
print "#$line\n";
next if ($line =~ /^-/);
if ($line =~ /^\+\+\+ (.*)$/) {
$filename = $1;
$filename =~ s/^b\///;
$pos = 0;
next;
}
next if $filename =~ /^\/dev\/null$/;
if ($line =~ /^@@ -\d+,\d+\s\+(\d+),\d+ @@$/) {
$pos = $1 - 1;
next;
}
$pos = $pos + 1;
if ($line =~ /^\+(.*)/) {
checkLine($1, $filename, $pos);
}
}
close(PIPE);
|
