diff options
author | Humble Devassy Chirammal <hchiramm@redhat.com> | 2015-03-09 19:12:03 +0530 |
---|---|---|
committer | Niels de Vos <ndevos@redhat.com> | 2015-03-17 02:02:09 -0700 |
commit | fb6858b47585244fd3cef2f0e8155a13752e0365 (patch) | |
tree | 885b5245313e79f956c4d88a20c7a781dcbd5cab | |
parent | 71210c021f7b6d7b79f31553d7bd8632d1f6e99f (diff) |
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 <hchiramm@redhat.com>
Reviewed-on: http://review.gluster.org/9846
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Anoop C S <achiraya@redhat.com>
Reviewed-by: Niels de Vos <ndevos@redhat.com>
Reviewed-by: Poornima G <pgurusid@redhat.com>
-rw-r--r-- | libglusterfs/src/inode.c | 7 |
1 files changed, 2 insertions, 5 deletions
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 |