diff options
author | Julia Lawall <julia@diku.dk> | 2010-10-24 17:37:32 -0400 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2010-10-27 18:32:23 -0400 |
commit | 97c1cf8fdbce332bfacc3e38aae3fe2af1369906 (patch) | |
tree | 06b49b7a722fe927b3a41e061c025dbc4deec575 | |
parent | 10247179a7af4f000468e12ae73f3cf657b66235 (diff) |
Coccinelle: Find doubled arguments to boolean or bit operators.
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Nicolas Palix <npalix.work@gmail.com>
Signed-off-by: Michal Marek <mmarek@suse.cz>
-rw-r--r-- | scripts/coccinelle/tests/doublebitand.cocci | 54 | ||||
-rw-r--r-- | scripts/coccinelle/tests/doubletest.cocci | 40 |
2 files changed, 94 insertions, 0 deletions
diff --git a/scripts/coccinelle/tests/doublebitand.cocci b/scripts/coccinelle/tests/doublebitand.cocci new file mode 100644 index 000000000000..9ba73d05a77e --- /dev/null +++ b/scripts/coccinelle/tests/doublebitand.cocci | |||
@@ -0,0 +1,54 @@ | |||
1 | /// Find bit operations that include the same argument more than once | ||
2 | //# One source of false positives is when the argument performs a side | ||
3 | //# effect. Another source of false positives is when a neutral value | ||
4 | //# such as 0 for | is used to indicate no information, to maintain the | ||
5 | //# same structure as other similar expressions | ||
6 | /// | ||
7 | // Confidence: Moderate | ||
8 | // Copyright: (C) 2010 Nicolas Palix, DIKU. GPLv2. | ||
9 | // Copyright: (C) 2010 Julia Lawall, DIKU. GPLv2. | ||
10 | // Copyright: (C) 2010 Gilles Muller, INRIA/LiP6. GPLv2. | ||
11 | // URL: http://coccinelle.lip6.fr/ | ||
12 | // Comments: | ||
13 | // Options: -no_includes -include_headers | ||
14 | |||
15 | virtual context | ||
16 | virtual org | ||
17 | virtual report | ||
18 | |||
19 | @r expression@ | ||
20 | expression E; | ||
21 | position p; | ||
22 | @@ | ||
23 | |||
24 | ( | ||
25 | * E@p | ||
26 | & ... & E | ||
27 | | | ||
28 | * E@p | ||
29 | | ... | E | ||
30 | | | ||
31 | * E@p | ||
32 | & ... & !E | ||
33 | | | ||
34 | * E@p | ||
35 | | ... | !E | ||
36 | | | ||
37 | * !E@p | ||
38 | & ... & E | ||
39 | | | ||
40 | * !E@p | ||
41 | | ... | E | ||
42 | ) | ||
43 | |||
44 | @script:python depends on org@ | ||
45 | p << r.p; | ||
46 | @@ | ||
47 | |||
48 | cocci.print_main("duplicated argument to & or |",p) | ||
49 | |||
50 | @script:python depends on report@ | ||
51 | p << r.p; | ||
52 | @@ | ||
53 | |||
54 | coccilib.report.print_report(p[0],"duplicated argument to & or |") | ||
diff --git a/scripts/coccinelle/tests/doubletest.cocci b/scripts/coccinelle/tests/doubletest.cocci new file mode 100644 index 000000000000..13a2c0e8a4bf --- /dev/null +++ b/scripts/coccinelle/tests/doubletest.cocci | |||
@@ -0,0 +1,40 @@ | |||
1 | /// Find &&/|| operations that include the same argument more than once | ||
2 | //# A common source of false positives is when the argument performs a side | ||
3 | //# effect. | ||
4 | /// | ||
5 | // Confidence: Moderate | ||
6 | // Copyright: (C) 2010 Nicolas Palix, DIKU. GPLv2. | ||
7 | // Copyright: (C) 2010 Julia Lawall, DIKU. GPLv2. | ||
8 | // Copyright: (C) 2010 Gilles Muller, INRIA/LiP6. GPLv2. | ||
9 | // URL: http://coccinelle.lip6.fr/ | ||
10 | // Comments: | ||
11 | // Options: -no_includes -include_headers | ||
12 | |||
13 | virtual context | ||
14 | virtual org | ||
15 | virtual report | ||
16 | |||
17 | @r expression@ | ||
18 | expression E; | ||
19 | position p; | ||
20 | @@ | ||
21 | |||
22 | ( | ||
23 | * E@p | ||
24 | || ... || E | ||
25 | | | ||
26 | * E@p | ||
27 | && ... && E | ||
28 | ) | ||
29 | |||
30 | @script:python depends on org@ | ||
31 | p << r.p; | ||
32 | @@ | ||
33 | |||
34 | cocci.print_main("duplicated argument to && or ||",p) | ||
35 | |||
36 | @script:python depends on report@ | ||
37 | p << r.p; | ||
38 | @@ | ||
39 | |||
40 | coccilib.report.print_report(p[0],"duplicated argument to && or ||") | ||