Commit 4d2a71d0 authored Feb 24, 2016 by Amitoj Kaur Chawla Committed by Greg Kroah-Hartman Feb 25, 2016

staging: lustre: libcfs: Use swap() in cfs_hash_bd_order()



Use swap() function instead of using a temporary variable for swapping
two variables.

The Coccinelle semantic patch used to make this change is as follows:
//<smpl>
@@
type T;
T a,b,c;
@@
- a = b;
- b = c;
- c = a;
+ swap(b, c);
//<smpl>

Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent cb776592

drivers/staging/lustre/lustre/libcfs/hash.c

+2 −6

Original line number	Diff line number	Diff line
		@@ -803,12 +803,8 @@ cfs_hash_bd_order(struct cfs_hash_bd bd1, struct cfs_hash_bd bd2)
		if (rc == 0) {
		bd2->bd_bucket = NULL;

		} else if (rc > 0) { /* swab bd1 and bd2 */
		struct cfs_hash_bd tmp;

		tmp = *bd2;
		bd2 = bd1;
		*bd1 = tmp;
		} else if (rc > 0) {
		swap(bd1, bd2); /* swap bd1 and bd2 */
		}
		}