projects
/
pspp-builds.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Merge commit 'origin/stable'
[pspp-builds.git]
/
src
/
data
/
por-file-writer.c
diff --git
a/src/data/por-file-writer.c
b/src/data/por-file-writer.c
index 8de293c5eafb2904d065db08f7779c9980f4e805..7b1b25512a89358b966d44416467dfd5e4bf2b10 100644
(file)
--- a/
src/data/por-file-writer.c
+++ b/
src/data/por-file-writer.c
@@
-45,6
+45,7
@@
#include <libpspp/str.h>
#include <libpspp/version.h>
#include <libpspp/str.h>
#include <libpspp/version.h>
+#include "minmax.h"
#include "xalloc.h"
#include "gettext.h"
#include "xalloc.h"
#include "gettext.h"
@@
-172,7
+173,7
@@
pfm_open_writer (struct file_handle *fh, struct dictionary *dict,
buf_write (w, "F", 1);
if (ferror (w->file))
goto error;
buf_write (w, "F", 1);
if (ferror (w->file))
goto error;
- return casewriter_create (dict_get_
next_value_idx
(dict),
+ return casewriter_create (dict_get_
proto
(dict),
&por_file_casewriter_class, w);
error:
&por_file_casewriter_class, w);
error:
@@
-306,17
+307,17
@@
write_format (struct pfm_writer *w, struct fmt_spec f, int width)
write_int (w, f.d);
}
write_int (w, f.d);
}
-/* Write value V
for variable VV
to file H. */
+/* Write value V
with width WIDTH
to file H. */
static void
static void
-write_value (struct pfm_writer *w,
union value *v, struct variable *vv
)
+write_value (struct pfm_writer *w,
const union value *v, int width
)
{
{
- if (
var_is_numeric (vv)
)
+ if (
width == 0
)
write_float (w, v->f);
else
{
write_float (w, v->f);
else
{
-
int width = MIN (var_get_width (vv)
, MAX_POR_WIDTH);
+
width = MIN (width
, MAX_POR_WIDTH);
write_int (w, width);
write_int (w, width);
- buf_write (w, v
->s
, width);
+ buf_write (w, v
alue_str (v, width)
, width);
}
}
}
}
@@
-343,6
+344,7
@@
write_variables (struct pfm_writer *w, struct dictionary *dict)
struct variable *v = dict_get_var (dict, i);
struct missing_values mv;
int width = MIN (var_get_width (v), MAX_POR_WIDTH);
struct variable *v = dict_get_var (dict, i);
struct missing_values mv;
int width = MIN (var_get_width (v), MAX_POR_WIDTH);
+ int j;
buf_write (w, "7", 1);
write_int (w, width);
buf_write (w, "7", 1);
write_int (w, width);
@@
-352,10
+354,12
@@
write_variables (struct pfm_writer *w, struct dictionary *dict)
/* Write missing values. */
mv_copy (&mv, var_get_missing_values (v));
/* Write missing values. */
mv_copy (&mv, var_get_missing_values (v));
- while (mv_has_range (&mv))
+ if (var_get_width (v) > 8)
+ mv_resize (&mv, 8);
+ if (mv_has_range (&mv))
{
double x, y;
{
double x, y;
- mv_
pop
_range (&mv, &x, &y);
+ mv_
get
_range (&mv, &x, &y);
if (x == LOWEST)
{
buf_write (w, "9", 1);
if (x == LOWEST)
{
buf_write (w, "9", 1);
@@
-373,13
+377,12
@@
write_variables (struct pfm_writer *w, struct dictionary *dict)
write_float (w, y);
}
}
write_float (w, y);
}
}
-
while (mv_has_value (&mv)
)
+
for (j = 0; j < mv_n_values (&mv); j++
)
{
{
- union value value;
- mv_pop_value (&mv, &value);
buf_write (w, "8", 1);
buf_write (w, "8", 1);
- write_value (w,
&value, v
);
+ write_value (w,
mv_get_value (&mv, j), mv_get_width (&mv)
);
}
}
+ mv_destroy (&mv);
/* Write variable label. */
if (var_get_label (v) != NULL)
/* Write variable label. */
if (var_get_label (v) != NULL)
@@
-398,12
+401,12
@@
write_value_labels (struct pfm_writer *w, const struct dictionary *dict)
for (i = 0; i < dict_get_var_cnt (dict); i++)
{
for (i = 0; i < dict_get_var_cnt (dict); i++)
{
- struct val_labs_iterator *j;
struct variable *v = dict_get_var (dict, i);
const struct val_labs *val_labs = var_get_value_labels (v);
struct variable *v = dict_get_var (dict, i);
const struct val_labs *val_labs = var_get_value_labels (v);
- struct val_lab *vl;
+ size_t n_labels = val_labs_count (val_labs);
+ const struct val_lab **labels;
- if (
val_labs == NULL
)
+ if (
n_labels == 0
)
continue;
buf_write (w, "D", 1);
continue;
buf_write (w, "D", 1);
@@
-411,12
+414,15
@@
write_value_labels (struct pfm_writer *w, const struct dictionary *dict)
write_string (w, var_get_short_name (v, 0));
write_int (w, val_labs_count (val_labs));
write_string (w, var_get_short_name (v, 0));
write_int (w, val_labs_count (val_labs));
- for (vl = val_labs_first_sorted (val_labs, &j); vl != NULL;
- vl = val_labs_next (val_labs, &j))
+ n_labels = val_labs_count (val_labs);
+ labels = val_labs_sorted (val_labs);
+ for (i = 0; i < n_labels; i++)
{
{
- write_value (w, &vl->value, v);
- write_string (w, vl->label);
+ const struct val_lab *vl = labels[i];
+ write_value (w, val_lab_get_value (vl), var_get_width (v));
+ write_string (w, val_lab_get_label (vl));
}
}
+ free (labels);
}
}
}
}