diff mbox series

[-next,v2,2/9] coccinelle: Add rules to find str_false_true() replacements

Message ID 20240904091451.3328272-3-lihongbo22@huawei.com (mailing list archive)
State Superseded
Headers show
Series coccinelle: Add some rules for string_chioces helpers. | expand

Commit Message

Hongbo Li Sept. 4, 2024, 9:14 a.m. UTC 
As done with str_true_false(), add checks for str_false_true()
opportunities. A simple test can find over 9 cases currently
exist in the tree.

Signed-off-by: Hongbo Li <lihongbo22@huawei.com>
---
 scripts/coccinelle/api/string_choices.cocci | 23 +++++++++++++++++++++
 1 file changed, 23 insertions(+)
diff mbox series

Patch

diff --git a/scripts/coccinelle/api/string_choices.cocci b/scripts/coccinelle/api/string_choices.cocci
index 50329c4c8acd..a1141bd29b3c 100644
--- a/scripts/coccinelle/api/string_choices.cocci
+++ b/scripts/coccinelle/api/string_choices.cocci
@@ -108,3 +108,26 @@  e << str_true_false_r.E;
 @@
 
 coccilib.report.print_report(p[0], "opportunity for str_true_false(%s)" % e)
+
+@str_false_true depends on patch@
+expression E;
+@@
+(
+-      ((E) ? "false" : "true")
++      str_false_true(E)
+)
+
+@str_false_true_r depends on !patch exists@
+expression E;
+position P;
+@@
+(
+*      ((E@P) ? "false" : "true")
+)
+
+@script:python depends on report@
+p << str_false_true_r.P;
+e << str_false_true_r.E;
+@@
+
+coccilib.report.print_report(p[0], "opportunity for str_false_true(%s)" % e)