treewide: Replace <name>_cnt by n_<name>s and <name>_cap by allocated_<name>.
[pspp] / src / data / datasheet.c
index b64df14563865875b7d60374f515007d87353b28..74da1116a579e539178461917c5561259b557544 100644 (file)
@@ -1,5 +1,5 @@
 /* PSPP - a program for statistical analysis.
-   Copyright (C) 2007, 2009, 2010, 2011, 2012 Free Software Foundation, Inc.
+   Copyright (C) 2007, 2009, 2010, 2011, 2012, 2013 Free Software Foundation, Inc.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -80,9 +80,10 @@ static int source_allocate_column (struct source *, int width);
 static void source_release_column (struct source *, int ofs, int width);
 static bool source_in_use (const struct source *);
 
-static bool source_read (const struct column *, casenumber row, union value *);
+static bool source_read (const struct column *, casenumber row, union value *,
+                         size_t n);
 static bool source_write (const struct column *, casenumber row,
-                          const union value *);
+                          const union value *, size_t n);
 static bool source_write_column (struct column *, const union value *);
 static bool source_has_backing (const struct source *);
 
@@ -174,7 +175,7 @@ value_to_data (const union value *value_, int width)
   if (width == 0)
     return &value->f;
   else
-    return value_str_rw (value, width);
+    return value->s;
 }
 
 /* Returns the number of bytes needed to store all the values in
@@ -431,8 +432,8 @@ struct resize_datasheet_value_aux
     size_t src_ofs;
     int src_width;
 
-    void (*resize_cb) (const union value *, union value *, void *aux);
-    void *resize_cb_aux;
+    void (*resize_cb) (const union value *, union value *, const void *aux);
+    const void *resize_cb_aux;
 
     union value dst_value;
     size_t dst_ofs;
@@ -460,8 +461,8 @@ resize_datasheet_value (const void *src, void *dst, void *aux_)
 bool
 datasheet_resize_column (struct datasheet *ds, size_t column, int new_width,
                          void (*resize_cb) (const union value *,
-                                            union value *, void *aux),
-                         void *resize_cb_aux)
+                                            union value *, const void *aux),
+                         const void *resize_cb_aux)
 {
   struct column old_col;
   struct column *col;
@@ -506,18 +507,22 @@ datasheet_resize_column (struct datasheet *ds, size_t column, int new_width,
       for (lrow = 0; lrow < n_rows; lrow++)
         {
           unsigned long int prow = axis_map (ds->rows, lrow);
-          if (!source_read (&old_col, prow, &src))
+          if (!source_read (&old_col, prow, &src, 1))
             {
               /* FIXME: back out col changes. */
-              return false;
+              break;
             }
           resize_cb (&src, &dst, resize_cb_aux);
-          if (!source_write (col, prow, &dst))
+          if (!source_write (col, prow, &dst, 1))
             {
               /* FIXME: back out col changes. */
-              return false;
+              break;
             }
         }
+      value_destroy (&src, old_width);
+      value_destroy (&dst, new_width);
+      if (lrow < n_rows)
+       return false;
 
       release_source (ds, old_col.source);
     }
@@ -599,8 +604,8 @@ datasheet_get_value (const struct datasheet *ds, casenumber row,
    successful, false on I/O error.  On failure, ROW might be
    partially modified or corrupted. */
 bool
-datasheet_put_value (struct datasheet *ds UNUSED, casenumber row UNUSED,
-                     size_t column UNUSED, const union value *value UNUSED)
+datasheet_put_value (struct datasheet *ds, casenumber row,
+                     size_t column, const union value *value)
 {
   return rw_case (ds, OP_WRITE, row, column, 1, (union value *) value);
 }
@@ -620,37 +625,37 @@ datasheet_insert_rows (struct datasheet *ds,
   while (cnt > 0)
     {
       unsigned long first_phy;
-      unsigned long phy_cnt;
+      unsigned long n_phys;
       unsigned long i;
 
       /* Allocate physical rows from the pool of available
          rows. */
-      if (!axis_allocate (ds->rows, cnt, &first_phy, &phy_cnt))
+      if (!axis_allocate (ds->rows, cnt, &first_phy, &n_phys))
         {
           /* No rows were available.  Extend the row axis to make
              some new ones available. */
-          phy_cnt = cnt;
+          n_phys = cnt;
           first_phy = axis_extend (ds->rows, cnt);
         }
 
       /* Insert the new rows into the row mapping. */
-      axis_insert (ds->rows, before, first_phy, phy_cnt);
+      axis_insert (ds->rows, before, first_phy, n_phys);
 
       /* Initialize the new rows. */
-      for (i = 0; i < phy_cnt; i++)
+      for (i = 0; i < n_phys; i++)
         if (!datasheet_put_row (ds, before + i, c[i]))
           {
             while (++i < cnt)
               case_unref (c[i]);
-            datasheet_delete_rows (ds, before - added, phy_cnt + added);
+            datasheet_delete_rows (ds, before - added, n_phys + added);
             return false;
           }
 
       /* Advance. */
-      c += phy_cnt;
-      cnt -= phy_cnt;
-      before += phy_cnt;
-      added += phy_cnt;
+      c += n_phys;
+      cnt -= n_phys;
+      before += n_phys;
+      added += n_phys;
     }
   return true;
 }
@@ -729,10 +734,10 @@ datasheet_reader_destroy (struct casereader *reader UNUSED, void *ds_)
 /* "advance" function for the datasheet random casereader. */
 static void
 datasheet_reader_advance (struct casereader *reader UNUSED, void *ds_,
-                          casenumber case_cnt)
+                          casenumber n_cases)
 {
   struct datasheet *ds = ds_;
-  datasheet_delete_rows (ds, 0, case_cnt);
+  datasheet_delete_rows (ds, 0, n_cases);
 }
 
 /* Random casereader class for a datasheet. */
