From fb6858b47585244fd3cef2f0e8155a13752e0365 Mon Sep 17 00:00:00 2001 From: Humble Devassy Chirammal Date: Mon, 9 Mar 2015 19:12:03 +0530 Subject: inode: 'this' has been set unwantedly. Problem: CC libglusterfs_la-inode.lo inode.c: In function 'inode_table_destroy': inode.c:1630:19: warning: variable 'this' set but not used [-Wunused-but-set-variable] xlator_t *this = NULL; Change-Id: If4b37ab896ee0a309826d4be48c6599d6ec2710b Signed-off-by: Humble Devassy Chirammal Reviewed-on: http://review.gluster.org/9846 Tested-by: Gluster Build System Reviewed-by: Anoop C S Reviewed-by: Niels de Vos Reviewed-by: Poornima G --- libglusterfs/src/inode.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) (limited to 'libglusterfs/src') diff --git a/libglusterfs/src/inode.c b/libglusterfs/src/inode.c index a6b184fc5d9..a8958fa3bcc 100644 --- a/libglusterfs/src/inode.c +++ b/libglusterfs/src/inode.c @@ -1627,11 +1627,8 @@ inode_table_destroy_all (glusterfs_ctx_t *ctx) { void inode_table_destroy (inode_table_t *inode_table) { - xlator_t *this = NULL; inode_t *tmp = NULL, *trav = NULL; - this = THIS; - if (inode_table == NULL) return; @@ -1642,8 +1639,8 @@ inode_table_destroy (inode_table_t *inode_table) { * in lru list.(If no leaks there should be no inodes in active list) * 2. Knowing there could be leaks and not freeing those inodes will * also not free its inode context and this could leak a lot of - * memory, force free the inodes by changeing the ref to 0. - * The problem with this is that any refence ti inode after this + * memory, force free the inodes by changing the ref to 0. + * The problem with this is that any reference to inode after this * calling this funtion will lead to a crash. * 3. Knowing there could be leakes, just free the inode contexts of * all the inodes. and let the inodes be alive. This way the major -- cgit