diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2016-09-03 01:35:29 -0400 |
---|---|---|
committer | Dave Kleikamp <dave.kleikamp@oracle.com> | 2016-09-06 13:17:24 -0400 |
commit | 240c5185c52d00725ae7da7d1ee82ad8b8306d52 (patch) | |
tree | b61b90fcd932e17ab0ef70e544f4c5fec670a522 | |
parent | 7cfcd8b79ab2472102d9229863e86ff208f959d4 (diff) |
jfs: Simplify code
Calling 'list_splice' followed by 'INIT_LIST_HEAD' is equivalent to
'list_splice_init'.
This has been spotted with the following coccinelle script:
/////
@@
expression y,z;
@@
- list_splice(y,z);
- INIT_LIST_HEAD(y);
+ list_splice_init(y,z);
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
-rw-r--r-- | fs/jfs/jfs_txnmgr.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jfs/jfs_txnmgr.c b/fs/jfs/jfs_txnmgr.c index 2e58978d6f45..4d973524c887 100644 --- a/fs/jfs/jfs_txnmgr.c +++ b/fs/jfs/jfs_txnmgr.c | |||
@@ -2893,8 +2893,7 @@ restart: | |||
2893 | * on anon_list2. Let's check. | 2893 | * on anon_list2. Let's check. |
2894 | */ | 2894 | */ |
2895 | if (!list_empty(&TxAnchor.anon_list2)) { | 2895 | if (!list_empty(&TxAnchor.anon_list2)) { |
2896 | list_splice(&TxAnchor.anon_list2, &TxAnchor.anon_list); | 2896 | list_splice_init(&TxAnchor.anon_list2, &TxAnchor.anon_list); |
2897 | INIT_LIST_HEAD(&TxAnchor.anon_list2); | ||
2898 | goto restart; | 2897 | goto restart; |
2899 | } | 2898 | } |
2900 | TXN_UNLOCK(); | 2899 | TXN_UNLOCK(); |