[PATCH] vfat:remove unused variable

[PATCH] vfat:remove unused variable

[zhoumin]

Remove unused variable definition and related function definition
and redundant variable assignments within functions.

Signed-off-by: zhoumin <teczm@foxmail.com>
---
 fs/fat/dir.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/fs/fat/dir.c b/fs/fat/dir.c
index acbec5bdd521..92b091783966 100644
--- a/fs/fat/dir.c
+++ b/fs/fat/dir.c
@@ -1209,7 +1209,7 @@ EXPORT_SYMBOL_GPL(fat_alloc_new_dir);
 
 static int fat_add_new_entries(struct inode *dir, void *slots, int nr_slots,
 			       int *nr_cluster, struct msdos_dir_entry **de,
-			       struct buffer_head **bh, loff_t *i_pos)
+			       struct buffer_head **bh)
 {
 	struct super_block *sb = dir->i_sb;
 	struct msdos_sb_info *sbi = MSDOS_SB(sb);
@@ -1269,7 +1269,6 @@ static int fat_add_new_entries(struct inode *dir, void *slots, int nr_slots,
 	get_bh(bhs[n]);
 	*bh = bhs[n];
 	*de = (struct msdos_dir_entry *)((*bh)->b_data + offset);
-	*i_pos = fat_make_i_pos(sb, *bh, *de);
 
 	/* Second stage: clear the rest of cluster, and write outs */
 	err = fat_zeroed_cluster(dir, start_blknr, ++n, bhs, MAX_BUF_PER_PAGE);
@@ -1298,7 +1297,7 @@ int fat_add_entries(struct inode *dir, void *slots, int nr_slots,
 	struct buffer_head *bh, *prev, *bhs[3]; /* 32*slots (672bytes) */
 	struct msdos_dir_entry *de;
 	int err, free_slots, i, nr_bhs;
-	loff_t pos, i_pos;
+	loff_t pos;
 
 	sinfo->nr_slots = nr_slots;
 
@@ -1386,7 +1385,7 @@ int fat_add_entries(struct inode *dir, void *slots, int nr_slots,
 		 * add the cluster to dir.
 		 */
 		cluster = fat_add_new_entries(dir, slots, nr_slots, &nr_cluster,
-					      &de, &bh, &i_pos);
+					      &de, &bh);
 		if (cluster < 0) {
 			err = cluster;
 			goto error_remove;
-- 
2.43.0

Re: [PATCH] vfat:remove unused variable

[OGAWA Hirofumi]

zhoumin <teczm@foxmail.com> writes:

> Remove unused variable definition and related function definition
> and redundant variable assignments within functions.
>
> Signed-off-by: zhoumin <teczm@foxmail.com>

Looks good. Thanks.

Acked-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>

> ---
>  fs/fat/dir.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/fs/fat/dir.c b/fs/fat/dir.c
> index acbec5bdd521..92b091783966 100644
> --- a/fs/fat/dir.c
> +++ b/fs/fat/dir.c
> @@ -1209,7 +1209,7 @@ EXPORT_SYMBOL_GPL(fat_alloc_new_dir);
>  
>  static int fat_add_new_entries(struct inode *dir, void *slots, int nr_slots,
>  			       int *nr_cluster, struct msdos_dir_entry **de,
> -			       struct buffer_head **bh, loff_t *i_pos)
> +			       struct buffer_head **bh)
>  {
>  	struct super_block *sb = dir->i_sb;
>  	struct msdos_sb_info *sbi = MSDOS_SB(sb);
> @@ -1269,7 +1269,6 @@ static int fat_add_new_entries(struct inode *dir, void *slots, int nr_slots,
>  	get_bh(bhs[n]);
>  	*bh = bhs[n];
>  	*de = (struct msdos_dir_entry *)((*bh)->b_data + offset);
> -	*i_pos = fat_make_i_pos(sb, *bh, *de);
>  
>  	/* Second stage: clear the rest of cluster, and write outs */
>  	err = fat_zeroed_cluster(dir, start_blknr, ++n, bhs, MAX_BUF_PER_PAGE);
> @@ -1298,7 +1297,7 @@ int fat_add_entries(struct inode *dir, void *slots, int nr_slots,
>  	struct buffer_head *bh, *prev, *bhs[3]; /* 32*slots (672bytes) */
>  	struct msdos_dir_entry *de;
>  	int err, free_slots, i, nr_bhs;
> -	loff_t pos, i_pos;
> +	loff_t pos;
>  
>  	sinfo->nr_slots = nr_slots;
>  
> @@ -1386,7 +1385,7 @@ int fat_add_entries(struct inode *dir, void *slots, int nr_slots,
>  		 * add the cluster to dir.
>  		 */
>  		cluster = fat_add_new_entries(dir, slots, nr_slots, &nr_cluster,
> -					      &de, &bh, &i_pos);
> +					      &de, &bh);
>  		if (cluster < 0) {
>  			err = cluster;
>  			goto error_remove;

-- 
OGAWA Hirofumi <hirofumi@mail.parknet.co.jp>