[tor-commits] [tor/master] Remove variables; fix gcc 4.7 warnings
andrea at torproject.org
andrea at torproject.org
Thu Oct 11 02:05:24 UTC 2012
commit bd28322d3815a03ca04beb6d00052d613dbe226f
Author: Nick Mathewson <nickm at torproject.org>
Date: Wed Oct 10 21:25:52 2012 -0400
Remove variables; fix gcc 4.7 warnings
My GCC warns when variables are assigned to but never used. There
were a few like that in the 6816/6465 branches.
---
src/or/channel.c | 3 +--
src/or/channeltls.c | 3 ---
src/or/circuitlist.c | 4 +---
src/or/circuitmux_ewma.c | 5 -----
4 files changed, 2 insertions(+), 13 deletions(-)
diff --git a/src/or/channel.c b/src/or/channel.c
index 570bb37..b52db40 100644
--- a/src/or/channel.c
+++ b/src/or/channel.c
@@ -552,7 +552,7 @@ channel_add_to_digest_map(channel_t *chan)
static void
channel_remove_from_digest_map(channel_t *chan)
{
- channel_t *tmp, *head;
+ channel_t *tmp;
tor_assert(chan);
@@ -588,7 +588,6 @@ channel_remove_from_digest_map(channel_t *chan)
tmp = digestmap_get(channel_identity_map, chan->identity_digest);
if (tmp) {
/* Okay, it's here */
- head = tmp; /* Keep track of list head */
/* Look for this channel */
while (tmp && tmp != chan) {
tmp = tmp->next_with_same_id;
diff --git a/src/or/channeltls.c b/src/or/channeltls.c
index 764de80..1a2956b 100644
--- a/src/or/channeltls.c
+++ b/src/or/channeltls.c
@@ -917,7 +917,6 @@ void
channel_tls_handle_var_cell(var_cell_t *var_cell, or_connection_t *conn)
{
channel_tls_t *chan;
- int handshaking;
#ifdef KEEP_TIMING_STATS
/* how many of each cell have we seen so far this second? needs better
@@ -953,8 +952,6 @@ channel_tls_handle_var_cell(var_cell_t *var_cell, or_connection_t *conn)
return;
}
- handshaking = (TO_CONN(conn)->state != OR_CONN_STATE_OPEN);
-
if (TO_CONN(conn)->marked_for_close)
return;
diff --git a/src/or/circuitlist.c b/src/or/circuitlist.c
index 5140600..b738226 100644
--- a/src/or/circuitlist.c
+++ b/src/or/circuitlist.c
@@ -100,18 +100,16 @@ circuit_set_circid_chan_helper(circuit_t *circ, int direction,
chan_circid_circuit_map_t *found;
channel_t *old_chan, **chan_ptr;
circid_t old_id, *circid_ptr;
- int was_active, make_active, attached = 0;
+ int make_active, attached = 0;
if (direction == CELL_DIRECTION_OUT) {
chan_ptr = &circ->n_chan;
circid_ptr = &circ->n_circ_id;
- was_active = circ->next_active_on_n_chan != NULL;
make_active = circ->n_chan_cells.n > 0;
} else {
or_circuit_t *c = TO_OR_CIRCUIT(circ);
chan_ptr = &c->p_chan;
circid_ptr = &c->p_circ_id;
- was_active = c->next_active_on_p_chan != NULL;
make_active = c->p_chan_cells.n > 0;
}
old_chan = *chan_ptr;
diff --git a/src/or/circuitmux_ewma.c b/src/or/circuitmux_ewma.c
index 82ad5c7..c280bf4 100644
--- a/src/or/circuitmux_ewma.c
+++ b/src/or/circuitmux_ewma.c
@@ -265,7 +265,6 @@ ewma_alloc_circ_data(circuitmux_t *cmux,
cell_direction_t direction,
unsigned int cell_count)
{
- ewma_policy_data_t *pol = NULL;
ewma_policy_circ_data_t *cdata = NULL;
tor_assert(cmux);
@@ -276,8 +275,6 @@ ewma_alloc_circ_data(circuitmux_t *cmux,
/* Shut the compiler up */
tor_assert(cell_count == cell_count);
- pol = TO_EWMA_POL_DATA(pol_data);
-
cdata = tor_malloc_zero(sizeof(*cdata));
cdata->_base.magic = EWMA_POL_CIRC_DATA_MAGIC;
cdata->circ = circ;
@@ -309,13 +306,11 @@ ewma_free_circ_data(circuitmux_t *cmux,
circuitmux_policy_circ_data_t *pol_circ_data)
{
- ewma_policy_data_t *pol = NULL;
ewma_policy_circ_data_t *cdata = NULL;
tor_assert(cmux);
tor_assert(circ);
tor_assert(pol_data);
- pol = TO_EWMA_POL_DATA(pol_data);
if (!pol_circ_data) return;
More information about the tor-commits
mailing list