* [PATCH] ext4: delete unused mapping variable
@ 2013-06-13 13:48 Paul Gortmaker
2013-06-13 14:19 ` Theodore Ts'o
0 siblings, 1 reply; 2+ messages in thread
From: Paul Gortmaker @ 2013-06-13 13:48 UTC (permalink / raw)
To: Theodore Ts'o; +Cc: linux-ext4, Paul Gortmaker
The code refactoring in commit "ext4: refactor punch hole code"
left behind a now unused variable, causing:
fs/ext4/inode.c: In function ‘ext4_punch_hole’:
fs/ext4/inode.c:3495:24: warning: unused variable ‘mapping’ [-Wunused-variable]
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
[Feel free to also fold this trivial change back into the original
punch hole commit on the dev branch]
fs/ext4/inode.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
index 4411130..2644679 100644
--- a/fs/ext4/inode.c
+++ b/fs/ext4/inode.c
@@ -3492,7 +3492,6 @@ int ext4_punch_hole(struct file *file, loff_t offset, loff_t length)
struct inode *inode = file_inode(file);
struct super_block *sb = inode->i_sb;
ext4_lblk_t first_block, stop_block;
- struct address_space *mapping = inode->i_mapping;
loff_t first_block_offset, last_block_offset;
handle_t *handle;
unsigned int credits;
--
1.8.1.2
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ext4: delete unused mapping variable
2013-06-13 13:48 [PATCH] ext4: delete unused mapping variable Paul Gortmaker
@ 2013-06-13 14:19 ` Theodore Ts'o
0 siblings, 0 replies; 2+ messages in thread
From: Theodore Ts'o @ 2013-06-13 14:19 UTC (permalink / raw)
To: Paul Gortmaker; +Cc: linux-ext4
On Thu, Jun 13, 2013 at 09:48:18AM -0400, Paul Gortmaker wrote:
> The code refactoring in commit "ext4: refactor punch hole code"
> left behind a now unused variable, causing:
>
> fs/ext4/inode.c: In function ‘ext4_punch_hole’:
> fs/ext4/inode.c:3495:24: warning: unused variable ‘mapping’ [-Wunused-variable]
>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Thanks, I had actually noticed this already, but I had pushed out the
wrong version to the dev branch.
- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-06-13 14:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-13 13:48 [PATCH] ext4: delete unused mapping variable Paul Gortmaker
2013-06-13 14:19 ` Theodore Ts'o
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).