* cleanup:
authorBernd Petrovitsch <bernd@firmix.at>
Sun, 4 Nov 2007 17:52:13 +0000 (17:52 +0000)
committerBernd Petrovitsch <bernd@firmix.at>
Sun, 4 Nov 2007 17:52:13 +0000 (17:52 +0000)
  - #ifdef more away if NODEBUG is enabled.
  - reduce the size of the data and code

src/tc_set.c
src/tc_set.h

index 3bcaefe..25f3503 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: tc_set.c,v 1.34 2007/11/02 20:58:06 bernd67 Exp $
+ * $Id: tc_set.c,v 1.35 2007/11/04 17:52:13 bernd67 Exp $
  */
 
 #include "tc_set.h"
@@ -609,42 +609,32 @@ olsr_time_out_tc_set(void)
 /**
  * Print the topology table to stdout
  */
-int
+void
 olsr_print_tc_table(void)
 {
+#ifndef NODEBUG
+  /* The whole function makes no sense without it. */
   struct tc_entry *tc;
-  struct tc_edge_entry *tc_edge;
-  char *fstr;
-  
-  OLSR_PRINTF(1, "\n--- %02d:%02d:%02d.%02d ------------------------------------------------- TOPOLOGY\n\n",
-              nowtm->tm_hour,
-              nowtm->tm_min,
-              nowtm->tm_sec,
-              (int)now.tv_usec / 10000);
-
-  if (olsr_cnf->ip_version == AF_INET)
-    {
-      OLSR_PRINTF(1, "Source IP addr   Dest IP addr     LQ     ILQ    ETX\n");
-      fstr = "%-15s  %-15s  %5.3f  %5.3f  %.2f\n";
-    }
-  else
-    {
-      OLSR_PRINTF(1, "Source IP addr                Dest IP addr                    LQ     ILQ    ETX\n");
-      fstr = "%-30s%-30s  %5.3f  %5.3f  %.2f\n";
-    }
+  const int ipwidth = olsr_cnf->ip_version == AF_INET ? 15 : 30;
+
+  OLSR_PRINTF(1,
+              "\n--- %02d:%02d:%02d.%02d ------------------------------------------------- TOPOLOGY\n\n"
+              "%-*s %-*s %-5s  %-5s  %s\n",
+              nowtm->tm_hour, nowtm->tm_min, nowtm->tm_sec, (int)now.tv_usec / 10000,
+              ipwidth, "Source IP addr", ipwidth, "Dest IP addr", "LQ", "ILQ", "ETX");
 
   OLSR_FOR_ALL_TC_ENTRIES(tc) {
+    struct tc_edge_entry *tc_edge;
     OLSR_FOR_ALL_TC_EDGE_ENTRIES(tc, tc_edge) {
-      OLSR_PRINTF(1, fstr, olsr_ip_to_string(&tc->addr),
-                  olsr_ip_to_string(&tc_edge->T_dest_addr),
+      OLSR_PRINTF(1, "%-*s %-*s  %5.3f  %5.3f  %.2f\n",
+                  ipwidth, olsr_ip_to_string(&tc->addr),
+                  ipwidth, olsr_ip_to_string(&tc_edge->T_dest_addr),
                   tc_edge->link_quality,
                   tc_edge->inverse_link_quality,
                   olsr_calc_tc_etx(tc_edge));
-
     } OLSR_FOR_ALL_TC_EDGE_ENTRIES_END(tc, tc_edge);
   } OLSR_FOR_ALL_TC_ENTRIES_END(tc);
-
-  return 1;
+#endif
 }
 
 float olsr_calc_tc_etx(const struct tc_edge_entry *tc_edge)
index fdec2bb..fb535d5 100644 (file)
@@ -37,7 +37,7 @@
  * to the project. For more information see the website or contact
  * the copyright holders.
  *
- * $Id: tc_set.h,v 1.20 2007/11/02 20:58:06 bernd67 Exp $
+ * $Id: tc_set.h,v 1.21 2007/11/04 17:52:13 bernd67 Exp $
  */
 
 #ifndef _OLSR_TOP_SET
@@ -67,12 +67,12 @@ struct tc_edge_entry
   float              inverse_link_quality;
 };
 
-#define OLSR_TC_EDGE_DOWN (1 <<  0) /* this edge is down */
+#define OLSR_TC_EDGE_DOWN (1 << 0) /* this edge is down */
 
 /*
  * Garbage collection time for downed edges
  */
-#define OLSR_TC_EDGE_GC_TIME 15*1000 /* milliseconds */
+#define OLSR_TC_EDGE_GC_TIME (15*1000) /* milliseconds */
 
 struct tc_entry
 {
@@ -119,7 +119,7 @@ int olsr_init_tc(void);
 void olsr_change_myself_tc(void);
 int olsr_tc_delete_mprs(struct tc_entry *, struct tc_message *);
 int olsr_tc_update_mprs(struct tc_entry *, struct tc_message *);
-int olsr_print_tc_table(void);
+void olsr_print_tc_table(void);
 void olsr_time_out_tc_set(void);
 
 /* tc_entry manipulation */