[PATCH i-g-t 21/33] cocci: Split out igt_swap() rules

[Lyude]

From: Lyude Paul <lyude at redhat.com>

Signed-off-by: Lyude Paul <lyude at redhat.com>
---
 scripts/cocci/igt.cocci      | 27 ---------------------------
 scripts/cocci/igt_swap.cocci | 30 ++++++++++++++++++++++++++++++
 2 files changed, 30 insertions(+), 27 deletions(-)
 create mode 100644 scripts/cocci/igt_swap.cocci

diff --git a/scripts/cocci/igt.cocci b/scripts/cocci/igt.cocci
index 84df03ca..ef3a5a40 100644
--- a/scripts/cocci/igt.cocci
+++ b/scripts/cocci/igt.cocci
@@ -31,33 +31,6 @@ def warn_cmp_macro(p, type_prefix, op, e1, e2):
     msg = 'WARNING: igt_assert_%s%s(%s, %s) should be used instead' % args
     coccilib.report.print_report(p[0], msg)
 
-// Replace open-coded igt_swap()
- at depends on patch@
-type T;
-T a, b, tmp;
-@@
-- tmp = a;
-- a = b;
-- b = tmp;
-+ igt_swap(a, b);
-
- at igt_swap_report depends on report@
-position p;
-type T;
-T a, b, tmp;
-@@
-tmp = a;
-a = b;
-b = tmp at p;
-
- at script:python depends on report@
-p << igt_swap_report.p;
-a << igt_swap_report.a;
-b << igt_swap_report.b;
-@@
-msg = "WARNING: igt_swap(%s, %s) should be used instead" % (a, b)
-coccilib.report.print_report(p[0], msg)
-
 // So we don't complain about #define min(), #define max(), etc.
 @min_max_ignore@
 identifier i =~ "^m(in|ax)$";
diff --git a/scripts/cocci/igt_swap.cocci b/scripts/cocci/igt_swap.cocci
new file mode 100644
index 00000000..9601bfb7
--- /dev/null
+++ b/scripts/cocci/igt_swap.cocci
@@ -0,0 +1,30 @@
+// Replace open-coded igt_swap()
+
+virtual report
+virtual patch
+
+ at depends on patch@
+type T;
+T a, b, tmp;
+@@
+- tmp = a;
+- a = b;
+- b = tmp;
++ igt_swap(a, b);
+
+ at igt_swap_report depends on report@
+position p;
+type T;
+T a, b, tmp;
+@@
+tmp = a;
+a = b;
+b = tmp at p;
+
+ at script:python depends on report@
+p << igt_swap_report.p;
+a << igt_swap_report.a;
+b << igt_swap_report.b;
+@@
+msg = "WARNING: igt_swap(%s, %s) should be used instead" % (a, b)
+coccilib.report.print_report(p[0], msg)
-- 
2.21.0