ctdb-recoverd: Remove unused struct members num_active and num_connected
authorMartin Schwenke <martin@meltin.net>
Sun, 29 Mar 2015 06:28:57 +0000 (17:28 +1100)
committerAmitay Isaacs <amitay@samba.org>
Sun, 10 May 2015 01:22:13 +0000 (03:22 +0200)
They are initialised and updated but the values are never used.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
ctdb/server/ctdb_recoverd.c

index 53b1467a89ae413074e6df0ffc58ccf026fb5609..49d69f09c7457daababd7641e8998c46712089ad 100644 (file)
@@ -225,9 +225,7 @@ struct ctdb_banning_state {
 struct ctdb_recoverd {
        struct ctdb_context *ctdb;
        uint32_t recmaster;
-       uint32_t num_active;
        uint32_t num_lmasters;
-       uint32_t num_connected;
        uint32_t last_culprit_node;
        struct ctdb_node_map *nodemap;
        struct timeval priority_time;
@@ -3591,21 +3589,15 @@ static void main_loop(struct ctdb_context *ctdb, struct ctdb_recoverd *rec,
        }
 
        /* count how many active nodes there are */
-       rec->num_active    = 0;
        rec->num_lmasters  = 0;
-       rec->num_connected = 0;
        for (i=0; i<nodemap->num; i++) {
                if (!(nodemap->nodes[i].flags & NODE_FLAGS_INACTIVE)) {
-                       rec->num_active++;
                        if (ctdb_node_has_capabilities(rec->caps,
                                                       ctdb->nodes[i]->pnn,
                                                       CTDB_CAP_LMASTER)) {
                                rec->num_lmasters++;
                        }
                }
-               if (!(nodemap->nodes[i].flags & NODE_FLAGS_DISCONNECTED)) {
-                       rec->num_connected++;
-               }
        }