diff mbox

hfs/hfsplus: Clean up unused variables in bnode.c

Message ID 1507977146-15875-1-git-send-email-chris.gekas@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Christos Gkekas Oct. 14, 2017, 10:32 a.m. UTC 
Delete variables 'tree' and 'sb', which are set but never used.

Signed-off-by: Christos Gkekas <chris.gekas@gmail.com>
---
 fs/hfs/bnode.c     | 4 ----
 fs/hfsplus/bnode.c | 4 ----
 2 files changed, 8 deletions(-)

Comments

Ernesto A. Fernández Oct. 18, 2017, 4:10 a.m. UTC | #1 
On Sat, Oct 14, 2017 at 11:32:26AM +0100, Christos Gkekas wrote:
> Delete variables 'tree' and 'sb', which are set but never used.
> 
> Signed-off-by: Christos Gkekas <chris.gekas@gmail.com>

Looks good. If it helps you can add:

Reviewed-by: Ernesto A. Fernández <ernesto.mnd.fernandez@gmail.com>

> ---
>  fs/hfs/bnode.c     | 4 ----
>  fs/hfsplus/bnode.c | 4 ----
>  2 files changed, 8 deletions(-)
> 
> diff --git a/fs/hfs/bnode.c b/fs/hfs/bnode.c
> index d77d844..8d618c9 100644
> --- a/fs/hfs/bnode.c
> +++ b/fs/hfs/bnode.c
> @@ -97,13 +97,11 @@ void hfs_bnode_clear(struct hfs_bnode *node, int off, int len)
>  void hfs_bnode_copy(struct hfs_bnode *dst_node, int dst,
>  		struct hfs_bnode *src_node, int src, int len)
>  {
> -	struct hfs_btree *tree;
>  	struct page *src_page, *dst_page;
>  
>  	hfs_dbg(BNODE_MOD, "copybytes: %u,%u,%u\n", dst, src, len);
>  	if (!len)
>  		return;
> -	tree = src_node->tree;
>  	src += src_node->page_offset;
>  	dst += dst_node->page_offset;
>  	src_page = src_node->page[0];
> @@ -236,7 +234,6 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
>  
>  static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
>  {
> -	struct super_block *sb;
>  	struct hfs_bnode *node, *node2;
>  	struct address_space *mapping;
>  	struct page *page;
> @@ -248,7 +245,6 @@ static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
>  		return NULL;
>  	}
>  
> -	sb = tree->inode->i_sb;
>  	size = sizeof(struct hfs_bnode) + tree->pages_per_bnode *
>  		sizeof(struct page *);
>  	node = kzalloc(size, GFP_KERNEL);
> diff --git a/fs/hfsplus/bnode.c b/fs/hfsplus/bnode.c
> index ce014ce..9cd8b42 100644
> --- a/fs/hfsplus/bnode.c
> +++ b/fs/hfsplus/bnode.c
> @@ -126,14 +126,12 @@ void hfs_bnode_clear(struct hfs_bnode *node, int off, int len)
>  void hfs_bnode_copy(struct hfs_bnode *dst_node, int dst,
>  		    struct hfs_bnode *src_node, int src, int len)
>  {
> -	struct hfs_btree *tree;
>  	struct page **src_page, **dst_page;
>  	int l;
>  
>  	hfs_dbg(BNODE_MOD, "copybytes: %u,%u,%u\n", dst, src, len);
>  	if (!len)
>  		return;
> -	tree = src_node->tree;
>  	src += src_node->page_offset;
>  	dst += dst_node->page_offset;
>  	src_page = src_node->page + (src >> PAGE_SHIFT);
> @@ -400,7 +398,6 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
>  
>  static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
>  {
> -	struct super_block *sb;
>  	struct hfs_bnode *node, *node2;
>  	struct address_space *mapping;
>  	struct page *page;
> @@ -413,7 +410,6 @@ static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
>  		return NULL;
>  	}
>  
> -	sb = tree->inode->i_sb;
>  	size = sizeof(struct hfs_bnode) + tree->pages_per_bnode *
>  		sizeof(struct page *);
>  	node = kzalloc(size, GFP_KERNEL);
> -- 
> 2.7.4
>
diff mbox

Patch

diff --git a/fs/hfs/bnode.c b/fs/hfs/bnode.c
index d77d844..8d618c9 100644
--- a/fs/hfs/bnode.c
+++ b/fs/hfs/bnode.c
@@ -97,13 +97,11 @@  void hfs_bnode_clear(struct hfs_bnode *node, int off, int len)
 void hfs_bnode_copy(struct hfs_bnode *dst_node, int dst,
 		struct hfs_bnode *src_node, int src, int len)
 {
-	struct hfs_btree *tree;
 	struct page *src_page, *dst_page;
 
 	hfs_dbg(BNODE_MOD, "copybytes: %u,%u,%u\n", dst, src, len);
 	if (!len)
 		return;
-	tree = src_node->tree;
 	src += src_node->page_offset;
 	dst += dst_node->page_offset;
 	src_page = src_node->page[0];
@@ -236,7 +234,6 @@  struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
 
 static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 {
-	struct super_block *sb;
 	struct hfs_bnode *node, *node2;
 	struct address_space *mapping;
 	struct page *page;
@@ -248,7 +245,6 @@  static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 		return NULL;
 	}
 
-	sb = tree->inode->i_sb;
 	size = sizeof(struct hfs_bnode) + tree->pages_per_bnode *
 		sizeof(struct page *);
 	node = kzalloc(size, GFP_KERNEL);
diff --git a/fs/hfsplus/bnode.c b/fs/hfsplus/bnode.c
index ce014ce..9cd8b42 100644
--- a/fs/hfsplus/bnode.c
+++ b/fs/hfsplus/bnode.c
@@ -126,14 +126,12 @@  void hfs_bnode_clear(struct hfs_bnode *node, int off, int len)
 void hfs_bnode_copy(struct hfs_bnode *dst_node, int dst,
 		    struct hfs_bnode *src_node, int src, int len)
 {
-	struct hfs_btree *tree;
 	struct page **src_page, **dst_page;
 	int l;
 
 	hfs_dbg(BNODE_MOD, "copybytes: %u,%u,%u\n", dst, src, len);
 	if (!len)
 		return;
-	tree = src_node->tree;
 	src += src_node->page_offset;
 	dst += dst_node->page_offset;
 	src_page = src_node->page + (src >> PAGE_SHIFT);
@@ -400,7 +398,6 @@  struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid)
 
 static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 {
-	struct super_block *sb;
 	struct hfs_bnode *node, *node2;
 	struct address_space *mapping;
 	struct page *page;
@@ -413,7 +410,6 @@  static struct hfs_bnode *__hfs_bnode_create(struct hfs_btree *tree, u32 cnid)
 		return NULL;
 	}
 
-	sb = tree->inode->i_sb;
 	size = sizeof(struct hfs_bnode) + tree->pages_per_bnode *
 		sizeof(struct page *);
 	node = kzalloc(size, GFP_KERNEL);