Commit dee87445 authored by Colin Ian King's avatar Colin Ian King Committed by Dave Kleikamp
Browse files

jfs: remove redundant assignments to ipaimap and ipaimap2 



The pointers ipaimap and ipaimap2 are re-assigned with values a second
time with the same values when they were initialized. The re-assignments
are redundant and can be removed.

Cleans up two clang scan build warnings:
fs/jfs/jfs_umount.c:42:16: warning: Value stored to 'ipaimap' during
its initialization is never read [deadcode.DeadStores]

fs/jfs/jfs_umount.c:43:16: warning: Value stored to 'ipaimap2' during
its initialization is never read [deadcode.DeadStores]

Signed-off-by: default avatarColin Ian King <colin.i.king@gmail.com>
Signed-off-by: default avatarDave Kleikamp <dave.kleikamp@oracle.com>
parent 1ea66d71

fs/jfs/jfs_umount.c

+0 −2
Original line number Diff line number Diff line
@@ -68,7 +68,6 @@ int jfs_umount(struct super_block *sb) 
	/*

	 * close secondary aggregate inode allocation map

	 */

	ipaimap2 = sbi->ipaimap2;

	if (ipaimap2) {

		diUnmount(ipaimap2, 0);

		diFreeSpecial(ipaimap2);
@@ -78,7 +77,6 @@ int jfs_umount(struct super_block *sb) 
	/*

	 * close aggregate inode allocation map

	 */

	ipaimap = sbi->ipaimap;

	diUnmount(ipaimap, 0);

	diFreeSpecial(ipaimap);

	sbi->ipaimap = NULL;