staging: lustre: obdclass: expand double GOTO macros
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sun, 7 Sep 2014 16:18:33 +0000 (18:18 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 8 Sep 2014 19:46:32 +0000 (12:46 -0700)
The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
identifier lbl;
@@

if (...)
  GOTO(lbl,...);
+else
GOTO(lbl,...);

@@
identifier lbl;
expression e,e1,e2;
@@

if (e)
-  GOTO(lbl,e1);
-else GOTO(lbl,e2);
+  e1;
+else e2;
+goto lbl;

@@
identifier lbl1,lbl2;
expression e,e1,e2;
@@

+ e1=e2;
if (e)
-  GOTO(lbl1,e1=e2);
-GOTO(lbl2,e1=e2);
+  goto lbl1;
+goto lbl2;
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/obdclass/dt_object.c
drivers/staging/lustre/lustre/obdclass/llog.c

index 130b8dd..4060173 100644 (file)
@@ -689,8 +689,10 @@ static int dt_index_page_build(const struct lu_env *env, union lu_page *lp,
 
                if (nob < size) {
                        if (lip->lip_nr == 0)
-                               GOTO(out, rc = -EINVAL);
-                       GOTO(out, rc = 0);
+                               rc = -EINVAL;
+                       else
+                               rc = 0;
+                       goto out;
                }
 
                if ((ii->ii_flags & II_FL_NOHASH) == 0) {
index ebdc42f..1cf604d 100644 (file)
@@ -336,9 +336,10 @@ repeat:
 
                        if (rec->lrh_index == 0) {
                                /* probably another rec just got added? */
+                               rc = 0;
                                if (index <= loghandle->lgh_last_idx)
-                                       GOTO(repeat, rc = 0);
-                               GOTO(out, rc = 0); /* no more records */
+                                       goto repeat;
+                               goto out; /* no more records */
                        }
                        if (rec->lrh_len == 0 ||
                            rec->lrh_len > LLOG_CHUNK_SIZE) {