format: Reduce size of struct fmt_spec from 6 bytes to 4.

[pspp] / src / output / charts / barchart-cairo.c

diff --git a/src/output/charts/barchart-cairo.c b/src/output/charts/barchart-cairo.c
index c3cc2f3ca14225c29875e5a8d28b6031345e371d..4663471e566e9c81b22ac0fb9d3bc3435d4e5a39 100644 (file)
--- a/src/output/charts/barchart-cairo.c
+++ b/src/output/charts/barchart-cairo.c
@@ -43,10 +43,10 @@ abscissa_label (const struct barchart *bc, cairo_t *cr,
   size_t hash = value_hash (prev, bc->widths[0], 0);
   HMAP_FOR_EACH_WITH_HASH (foo, struct category, node, hash, &bc->primaries)
     {
-      if (value_equal (&foo->val, prev, bc->widths[0])) 
+      if (value_equal (&foo->val, prev, bc->widths[0]))
        break;
     }
-  
+
   draw_tick (cr, geom, SCALE_ABSCISSA, false,
             x_pos - (width * n_last_cat) / 2.0,
             "%s",  ds_cstr (&foo->label));
@@ -56,21 +56,20 @@ abscissa_label (const struct barchart *bc, cairo_t *cr,
 
 
 void
-xrchart_draw_barchart (const struct chart_item *chart_item, cairo_t *cr,
+xrchart_draw_barchart (const struct chart *chart, cairo_t *cr,
                        struct xrchart_geometry *geom)
 {
-  struct barchart *bc = to_barchart (chart_item);
+  struct barchart *bc = to_barchart (chart);
   int i;
 
   xrchart_write_title (cr, geom, _("Bar Chart"));
 
   xrchart_write_ylabel (cr, geom, bc->ylabel);
-  xrchart_write_xlabel (cr, geom, chart_item_get_title (chart_item));
+  xrchart_write_xlabel (cr, geom, chart_get_title (chart));
 
-  if (bc->percent)
-    xrchart_write_yscale (cr, geom, 0, bc->largest * 100.0 / bc->total_count );
-  else
-    xrchart_write_yscale (cr, geom, 0, bc->largest);
+  if (! xrchart_write_yscale (cr, geom, 0,
+                              bc->percent ? bc->largest * 100.0 / bc->total_count : bc->largest))
+    return;
 
   const double abscale = geom->axis[SCALE_ABSCISSA].data_max - geom->axis[SCALE_ABSCISSA].data_min;
   const double width = abscale / (double) (bc->n_nzcats + bc->n_pcats);
@@ -82,7 +81,7 @@ xrchart_draw_barchart (const struct chart_item *chart_item, cairo_t *cr,
     {
       const int blob_size = 13;
       const int height = blob_size * (hmap_count (&bc->secondaries) * 2);
-      
+
       cairo_rectangle (cr,
                       geom->axis[SCALE_ABSCISSA].data_max + 10,
                       geom->axis[SCALE_ORDINATE].data_max - height,
@@ -95,10 +94,10 @@ xrchart_draw_barchart (const struct chart_item *chart_item, cairo_t *cr,
        {
          const struct category *foo = bc->ss[i];
 
-         cairo_move_to (cr, 
+         cairo_move_to (cr,
                         geom->axis[SCALE_ABSCISSA].data_max + (1.5 * blob_size) + 20,
                         geom->axis[SCALE_ORDINATE].data_max - ypos);
-         
+
          xrchart_label (cr, 'l', 'b', geom->font_size, ds_cstr (&foo->label));
 
          cairo_rectangle (cr,
@@ -114,7 +113,7 @@ xrchart_draw_barchart (const struct chart_item *chart_item, cairo_t *cr,
          cairo_fill_preserve (cr);
 
          cairo_restore (cr);
-         
+
          cairo_stroke (cr);
 
          ypos += blob_size * 2;