batman-adv: introduce is_single_hop_neigh variable to increase readability
authorMarek Lindner <lindner_marek@yahoo.de>
Thu, 1 Mar 2012 07:35:16 +0000 (15:35 +0800)
committerAntonio Quartulli <ordex@autistici.org>
Fri, 11 May 2012 08:08:09 +0000 (10:08 +0200)
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Acked-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
net/batman-adv/bat_iv_ogm.c

index 8b2db2e76c7ee8ee903ecc5f3a70b6673dfcb975..cd8f473c1bd0299e50f04a398626a4a43353b6fb 100644 (file)
@@ -480,7 +480,8 @@ static void bat_iv_ogm_queue_add(struct bat_priv *bat_priv,
 static void bat_iv_ogm_forward(struct orig_node *orig_node,
                               const struct ethhdr *ethhdr,
                               struct batman_ogm_packet *batman_ogm_packet,
-                              int directlink, struct hard_iface *if_incoming)
+                              bool is_single_hop_neigh,
+                              struct hard_iface *if_incoming)
 {
        struct bat_priv *bat_priv = netdev_priv(if_incoming->soft_iface);
        struct neigh_node *router;
@@ -533,7 +534,7 @@ static void bat_iv_ogm_forward(struct orig_node *orig_node,
 
        /* switch of primaries first hop flag when forwarding */
        batman_ogm_packet->flags &= ~PRIMARIES_FIRST_HOP;
-       if (directlink)
+       if (is_single_hop_neigh)
                batman_ogm_packet->flags |= DIRECTLINK;
        else
                batman_ogm_packet->flags &= ~DIRECTLINK;
@@ -918,7 +919,8 @@ static void bat_iv_ogm_process(const struct ethhdr *ethhdr,
        struct neigh_node *orig_neigh_router = NULL;
        int has_directlink_flag;
        int is_my_addr = 0, is_my_orig = 0, is_my_oldorig = 0;
-       int is_broadcast = 0, is_bidirectional, is_single_hop_neigh;
+       int is_broadcast = 0, is_bidirectional;
+       bool is_single_hop_neigh = false;
        int is_duplicate;
        uint32_t if_incoming_seqno;
 
@@ -942,8 +944,8 @@ static void bat_iv_ogm_process(const struct ethhdr *ethhdr,
 
        has_directlink_flag = (batman_ogm_packet->flags & DIRECTLINK ? 1 : 0);
 
-       is_single_hop_neigh = (compare_eth(ethhdr->h_source,
-                                          batman_ogm_packet->orig) ? 1 : 0);
+       if (compare_eth(ethhdr->h_source, batman_ogm_packet->orig))
+               is_single_hop_neigh = true;
 
        bat_dbg(DBG_BATMAN, bat_priv,
                "Received BATMAN packet via NB: %pM, IF: %s [%pM] (from OG: %pM, via prev OG: %pM, seqno %u, ttvn %u, crc %u, changes %u, td %d, TTL %d, V %d, IDF %d)\n",
@@ -1114,7 +1116,7 @@ static void bat_iv_ogm_process(const struct ethhdr *ethhdr,
 
                /* mark direct link on incoming interface */
                bat_iv_ogm_forward(orig_node, ethhdr, batman_ogm_packet,
-                                  1, if_incoming);
+                                  is_single_hop_neigh, if_incoming);
 
                bat_dbg(DBG_BATMAN, bat_priv,
                        "Forwarding packet: rebroadcast neighbor packet with direct link flag\n");
@@ -1137,7 +1139,7 @@ static void bat_iv_ogm_process(const struct ethhdr *ethhdr,
        bat_dbg(DBG_BATMAN, bat_priv,
                "Forwarding packet: rebroadcast originator packet\n");
        bat_iv_ogm_forward(orig_node, ethhdr, batman_ogm_packet,
-                          0, if_incoming);
+                          is_single_hop_neigh, if_incoming);
 
 out_neigh:
        if ((orig_neigh_node) && (!is_single_hop_neigh))