summaryrefslogtreecommitdiffstats
path: root/libglusterfs/src
diff options
context:
space:
mode:
authorJeff Darcy <jdarcy@redhat.com>2011-09-07 20:03:24 -0400
committerAnand Avati <avati@gluster.com>2011-09-07 23:48:01 -0700
commit694ef54978f382507a5127ce66da7770929ba2c2 (patch)
treeb98ee679c8d5f4b3556c0bf9af44e6b9729c2881 /libglusterfs/src
parent81530d227deb52af38c7df770aef2200b9de539f (diff)
Eliminate many "var set but not used" warnings with newer gcc.
This fixes ~200 such warnings, but leaves three categories untouched. (1) Rpcgen code. (2) Macros which set variables in the outer (calling function) scope. (3) Variables which are set via function calls which may have side effects. Change-Id: I6554555f78ed26134251504b038da7e94adacbcd BUG: 2550 Reviewed-on: http://review.gluster.com/371 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@gluster.com>
Diffstat (limited to 'libglusterfs/src')
-rw-r--r--libglusterfs/src/call-stub.c6
-rw-r--r--libglusterfs/src/fd.c4
-rw-r--r--libglusterfs/src/graph.c4
-rw-r--r--libglusterfs/src/graph.y5
-rw-r--r--libglusterfs/src/inode.c10
5 files changed, 0 insertions, 29 deletions
diff --git a/libglusterfs/src/call-stub.c b/libglusterfs/src/call-stub.c
index 67543c64a..8761b40a0 100644
--- a/libglusterfs/src/call-stub.c
+++ b/libglusterfs/src/call-stub.c
@@ -3822,12 +3822,8 @@ call_stub_destroy_unwind (call_stub_t *stub)
void
call_stub_destroy (call_stub_t *stub)
{
- struct mem_pool *tmp_pool = NULL;
-
GF_VALIDATE_OR_GOTO ("call-stub", stub, out);
- tmp_pool = stub->stub_mem_pool;
-
if (stub->wind) {
call_stub_destroy_wind (stub);
} else {
@@ -3837,8 +3833,6 @@ call_stub_destroy (call_stub_t *stub)
stub->stub_mem_pool = NULL;
mem_put (stub);
out:
- tmp_pool = NULL;
-
return;
}
diff --git a/libglusterfs/src/fd.c b/libglusterfs/src/fd.c
index d6c4806a1..7779dfe25 100644
--- a/libglusterfs/src/fd.c
+++ b/libglusterfs/src/fd.c
@@ -383,7 +383,6 @@ fd_destroy (fd_t *fd)
xlator_t *xl = NULL;
int i = 0;
xlator_t *old_THIS = NULL;
- struct mem_pool *tmp_pool = NULL;
if (fd == NULL){
gf_log_callingfn ("xlator", GF_LOG_ERROR, "invalid arugument");
@@ -397,8 +396,6 @@ fd_destroy (fd_t *fd)
if (!fd->_ctx)
goto out;
- tmp_pool = fd->inode->table->fd_mem_pool;
-
if (IA_ISDIR (fd->inode->ia_type)) {
for (i = 0; i < fd->xl_count; i++) {
if (fd->_ctx[i].key) {
@@ -429,7 +426,6 @@ fd_destroy (fd_t *fd)
inode_unref (fd->inode);
fd->inode = (inode_t *)0xaaaaaaaa;
mem_put (fd);
- tmp_pool = NULL;
out:
return;
}
diff --git a/libglusterfs/src/graph.c b/libglusterfs/src/graph.c
index 1d7d943e6..7844a5475 100644
--- a/libglusterfs/src/graph.c
+++ b/libglusterfs/src/graph.c
@@ -283,7 +283,6 @@ gf_add_cmdline_options (glusterfs_graph_t *graph, cmd_args_t *cmd_args)
int
glusterfs_graph_validate_options (glusterfs_graph_t *graph)
{
- volume_opt_list_t *vol_opt = NULL;
xlator_t *trav = NULL;
int ret = -1;
char *errstr = NULL;
@@ -294,9 +293,6 @@ glusterfs_graph_validate_options (glusterfs_graph_t *graph)
if (list_empty (&trav->volume_options))
continue;
- vol_opt = list_entry (trav->volume_options.next,
- volume_opt_list_t, list);
-
ret = xlator_options_validate (trav, trav->options, &errstr);
if (ret) {
gf_log (trav->name, GF_LOG_ERROR,
diff --git a/libglusterfs/src/graph.y b/libglusterfs/src/graph.y
index 10aa4546b..62c57dd6c 100644
--- a/libglusterfs/src/graph.y
+++ b/libglusterfs/src/graph.y
@@ -445,8 +445,6 @@ preprocess (FILE *srcfp, FILE *dstfp)
char in_backtick = 0;
int line = 1;
int column = 0;
- int backtick_line = 0;
- int backtick_column = 0;
int character = 0;
@@ -484,9 +482,6 @@ preprocess (FILE *srcfp, FILE *dstfp)
} else {
i = 0;
cmd[i] = '\0';
-
- backtick_column = column;
- backtick_line = line;
}
in_backtick = !in_backtick;
diff --git a/libglusterfs/src/inode.c b/libglusterfs/src/inode.c
index 1b4320311..7f9382367 100644
--- a/libglusterfs/src/inode.c
+++ b/libglusterfs/src/inode.c
@@ -134,14 +134,11 @@ __dentry_unhash (dentry_t *dentry)
static void
__dentry_unset (dentry_t *dentry)
{
- struct mem_pool *tmp_pool = NULL;
-
if (!dentry) {
gf_log_callingfn (THIS->name, GF_LOG_WARNING, "dentry not found");
return;
}
- tmp_pool = dentry->inode->table->dentry_pool;
__dentry_unhash (dentry);
list_del_init (&dentry->inode_list);
@@ -155,8 +152,6 @@ __dentry_unset (dentry_t *dentry)
}
mem_put (dentry);
- tmp_pool = NULL;
-
}
@@ -307,7 +302,6 @@ __inode_destroy (inode_t *inode)
int index = 0;
xlator_t *xl = NULL;
xlator_t *old_THIS = NULL;
- struct mem_pool *tmp_pool = NULL;
if (!inode) {
gf_log_callingfn (THIS->name, GF_LOG_WARNING, "inode not found");
@@ -319,8 +313,6 @@ __inode_destroy (inode_t *inode)
goto noctx;
}
- tmp_pool = inode->table->inode_pool;
-
for (index = 0; index < inode->table->xl->graph->xl_count; index++) {
if (inode->_ctx[index].xl_key) {
xl = (xlator_t *)(long)inode->_ctx[index].xl_key;
@@ -337,8 +329,6 @@ noctx:
LOCK_DESTROY (&inode->lock);
// memset (inode, 0xb, sizeof (*inode));
mem_put (inode);
- tmp_pool = NULL;
-
}