1 files changed, 74 insertions, 13 deletions

diff --git a/src/statistics/gtk_statistics.c b/src/statistics/gtk_statistics.c
index b1d7fe84..c483794d 100644
--- a/src/statistics/gtk_statistics.c
+++ b/src/statistics/gtk_statistics.c
@@ -39,6 +39,9 @@
  */
 #define YTICKS 5
 
+/**
+ * Additional distance between text and lines / borders in pixels.
+ */
 #define BORDER 10.0
 
 
@@ -66,6 +69,11 @@ struct ValueHistory
 {
 
   /**
+   * ID for value updates.
+   */
+  char *id;
+
+  /**
    * Label for the subsystem.
    */
   char *label;
@@ -172,9 +180,16 @@ gtk_statistics_new ()
 
 
 /**
+ * Add another data series to plot by the statistics widget.
+ *
+ * @param statistics widget to modify
+ * @param id identifier for the series
+ * @param label label to use
+ * @param color_name name of the color to use
  */
 void
 gtk_statistics_add_line (GtkStatistics *statistics,
+                         const char *id,
                          const char *label,
                          const char *color_name)
 {
@@ -188,6 +203,7 @@ gtk_statistics_add_line (GtkStatistics *statistics,
   priv->values = g_realloc (priv->values,
                             sizeof (struct ValueHistory*) * (1 + priv->num_values));
   priv->values[priv->num_values++] = vh = g_malloc (sizeof (struct ValueHistory));
+  vh->id = strdup (id);
   vh->label = strdup (label);
   vh->red = c.red / 65535.0;
   vh->green = c.green / 65535.0;
@@ -196,10 +212,16 @@ gtk_statistics_add_line (GtkStatistics *statistics,
 
 
 /**
+ * Add another value to a data series.
+ * 
+ * @param statistics widget to update
+ * @param id identifier of the series
+ * @param x new x-value
+ * @param y new y-value
  */
 void
 gtk_statistics_update_value (GtkStatistics      *statistics,
-                             const char *label,
+                             const char *id,
                              uint64_t x,
                              uint64_t y)
 {
@@ -213,14 +235,15 @@ gtk_statistics_update_value (GtkStatistics      *statistics,
   for (i=0;i<priv->num_values;i++)
   {
     vh = priv->values[i];
-    if (0 != strcmp (label, vh->label))
+    if (0 != strcmp (id, vh->id))
       continue;
+
     if (++vh->last_history_offset == MAX_HISTORY)
       vh->last_history_offset = 0;
     if (vh->history_size < MAX_HISTORY)
       vh->history_size++;
     vh->history[vh->last_history_offset].x = x;
-    vh->history[vh->last_history_offset].y = y;
+    vh->history[vh->last_history_offset].y = y;  
     widget = GTK_WIDGET (statistics);
     if (gtk_widget_is_drawable (widget))
       gtk_widget_queue_draw (widget);
@@ -228,11 +251,19 @@ gtk_statistics_update_value (GtkStatistics      *statistics,
 }
 
 
+/**
+ * Convert a number to a nice label for the axis.
+ *
+ * @param num number to convert
+ * @param label where to store the string (must be big enough!)
+ */
 static void
 num_to_label (unsigned long long num,
               char *label)
 {
-  if (num > 1000LL * 1000 * 1000 * 3)
+  if (num > 1000LL * 1000 * 1000 * 1000 * 3)
+    sprintf (label, "%llu t", num / 1000LL / 1000LL / 1000LL / 1000LL);
+  else if (num > 1000LL * 1000 * 1000 * 3)
     sprintf (label, "%llu g", num / 1000LL / 1000LL / 1000LL);
   else if (num > 1000LL * 1000 * 3)
     sprintf (label, "%llu m", num / 1000LL / 1000LL);
@@ -243,6 +274,12 @@ num_to_label (unsigned long long num,
 }
 
 
+/**
+ * Draw the statistics widget.
+ *
+ * @param widget widget to draw
+ * @param cr drawing context
+ */
 static gboolean
 gtk_statistics_draw (GtkWidget *widget,
                 cairo_t   *cr)
@@ -264,6 +301,7 @@ gtk_statistics_draw (GtkWidget *widget,
   cairo_text_extents_t tey_max;
   double rx;
   double ry;
+  unsigned int h;
 
   /* collect basic data */
   xmin = UINT64_MAX;
@@ -292,7 +330,7 @@ gtk_statistics_draw (GtkWidget *widget,
   cairo_set_source_rgb(cr, 0.0, 0.0, 0.0);
   cairo_paint(cr);
   
-  if (0 == priv->num_values)
+ if ( (0 == priv->num_values) || (ymax == 0) )
     return FALSE; /* done */
   /* select font */
   cairo_set_source_rgb (cr, 1.0, 1.0, 1.0);
@@ -379,10 +417,26 @@ gtk_statistics_draw (GtkWidget *widget,
 
   /* finally, plot lines */
   cairo_set_line_width (cr, 2.0);
+
+  cairo_set_font_size (cr, 
+                       GNUNET_MIN (20, (height - 3.0 * BORDER - tex_max.height - tey_max.height / 2) / (priv->num_values + 1)));
+
+  h = 2.0 * BORDER + tey_max.height / 2;
+
   for (i=0;i<priv->num_values;i++)
   {
     vh = priv->values[i];
     cairo_set_source_rgb(cr, vh->red, vh->green, vh->blue);
+    cairo_text_extents (cr, vh->label, &te);
+    h += te.height / 2;
+    cairo_move_to (cr, 
+                   3.0 * BORDER + tey_max.width,
+                   h);
+    h += te.height / 2 + 1.0;
+    cairo_show_text (cr, vh->label);
+    if (xmax == xmin) 
+      continue;
+
     for (j=0;j<vh->history_size;j++)
     {
       hv = &vh->history[(vh->last_history_offset - j + MAX_HISTORY) % MAX_HISTORY];
@@ -391,14 +445,15 @@ gtk_statistics_draw (GtkWidget *widget,
       
       rx = tey_max.width + BORDER * 2.0 + (rx * (width - BORDER * 3.0 - tey_max.width - tex_max.width / 2.0));
       ry = BORDER + tex_max.height / 2.0 + (1.0 - ry) * (height - BORDER * 2.0 - tey_max.height - tex_max.height);
-      if (j == 0)
-      {
-        cairo_move_to (cr, rx, ry);
-      }
-      else
-      {
-        cairo_line_to (cr, rx, ry);
-      }
+
+      /* if y-values are small, offset y-values a bit to allow overlapping curves to still show up */
+      if (ymax < height / (priv->num_values * 4))
+        ry += priv->num_values * 2 - (4 * i);
+      if (j == 0) 
+        cairo_move_to (cr, 
+                       width - BORDER - tex_max.width / 2.0, 
+                       ry);      
+      cairo_line_to (cr, rx, ry);
     }
     cairo_stroke (cr);
   }
@@ -406,6 +461,11 @@ gtk_statistics_draw (GtkWidget *widget,
 }
 
 
+/**
+ * Free memory used by statistics object.
+ *
+ * @param object object to release
+ */
 static void
 gtk_statistics_finalize (GObject *object)
 {
@@ -416,6 +476,7 @@ gtk_statistics_finalize (GObject *object)
   for (i=0;i<priv->num_values;i++)
   {
     g_free (priv->values[i]->label);
+    g_free (priv->values[i]->id);
     g_free (priv->values[i]);  
   }
   g_free (priv->values);