Add necessary -lm flag to link command

[acre] / acre.c

diff --git a/acre.c b/acre.c
index 35d0541d9c4b14bc58eef0da4d86bc61121be27a..b844d755153911710ebad19570b710a06d37a2db 100644 (file)
--- a/acre.c
+++ b/acre.c
@@ -28,6 +28,8 @@
 #include <stdarg.h>
 #include <math.h>
 
+#include <lcms.h>
+
 typedef struct _acre_data_point_2d {
     double x;
     double y;
@@ -47,6 +49,12 @@ typedef struct _acre_axis {
     double max;
 } acre_axis_t;
 
+typedef struct _acre_color {
+    double red;
+    double green;
+    double blue;
+} acre_color_t;
+
 struct _acre {
     char *title;
     acre_axis_t x_axis;
@@ -59,7 +67,8 @@ struct _acre {
     /* Data for drawing. */
     cairo_t *cr;
     PangoFontDescription *font;
-    struct { double r; double g; double b;} colors[3];
+    acre_color_t *colors;
+    int colors_size;
     int num_colors;
 
     /* Total size including labels. */
@@ -92,6 +101,14 @@ acre_create (void)
     acre->data_size = 0;
     acre->num_data = 0;
 
+    acre->cr = NULL;
+    acre->font = NULL;
+    acre->colors = NULL;
+    acre->num_colors = 0;
+
+    acre->width = 0;
+    acre->height = 0;
+
     acre->chart.x = 0;
     acre->chart.y = 0;
     acre->chart.width = 0;
@@ -115,6 +132,8 @@ acre_destroy (acre_t *acre)
 
     free (acre->data);
 
+    free (acre->colors);
+
     free (acre);
 }
 
@@ -500,24 +519,96 @@ _compute_axis_ranges (acre_t *acre)
     cairo_restore (cr);
 }
 
+static void
+_acre_color_from_hsv (acre_color_t *color,
+                     double hue,
+                     double saturation,
+                     double value)
+{
+    double f, p, q, t;
+    int hmod6;
+
+    hmod6 = (int) floor (hue / 60) % 6;
+    f = hue / 60 - floor (hue / 60);
+    p = value * (1 - saturation);
+    q = value * (1 - f * saturation);
+    t = value * (1 - (1 - f) * saturation);
+    
+    switch (hmod6) {
+      case 0:
+       color->red = value;
+       color->green = t;
+       color->blue = p;
+       break;
+      case 1:
+       color->red = q;
+       color->green = value;
+       color->blue = p;
+       break;
+      case 2:
+       color->red = p;
+       color->green = value;
+       color->blue = t;
+       break;
+      case 3:
+       color->red = p;
+       color->green = q;
+       color->blue = value;
+       break;
+      case 4:
+       color->red = t;
+       color->green = p;
+       color->blue = value;
+       break;
+      case 5:
+       color->red = value;
+       color->green = p;
+       color->blue = q;
+       break;
+    }
+}
+
 static void
 _choose_colors (acre_t *acre)
 {
-    /* XXX: Should choose N, equally-spaced colors from a perceptually
-     * linear space here. */
-    acre->num_colors = 3;
+    cmsHPROFILE lab_profile, srgb_profile;
+    cmsHTRANSFORM lab_to_srgb;
+    int i;
+    double theta, radius, srgb[3];
+    cmsCIELab lab;
+
+    lab_profile = cmsCreateLabProfile (NULL); /* D50 */
+    srgb_profile = cmsCreate_sRGBProfile ();
+
+    lab_to_srgb = cmsCreateTransform (lab_profile, TYPE_Lab_DBL,
+                                     srgb_profile, TYPE_RGB_DBL,
+                                     INTENT_PERCEPTUAL, 0);
+
+    acre->num_colors = acre->num_data;
 
-    acre->colors[0].r = 1;
-    acre->colors[0].g = 0;
-    acre->colors[0].b = 0;
+    if (acre->num_colors > acre->colors_size) {
+       acre->colors_size = acre->num_colors;
+       acre->colors = xrealloc (acre->colors,
+                                acre->colors_size * sizeof (acre_color_t));
+    }
+
+    lab.L = 36;
+    radius = 130;
+    for (i = 0; i < acre->num_colors; i++) {
+       theta = 0.713 + 2 * M_PI * (double) i / acre->num_colors;
+       lab.a = radius * cos (theta);
+       lab.b = radius * sin (theta);
 
-    acre->colors[1].r = 0;
-    acre->colors[1].g = 1;
-    acre->colors[1].b = 0;
+       cmsDoTransform (lab_to_srgb, &lab, srgb, 1);
+
+       acre->colors[i].red = srgb[0];
+       acre->colors[i].green = srgb[1];
+       acre->colors[i].blue = srgb[2];
+    }
 
-    acre->colors[2].r = 0;
-    acre->colors[2].g = 0;
-    acre->colors[2].b = 1;
+    cmsDeleteTransform (lab_to_srgb);
+    cmsCloseProfile (lab_profile);
+    cmsCloseProfile (srgb_profile);
 }
 
 static void
@@ -536,9 +627,9 @@ _draw_data (acre_t *acre)
     for (d = 0; d < acre->num_data; d++) {
        int color = d % acre->num_colors;
        cairo_set_source_rgb (cr,
-                             acre->colors[color].r,
-                             acre->colors[color].g,
-                             acre->colors[color].b);
+                             acre->colors[color].red,
+                             acre->colors[color].green,
+                             acre->colors[color].blue);
        data = acre->data[d];
        cairo_new_path (cr);
        for (i = 0; i < data->num_points; i++) {
@@ -680,13 +771,14 @@ _draw_legend (acre_t *acre)
     cairo_translate (cr, ACRE_LEGEND_PAD, ACRE_LEGEND_PAD);
 
     for (i = 0; i < acre->num_data; i++) {
-       cairo_move_to (cr, 0, ACRE_FONT_SIZE / 2 + 0.5);
-       cairo_rel_line_to (cr, ACRE_LEGEND_LINE_SIZE, 0);
+       cairo_rectangle (cr,
+                        0, ACRE_LEGEND_LINE_SIZE / 2,
+                        ACRE_LEGEND_LINE_SIZE, ACRE_LEGEND_LINE_SIZE / 2);
        cairo_set_source_rgb (cr,
-                             acre->colors[i % acre->num_colors].r,
-                             acre->colors[i % acre->num_colors].g,
-                             acre->colors[i % acre->num_colors].b);
-       cairo_stroke (cr);
+                             acre->colors[i % acre->num_colors].red,
+                             acre->colors[i % acre->num_colors].green,
+                             acre->colors[i % acre->num_colors].blue);
+       cairo_fill (cr);
 
        layout = _create_layout (acre, acre->data[i]->name);
        cairo_move_to (cr, ACRE_LEGEND_LINE_SIZE + ACRE_LEGEND_PAD, 0);