@@ -805,6 +810,7 @@ rw_case (struct datasheet *ds, enum rw_op op,
          casenumber lrow, size_t start_column, size_t n_columns,
          union value data[])
 {
+  struct column *columns = &ds->columns[start_column];
   casenumber prow;
   size_t i;
 
@@ -813,24 +819,34 @@ rw_case (struct datasheet *ds, enum rw_op op,
   assert (start_column + n_columns <= datasheet_get_n_columns (ds));
 
   prow = axis_map (ds->rows, lrow);
-  for (i = 0; i < n_columns; i++)
+  for (i = 0; i < n_columns;)
     {
-      struct column *c = &ds->columns[start_column + i];
-      if (c->width >= 0)
+      struct source *source = columns[i].source;
+      size_t j;
+      bool ok;
+
+      if (columns[i].width < 0)
         {
-          bool ok;
+          i++;
+          continue;
+        }
 
-          if (op == OP_READ)
-            ok = source_read (c, prow, &data[i]);
-          else
-            ok = source_write (c, prow, &data[i]);
+      for (j = i + 1; j < n_columns; j++)
+        if (columns[j].width < 0 || columns[j].source != source)
+          break;
 
-          if (!ok)
-            {
-              taint_set_taint (ds->taint);
-              return false;
-            }
+      if (op == OP_READ)
+        ok = source_read (&columns[i], prow, &data[i], j - i);
+      else
+        ok = source_write (&columns[i], prow, &data[i], j - i);
+
+      if (!ok)
+        {
+          taint_set_taint (ds->taint);
+          return false;
         }
+
+      i = j;
     }
   return true;
 }
@@ -1335,30 +1351,39 @@ source_get_backing_n_rows (const struct source *source)
   return source->backing_rows;
 }
 
-/* Reads the given COLUMN from SOURCE in the given ROW, into
-   VALUE.  Returns true if successful, false on I/O error.
+/* Reads the N COLUMNS in the given ROW, into the N VALUES.  Returns true if
+   successful, false on I/O error.
 
-   The caller must have initialized VALUE with the proper
-   width. */
+   All of the COLUMNS must have the same source.
+
+   The caller must have initialized VALUES with the proper width. */
 static bool
-source_read (const struct column *column, casenumber row, union value *value)
+source_read (const struct column columns[], casenumber row,
+             union value values[], size_t n)
 {
-  struct source *source = column->source;
+  struct source *source = columns[0].source;
+  size_t i;
 
-  assert (column->width >= 0);
   if (source->backing == NULL
       || sparse_xarray_contains_row (source->data, row))
-    return sparse_xarray_read (source->data, row, column->byte_ofs,
-                               width_to_n_bytes (column->width),
-                               value_to_data (value, column->width));
+    {
+      bool ok = true;
+
+      for (i = 0; i < n && ok; i++)
+        ok = sparse_xarray_read (source->data, row, columns[i].byte_ofs,
+                                 width_to_n_bytes (columns[i].width),
+                                 value_to_data (&values[i], columns[i].width));
+      return ok;
+    }
   else
     {
       struct ccase *c = casereader_peek (source->backing, row);
       bool ok = c != NULL;
       if (ok)
         {
-          value_copy (value, case_data_idx (c, column->value_ofs),
-                      column->width);
+          for (i = 0; i < n; i++)
+            value_copy (&values[i], case_data_idx (c, columns[i].value_ofs),
+                        columns[i].width);
           case_unref (c);
         }
       return ok;
@@ -1390,18 +1415,20 @@ copy_case_into_source (struct source *source, struct ccase *c, casenumber row)
   return true;
 }
 
-/* Writes VALUE to SOURCE in the given ROW and COLUMN.  Returns
-   true if successful, false on I/O error.  On error, the row's
-   data may be completely or partially corrupted, both inside and
-   outside the region to be written.  */
+/* Writes the N VALUES to their source in the given ROW and COLUMNS.  Returns
+   true if successful, false on I/O error.  On error, the row's data may be
+   completely or partially corrupted, both inside and outside the region to be
+   written.
+
+   All of the COLUMNS must have the same source. */
 static bool
-source_write (const struct column *column, casenumber row,
-              const union value *value)
+source_write (const struct column columns[], casenumber row,
+              const union value values[], size_t n)
 {
-  struct source *source = column->source;
+  struct source *source = columns[0].source;
   struct casereader *backing = source->backing;
+  size_t i;
 
-  assert (column->width >= 0);
   if (backing != NULL
       && !sparse_xarray_contains_row (source->data, row)
       && row < source->backing_rows)
@@ -1419,9 +1446,12 @@ source_write (const struct column *column, casenumber row,
         return false;
     }
 
-  return sparse_xarray_write (source->data, row, column->byte_ofs,
-                              width_to_n_bytes (column->width),
-                              value_to_data (value, column->width));
+  for (i = 0; i < n; i++)
+    if (!sparse_xarray_write (source->data, row, columns[i].byte_ofs,
+                              width_to_n_bytes (columns[i].width),
+                              value_to_data (&values[i], columns[i].width)))
+      return false;
+  return true;
 }
 
 /* Within SOURCE, which must not have a backing casereader,