| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 1 | /// These iterators only exit normally when the loop cursor is NULL, so there | 
|  | 2 | /// is no point to call of_node_put on the final value. | 
|  | 3 | /// | 
|  | 4 | // Confidence: High | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 5 | // Copyright: (C) 2010-2012 Nicolas Palix.  GPLv2. | 
|  | 6 | // Copyright: (C) 2010-2012 Julia Lawall, INRIA/LIP6.  GPLv2. | 
|  | 7 | // Copyright: (C) 2010-2012 Gilles Muller, INRIA/LiP6.  GPLv2. | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 8 | // URL: http://coccinelle.lip6.fr/ | 
|  | 9 | // Comments: | 
|  | 10 | // Options: -no_includes -include_headers | 
|  | 11 |  | 
|  | 12 | virtual patch | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 13 | virtual context | 
|  | 14 | virtual org | 
|  | 15 | virtual report | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 16 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 17 | @depends on patch@ | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 18 | iterator name for_each_node_by_name; | 
|  | 19 | expression np,E; | 
|  | 20 | identifier l; | 
|  | 21 | @@ | 
|  | 22 |  | 
|  | 23 | for_each_node_by_name(np,...) { | 
|  | 24 | ... when != break; | 
|  | 25 | when != goto l; | 
|  | 26 | } | 
|  | 27 | ... when != np = E | 
|  | 28 | - of_node_put(np); | 
|  | 29 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 30 | @depends on patch@ | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 31 | iterator name for_each_node_by_type; | 
|  | 32 | expression np,E; | 
|  | 33 | identifier l; | 
|  | 34 | @@ | 
|  | 35 |  | 
|  | 36 | for_each_node_by_type(np,...) { | 
|  | 37 | ... when != break; | 
|  | 38 | when != goto l; | 
|  | 39 | } | 
|  | 40 | ... when != np = E | 
|  | 41 | - of_node_put(np); | 
|  | 42 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 43 | @depends on patch@ | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 44 | iterator name for_each_compatible_node; | 
|  | 45 | expression np,E; | 
|  | 46 | identifier l; | 
|  | 47 | @@ | 
|  | 48 |  | 
|  | 49 | for_each_compatible_node(np,...) { | 
|  | 50 | ... when != break; | 
|  | 51 | when != goto l; | 
|  | 52 | } | 
|  | 53 | ... when != np = E | 
|  | 54 | - of_node_put(np); | 
|  | 55 |  | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 56 | @depends on patch@ | 
| Nicolas Palix | 4045159 | 2010-08-24 17:39:01 +0200 | [diff] [blame] | 57 | iterator name for_each_matching_node; | 
|  | 58 | expression np,E; | 
|  | 59 | identifier l; | 
|  | 60 | @@ | 
|  | 61 |  | 
|  | 62 | for_each_matching_node(np,...) { | 
|  | 63 | ... when != break; | 
|  | 64 | when != goto l; | 
|  | 65 | } | 
|  | 66 | ... when != np = E | 
|  | 67 | - of_node_put(np); | 
| Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 68 |  | 
|  | 69 | // ---------------------------------------------------------------------- | 
|  | 70 |  | 
|  | 71 | @r depends on !patch forall@ | 
|  | 72 | //iterator name for_each_node_by_name; | 
|  | 73 | //iterator name for_each_node_by_type; | 
|  | 74 | //iterator name for_each_compatible_node; | 
|  | 75 | //iterator name for_each_matching_node; | 
|  | 76 | expression np,E; | 
|  | 77 | identifier l; | 
|  | 78 | position p1,p2; | 
|  | 79 | @@ | 
|  | 80 |  | 
|  | 81 | ( | 
|  | 82 | *for_each_node_by_name@p1(np,...) | 
|  | 83 | { | 
|  | 84 | ... when != break; | 
|  | 85 | when != goto l; | 
|  | 86 | } | 
|  | 87 | | | 
|  | 88 | *for_each_node_by_type@p1(np,...) | 
|  | 89 | { | 
|  | 90 | ... when != break; | 
|  | 91 | when != goto l; | 
|  | 92 | } | 
|  | 93 | | | 
|  | 94 | *for_each_compatible_node@p1(np,...) | 
|  | 95 | { | 
|  | 96 | ... when != break; | 
|  | 97 | when != goto l; | 
|  | 98 | } | 
|  | 99 | | | 
|  | 100 | *for_each_matching_node@p1(np,...) | 
|  | 101 | { | 
|  | 102 | ... when != break; | 
|  | 103 | when != goto l; | 
|  | 104 | } | 
|  | 105 | ) | 
|  | 106 | ... when != np = E | 
|  | 107 | * of_node_put@p2(np); | 
|  | 108 |  | 
|  | 109 | @script:python depends on org@ | 
|  | 110 | p1 << r.p1; | 
|  | 111 | p2 << r.p2; | 
|  | 112 | @@ | 
|  | 113 |  | 
|  | 114 | cocci.print_main("unneeded of_node_put",p2) | 
|  | 115 | cocci.print_secs("iterator",p1) | 
|  | 116 |  | 
|  | 117 | @script:python depends on report@ | 
|  | 118 | p1 << r.p1; | 
|  | 119 | p2 << r.p2; | 
|  | 120 | @@ | 
|  | 121 |  | 
|  | 122 | msg = "ERROR: of_node_put not needed after iterator on line %s" % (p1[0].line) | 
|  | 123 | coccilib.report.print_report(p2[0], msg) |