diff options
author | Michal Marek <mmarek@suse.cz> | 2010-12-14 16:01:55 -0500 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2010-12-14 16:01:55 -0500 |
commit | 8990c1bc4be46473ad19bf2fa612ca57286f3df4 (patch) | |
tree | 3cea60576903a1d26c67e6ec62891b524d390e95 /scripts/coccinelle/misc/ifcol.cocci | |
parent | 2979076fbf17a0947d6eba367b0cac19c907c160 (diff) | |
parent | c8ddb2713c624f432fa5fe3c7ecffcdda46ea0d4 (diff) |
Merge commit 'v2.6.37-rc1' into kbuild/kbuild
Diffstat (limited to 'scripts/coccinelle/misc/ifcol.cocci')
-rw-r--r-- | scripts/coccinelle/misc/ifcol.cocci | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/scripts/coccinelle/misc/ifcol.cocci b/scripts/coccinelle/misc/ifcol.cocci new file mode 100644 index 000000000000..b7ed91dbeb95 --- /dev/null +++ b/scripts/coccinelle/misc/ifcol.cocci | |||
@@ -0,0 +1,48 @@ | |||
1 | /// Find confusingly indented code in or after an if. An if branch should | ||
2 | /// be indented. The code following an if should not be indented. | ||
3 | /// Sometimes, code after an if that is indented is actually intended to be | ||
4 | /// part of the if branch. | ||
5 | /// | ||
6 | /// This has a high rate of false positives, because Coccinelle's column | ||
7 | /// calculation does not distinguish between spaces and tabs, so code that | ||
8 | /// is not visually aligned may be considered to be in the same column. | ||
9 | /// | ||
10 | // Confidence: Low | ||
11 | // Copyright: (C) 2010 Nicolas Palix, DIKU. GPLv2. | ||
12 | // Copyright: (C) 2010 Julia Lawall, DIKU. GPLv2. | ||
13 | // Copyright: (C) 2010 Gilles Muller, INRIA/LiP6. GPLv2. | ||
14 | // URL: http://coccinelle.lip6.fr/ | ||
15 | // Comments: | ||
16 | // Options: -no_includes -include_headers | ||
17 | |||
18 | virtual org | ||
19 | virtual report | ||
20 | |||
21 | @r disable braces4@ | ||
22 | position p1,p2; | ||
23 | statement S1,S2; | ||
24 | @@ | ||
25 | |||
26 | ( | ||
27 | if (...) { ... } | ||
28 | | | ||
29 | if (...) S1@p1 S2@p2 | ||
30 | ) | ||
31 | |||
32 | @script:python depends on org@ | ||
33 | p1 << r.p1; | ||
34 | p2 << r.p2; | ||
35 | @@ | ||
36 | |||
37 | if (p1[0].column == p2[0].column): | ||
38 | cocci.print_main("branch",p1) | ||
39 | cocci.print_secs("after",p2) | ||
40 | |||
41 | @script:python depends on report@ | ||
42 | p1 << r.p1; | ||
43 | p2 << r.p2; | ||
44 | @@ | ||
45 | |||
46 | if (p1[0].column == p2[0].column): | ||
47 | msg = "code aligned with following code on line %s" % (p2[0].line) | ||
48 | coccilib.report.print_report(p1[0],msg) | ||