Add another dataset style: ACRE_STYLE_BARS

[acre] / acre.c

diff --git a/acre.c b/acre.c
index 9a9851c568a684d3623e6b29b8d9a0b77c32301c..9f6f26ebaab1f854cd8cd392c92d19fa64a92e6c 100644 (file)
--- a/acre.c
+++ b/acre.c
@@ -848,28 +848,12 @@ _draw_data_line (acre_t *acre, acre_data_t *data)
 
 #define BARS_WIDTH 0.8
 
-/* Draw the given dataset as bars if there is room for that.
- *
- * Or, if the bars would run into each other, use a line instead.
- */
+/* Draw the given dataset as bars. */
 static void
-_draw_data_bars_or_line (acre_t *acre, acre_data_t *data)
+_draw_data_bars (acre_t *acre, acre_data_t *data)
 {
     unsigned i;
     cairo_t *cr = acre->cr;
-    double ignored, width;
-
-    /* Check device-space width available for inter-bad padding. */
-    width = 1.0 - BARS_WIDTH;
-    ignored = 0.0;
-    cairo_user_to_device_distance (cr, &width, &ignored);
-
-    /* If padding is less than two pixels, draw with a line instead. */
-    if (width < 2.0) {
-       _draw_data_line (acre, data);
-
-       return;
-    }
 
     cairo_save (cr);
 
@@ -888,6 +872,28 @@ _draw_data_bars_or_line (acre_t *acre, acre_data_t *data)
     cairo_restore (cr);
 }
 
+/* Draw the given dataset as bars if there is room for that.
+ *
+ * Or, if the bars would run into each other, use a line instead.
+ */
+static void
+_draw_data_bars_or_line (acre_t *acre, acre_data_t *data)
+{
+    cairo_t *cr = acre->cr;
+    double ignored, width;
+
+    /* Check device-space width available for inter-bar padding. */
+    width = 1.0 - BARS_WIDTH;
+    ignored = 0.0;
+    cairo_user_to_device_distance (cr, &width, &ignored);
+
+    /* If padding is less than two pixels, draw with a line instead. */
+    if (width < 2.0)
+       _draw_data_line (acre, data);
+    else
+       _draw_data_bars (acre, data);
+}
+
 #define TIMELINE_BAR_HEIGHT 0.6
 
 /* Draw the given dataset as a timeline. Each (X,Y) point (potentially
@@ -984,6 +990,9 @@ _draw_data (acre_t *acre)
        case ACRE_STYLE_LINE:
            _draw_data_line (acre, data);
            break;
+       case ACRE_STYLE_BARS:
+           _draw_data_bars (acre, data);
+           break;
        case ACRE_STYLE_BARS_OR_LINE:
            _draw_data_bars_or_line (acre, data);
            break;