[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Sun, 5 Jun 2011 19:26:52 -0500
From: Greg Dietsche <Gregory.Dietsche@....edu>
To: julia@...u.dk
Cc: Gilles.Muller@...6.fr, npalix.work@...il.com, cocci@...u.dk,
linux-kernel@...r.kernel.org,
Greg Dietsche <Gregory.Dietsche@....edu>
Subject: [PATCH] coccinelle: if(ret)return ret; return ret; semantic patch
This semantic patch finds code matching this pattern:
if(ret)
return ret;
return ret;
I will be submitting patches shortly against the mainline to cleanup all
code matching this pattern.
Signed-off-by: Greg Dietsche <Gregory.Dietsche@....edu>
---
scripts/coccinelle/misc/doublereturn.cocci | 20 ++++++++++++++++++++
1 files changed, 20 insertions(+), 0 deletions(-)
create mode 100644 scripts/coccinelle/misc/doublereturn.cocci
diff --git a/scripts/coccinelle/misc/doublereturn.cocci b/scripts/coccinelle/misc/doublereturn.cocci
new file mode 100644
index 0000000..656a118
--- /dev/null
+++ b/scripts/coccinelle/misc/doublereturn.cocci
@@ -0,0 +1,20 @@
+/// Remove unecessary if/return in code that follows this pattern:
+/// if(retval)
+/// return retval;
+/// return retval;
+//
+// Confidence: High
+// Copyright: (C) 2011 Greg Dietsche GPLv2.
+// URL: http://www.gregd.org
+// Comments:
+// Options: -no_includes
+
+virtual patch
+
+@@
+identifier retval;
+@@
+-if (retval)
+- return retval;
+-return retval;
++return retval;
--
1.7.2.5
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists