staging: Drop unnecessary null test
authorJulia Lawall <julia@diku.dk>
Tue, 3 Aug 2010 21:34:36 +0000 (23:34 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 31 Aug 2010 18:13:15 +0000 (11:13 -0700)
list_for_each_entry binds its first argument to a non-null value, and thus
any null test on the value of that argument is superfluous.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
iterator I;
expression x;
statement S;
@@

I(x,...) { <...
- if (x != NULL || ...)
  S
  ...> }
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/pohmelfs/config.c
drivers/staging/tm6000/tm6000-core.c

index 8c8d1c282e7e694e50d2afd252621bc36725160f..89279ba1b7378aee57230d1964d072fb4b38e703 100644 (file)
@@ -301,10 +301,8 @@ static int pohmelfs_cn_dump(struct cn_msg *msg)
 
        mutex_lock(&pohmelfs_config_lock);
 
-       list_for_each_entry(g, &pohmelfs_config_list, group_entry) {
-               if (g)
-                       total_msg += g->num_entry;
-       }
+       list_for_each_entry(g, &pohmelfs_config_list, group_entry)
+               total_msg += g->num_entry;
        if (total_msg == 0) {
                if (pohmelfs_send_reply(err, 0, POHMELFS_NOINFO_ACK, msg, NULL))
                        err = -ENOMEM;
@@ -312,15 +310,16 @@ static int pohmelfs_cn_dump(struct cn_msg *msg)
        }
 
        list_for_each_entry(g, &pohmelfs_config_list, group_entry) {
-               if (g) {
-                       list_for_each_entry_safe(c, tmp, &g->config_list, config_entry) {
-                               struct pohmelfs_ctl *sc = &c->state.ctl;
-                               if (pohmelfs_send_reply(err, total_msg - i, POHMELFS_CTLINFO_ACK, msg, sc)) {
-                                       err = -ENOMEM;
-                                       goto out_unlock;
-                               }
-                               i += 1;
+               list_for_each_entry_safe(c, tmp, &g->config_list,
+                                        config_entry) {
+                       struct pohmelfs_ctl *sc = &c->state.ctl;
+                       if (pohmelfs_send_reply(err, total_msg - i,
+                                               POHMELFS_CTLINFO_ACK, msg,
+                                               sc)) {
+                               err = -ENOMEM;
+                               goto out_unlock;
                        }
+                       i += 1;
                }
        }
 
@@ -354,12 +353,11 @@ static int pohmelfs_cn_flush(struct cn_msg *msg)
                }
        } else {
                list_for_each_entry(g, &pohmelfs_config_list, group_entry) {
-                       if (g) {
-                               list_for_each_entry_safe(c, tmp, &g->config_list, config_entry) {
-                                       list_del(&c->config_entry);
-                                       g->num_entry--;
-                                       kfree(c);
-                               }
+                       list_for_each_entry_safe(c, tmp, &g->config_list,
+                                                config_entry) {
+                               list_del(&c->config_entry);
+                               g->num_entry--;
+                               kfree(c);
                        }
                }
        }
index cded411d8bba552eff047f068eefbf9adb3720d4..80f2bf084505469a6728cfc839cffcc4edc3c057 100644 (file)
@@ -702,10 +702,8 @@ void tm6000_unregister_extension(struct tm6000_ops *ops)
        struct tm6000_core *dev = NULL;
 
        mutex_lock(&tm6000_devlist_mutex);
-       list_for_each_entry(dev, &tm6000_devlist, devlist) {
-               if (dev)
-                       ops->fini(dev);
-       }
+       list_for_each_entry(dev, &tm6000_devlist, devlist)
+               ops->fini(dev);
 
        mutex_lock(&tm6000_extension_devlist_lock);
        printk(KERN_INFO "tm6000: Remove (%s) extension\n", ops->name);