use INET_ADDRSTRLEN and INET6_ADDRSTRLEN instead of magic constants
authorFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 1 Nov 2012 13:17:01 +0000 (14:17 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 1 Nov 2012 13:23:41 +0000 (14:23 +0100)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/duplicate_set.c
src/gateway.c
src/hna_set.c
src/link_set.c
src/neighbor_table.c
src/tc_set.c
src/two_hop_neighbor_table.c

index 6ee020c..304d02c 100644 (file)
@@ -198,7 +198,7 @@ olsr_print_duplicate_table(void)
 {
   /* The whole function makes no sense without it. */
   struct dup_entry *entry;
-  const int ipwidth = olsr_cnf->ip_version == AF_INET ? 15 : 39;
+  const int ipwidth = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
   struct ipaddr_str addrbuf;
 
   OLSR_PRINTF(1, "\n--- %s ------------------------------------------------- DUPLICATE SET\n\n" "%-*s %8s %s\n",
index 7b690ef..266843a 100644 (file)
@@ -234,7 +234,7 @@ void olsr_trigger_inetgw_startup(void) {
 void olsr_print_gateway_entries(void) {
   struct ipaddr_str buf;
   struct gateway_entry *gw;
-  const int addrsize = olsr_cnf->ip_version == AF_INET ? 15 : 39;
+  const int addrsize = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
 
   OLSR_PRINTF(0, "\n--- %s ---------------------------------------------------- GATEWAYS\n\n", olsr_wallclock_string());
   OLSR_PRINTF(0, "%-*s %-6s %-9s %-9s %s\n",
index b474eae..ea0eb03 100644 (file)
@@ -318,6 +318,7 @@ olsr_print_hna_set(void)
   int idx;
   struct tm * nowtm;
   struct timeval now;
+  const int ipwidth = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
 
        (void)gettimeofday(&now, NULL);
   nowtm = localtime(&now.tv_sec);
@@ -338,15 +339,9 @@ olsr_print_hna_set(void)
       struct hna_net *tmp_net = tmp_hna->networks.next;
 
       while (tmp_net != &tmp_hna->networks) {
-        if (olsr_cnf->ip_version == AF_INET) {
-          struct ipaddr_str buf;
-          OLSR_PRINTF(1, "%-15s ", olsr_ip_prefix_to_string(&tmp_net->hna_prefix));
-          OLSR_PRINTF(1, "%-15s\n", olsr_ip_to_string(&buf, &tmp_hna->A_gateway_addr));
-        } else {
-          struct ipaddr_str buf;
-          OLSR_PRINTF(1, "%-27s ", olsr_ip_prefix_to_string(&tmp_net->hna_prefix));
-          OLSR_PRINTF(1, "%-27s\n", olsr_ip_to_string(&buf, &tmp_hna->A_gateway_addr));
-        }
+        struct ipaddr_str buf;
+        OLSR_PRINTF(1, "%-*s ", ipwidth, olsr_ip_prefix_to_string(&tmp_net->hna_prefix));
+        OLSR_PRINTF(1, "%-*s\n", ipwidth, olsr_ip_to_string(&buf, &tmp_hna->A_gateway_addr));
 
         tmp_net = tmp_net->next;
       }
index 53a2219..399a5ee 100644 (file)
@@ -805,7 +805,7 @@ olsr_print_link_set(void)
 {
   /* The whole function makes no sense without it. */
   struct link_entry *walker;
-  const int addrsize = olsr_cnf->ip_version == AF_INET ? 15 : 39;
+  const int addrsize = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
 
   OLSR_PRINTF(0, "\n--- %s ---------------------------------------------------- LINKS\n\n", olsr_wallclock_string());
   OLSR_PRINTF(1, "%-*s  %-6s %-14s %s\n", addrsize, "IP address", "hyst", "      LQ      ", "ETX");
index af4c413..406b646 100644 (file)
@@ -383,7 +383,7 @@ void
 olsr_print_neighbor_table(void)
 {
   /* The whole function doesn't do anything else. */
-  const int iplen = olsr_cnf->ip_version == AF_INET ? 15 : 39;
+  const int iplen = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
   int idx;
   struct tm * nowtm;
   struct timeval now;
index 89095e0..9c44323 100644 (file)
@@ -688,7 +688,7 @@ olsr_print_tc_table(void)
 {
   /* The whole function makes no sense without it. */
   struct tc_entry *tc;
-  const int ipwidth = olsr_cnf->ip_version == AF_INET ? 15 : 39;
+  const int ipwidth = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
 
   OLSR_PRINTF(1, "\n--- %s ------------------------------------------------- TOPOLOGY\n\n" "%-*s %-*s %-14s  %s\n",
               olsr_wallclock_string(), ipwidth, "Source IP addr", ipwidth, "Dest IP addr", "      LQ      ", "ETX");
index 63cb1e4..19a8da1 100644 (file)
@@ -210,6 +210,7 @@ olsr_print_two_hop_neighbor_table(void)
 {
   /* The whole function makes no sense without it. */
   int i;
+  const int ipwidth = olsr_cnf->ip_version == AF_INET ? (INET_ADDRSTRLEN - 1) : (INET6_ADDRSTRLEN - 1);
 
   OLSR_PRINTF(1, "\n--- %s ----------------------- TWO-HOP NEIGHBORS\n\n" "IP addr (2-hop)  IP addr (1-hop)  Total cost\n",
               olsr_wallclock_string());
@@ -224,12 +225,12 @@ olsr_print_two_hop_neighbor_table(void)
         struct ipaddr_str buf;
         struct lqtextbuffer lqbuffer;
         if (first) {
-          OLSR_PRINTF(1, "%-15s  ", olsr_ip_to_string(&buf, &neigh2->neighbor_2_addr));
+          OLSR_PRINTF(1, "%-*s  ", ipwidth, olsr_ip_to_string(&buf, &neigh2->neighbor_2_addr));
           first = false;
         } else {
           OLSR_PRINTF(1, "                 ");
         }
-        OLSR_PRINTF(1, "%-15s  %s\n", olsr_ip_to_string(&buf, &entry->neighbor->neighbor_main_addr),
+        OLSR_PRINTF(1, "%-*s  %s\n", ipwidth, olsr_ip_to_string(&buf, &entry->neighbor->neighbor_main_addr),
                     get_linkcost_text(entry->path_linkcost, false, &lqbuffer));
       }
     }