X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;ds=sidebyside;f=src%2Flanguage%2Fdata-io%2Fmatrix-data.c;h=a002e78dbc2265f1ab4813218825524db6ae6521;hb=e7d0a9f16192ceeff9243f0ede8e399ee1ef0d44;hp=eb2d226f29341bbe4933f3219f8ca005c98bd941;hpb=0d7b430cd1cef60b963a54307622d6f4aa8efeb4;p=pspp-builds.git diff --git a/src/language/data-io/matrix-data.c b/src/language/data-io/matrix-data.c index eb2d226f..a002e78d 100644 --- a/src/language/data-io/matrix-data.c +++ b/src/language/data-io/matrix-data.c @@ -27,20 +27,24 @@ #include #include #include +#include #include #include #include #include #include +#include #include #include +#include #include #include #include #include #include #include -#include + +#include "size_max.h" #include "gettext.h" #define _(msgid) gettext (msgid) @@ -111,7 +115,7 @@ static const int content_type[PROX + 1] = }; /* Name of each content type. */ -static const char *content_names[PROX + 1] = +static const char *const content_names[PROX + 1] = { "N", "N", "N_MATRIX", "MEAN", "STDDEV", "COUNT", "MSE", "DFE", "MAT", "COV", "CORR", "PROX", @@ -148,7 +152,7 @@ struct matrix_data_pgm /* Continuous variables. */ int n_continuous; /* Number of continuous variables. */ - int first_continuous; /* Index into default_dict.var of + int first_continuous; /* Index into dictionary of first continuous variable. */ }; @@ -164,13 +168,13 @@ static const struct case_source_class matrix_data_without_rowtype_source_class; static int compare_variables_by_mxd_var_type (const void *pa, const void *pb); -static bool read_matrices_without_rowtype (struct matrix_data_pgm *); -static bool read_matrices_with_rowtype (struct matrix_data_pgm *); +static bool read_matrices_without_rowtype (struct dataset *ds, struct matrix_data_pgm *); +static bool read_matrices_with_rowtype (struct dataset *ds, struct matrix_data_pgm *); static int string_to_content_type (char *, int *); static void attach_mxd_aux (struct variable *, int var_type, int sub_type); int -cmd_matrix_data (void) +cmd_matrix_data (struct dataset *ds) { struct pool *pool; struct matrix_data_pgm *mx; @@ -179,7 +183,7 @@ cmd_matrix_data (void) unsigned seen = 0; - discard_variables (); + discard_variables (ds); pool = pool_create (); mx = pool_alloc (pool, sizeof *mx); @@ -244,7 +248,7 @@ cmd_matrix_data (void) if (strcasecmp (v[i], "ROWTYPE_")) { - new_var = dict_create_var_assert (default_dict, v[i], 0); + new_var = dict_create_var_assert (dataset_dict (ds), v[i], 0); attach_mxd_aux (new_var, MXD_CONTINUOUS, i); } else @@ -254,7 +258,7 @@ cmd_matrix_data (void) free (v); } - mx->rowtype_ = dict_create_var_assert (default_dict, + mx->rowtype_ = dict_create_var_assert (dataset_dict (ds), "ROWTYPE_", 8); attach_mxd_aux (mx->rowtype_, MXD_ROWTYPE, 0); } @@ -309,7 +313,7 @@ cmd_matrix_data (void) goto lossage; } - if (dict_lookup_var (default_dict, tokid) == NULL + if (dict_lookup_var (dataset_dict (ds), tokid) == NULL && (lex_look_ahead () == '.' || lex_look_ahead () == '/')) { if (!strcasecmp (tokid, "ROWTYPE_") @@ -320,27 +324,27 @@ cmd_matrix_data (void) goto lossage; } - mx->single_split = dict_create_var_assert (default_dict, + mx->single_split = dict_create_var_assert (dataset_dict (ds), tokid, 0); attach_mxd_aux (mx->single_split, MXD_CONTINUOUS, 0); lex_get (); - dict_set_split_vars (default_dict, &mx->single_split, 1); + dict_set_split_vars (dataset_dict (ds), &mx->single_split, 1); } else { struct variable **split; size_t n; - if (!parse_variables (default_dict, &split, &n, PV_NO_DUPLICATE)) + if (!parse_variables (dataset_dict (ds), &split, &n, PV_NO_DUPLICATE)) goto lossage; - dict_set_split_vars (default_dict, split, n); + dict_set_split_vars (dataset_dict (ds), split, n); } { - struct variable *const *split = dict_get_split_vars (default_dict); - size_t split_cnt = dict_get_split_cnt (default_dict); + struct variable *const *split = dict_get_split_vars (dataset_dict (ds)); + size_t split_cnt = dict_get_split_cnt (dataset_dict (ds)); int i; for (i = 0; i < split_cnt; i++) @@ -369,7 +373,7 @@ cmd_matrix_data (void) } seen |= 4; - if (!parse_variables (default_dict, &mx->factors, &mx->n_factors, + if (!parse_variables (dataset_dict (ds), &mx->factors, &mx->n_factors, PV_NONE)) goto lossage; @@ -568,7 +572,7 @@ cmd_matrix_data (void) } /* Create VARNAME_. */ - mx->varname_ = dict_create_var_assert (default_dict, "VARNAME_", 8); + mx->varname_ = dict_create_var_assert (dataset_dict (ds), "VARNAME_", 8); attach_mxd_aux (mx->varname_, MXD_VARNAME, 0); /* Sort the dictionary variables into the desired order for the @@ -577,9 +581,9 @@ cmd_matrix_data (void) struct variable **v; size_t nv; - dict_get_vars (default_dict, &v, &nv, 0); + dict_get_vars (dataset_dict (ds), &v, &nv, 0); qsort (v, nv, sizeof *v, compare_variables_by_mxd_var_type); - dict_reorder_vars (default_dict, v, nv); + dict_reorder_vars (dataset_dict (ds), v, nv); free (v); } @@ -597,9 +601,9 @@ cmd_matrix_data (void) int i; mx->first_continuous = -1; - for (i = 0; i < dict_get_var_cnt (default_dict); i++) + for (i = 0; i < dict_get_var_cnt (dataset_dict (ds)); i++) { - struct variable *v = dict_get_var (default_dict, i); + struct variable *v = dict_get_var (dataset_dict (ds), i); struct mxd_var *mv = v->aux; int type = mv->var_type; @@ -624,9 +628,9 @@ cmd_matrix_data (void) goto lossage; if (mx->explicit_rowtype) - ok = read_matrices_with_rowtype (mx); + ok = read_matrices_with_rowtype (ds, mx); else - ok = read_matrices_without_rowtype (mx); + ok = read_matrices_without_rowtype (ds, mx); dfm_close_reader (mx->reader); @@ -635,7 +639,7 @@ cmd_matrix_data (void) return ok ? CMD_SUCCESS : CMD_CASCADING_FAILURE; lossage: - discard_variables (); + discard_variables (ds); free (mx->factors); pool_destroy (mx->container); return CMD_CASCADING_FAILURE; @@ -751,7 +755,7 @@ mdump_token (const struct matrix_token *token) printf (" '%.*s'", token->length, token->string); break; default: - assert (0); + NOT_REACHED (); } fflush (stdout); } @@ -769,138 +773,115 @@ mget_token_dump (struct matrix_token *token, struct dfm_reader *reader) static const char * context (struct dfm_reader *reader) { - static char buf[32]; + static struct string buf = DS_EMPTY_INITIALIZER; + ds_clear (&buf); if (dfm_eof (reader)) - strcpy (buf, "at end of file"); + ds_assign_cstr (&buf, "at end of file"); else { - struct fixed_string line; - const char *sp; + struct substring p; - dfm_get_record (reader, &line); - sp = ls_c_str (&line); - while (sp < ls_end (&line) && isspace ((unsigned char) *sp)) - sp++; - if (sp >= ls_end (&line)) - strcpy (buf, "at end of line"); + p = dfm_get_record (reader); + ss_ltrim (&p, ss_cstr (CC_SPACES)); + if (ss_is_empty (p)) + ds_assign_cstr (&buf, "at end of line"); else - { - char *dp; - size_t copy_cnt = 0; - - dp = stpcpy (buf, "before `"); - while (sp < ls_end (&line) && !isspace ((unsigned char) *sp) - && copy_cnt < 10) - { - *dp++ = *sp++; - copy_cnt++; - } - strcpy (dp, "'"); - } + ds_put_format (&buf, "before `%.*s'", + (int) ss_cspan (p, ss_cstr (CC_SPACES)), ss_data (p)); } - return buf; + return ds_cstr (&buf); } /* Is there at least one token left in the data file? */ -static int +static bool another_token (struct dfm_reader *reader) { for (;;) { - struct fixed_string line; - const char *cp; + struct substring p; + size_t space_cnt; if (dfm_eof (reader)) - return 0; - dfm_get_record (reader, &line); - - cp = ls_c_str (&line); - while (isspace ((unsigned char) *cp) && cp < ls_end (&line)) - cp++; + return false; - if (cp < ls_end (&line)) + p = dfm_get_record (reader); + space_cnt = ss_span (p, ss_cstr (CC_SPACES)); + if (space_cnt < ss_length (p)) { - dfm_forward_columns (reader, cp - ls_c_str (&line)); - return 1; + dfm_forward_columns (reader, space_cnt); + return true; } dfm_forward_record (reader); } + NOT_REACHED(); } /* Parse a MATRIX DATA token from READER into TOKEN. */ static int (mget_token) (struct matrix_token *token, struct dfm_reader *reader) { - struct fixed_string line; - int first_column; - char *cp; + struct substring line, p; + struct substring s; + int c; if (!another_token (reader)) return 0; - dfm_get_record (reader, &line); - first_column = dfm_column_start (reader); + line = p = dfm_get_record (reader); /* Three types of fields: quoted with ', quoted with ", unquoted. */ - cp = ls_c_str (&line); - if (*cp == '\'' || *cp == '"') + c = ss_first (p); + if (c == '\'' || c == '"') { - int quote = *cp; - - token->type = MSTR; - token->string = ++cp; - while (cp < ls_end (&line) && *cp != quote) - cp++; - token->length = cp - token->string; - if (cp < ls_end (&line)) - cp++; - else - msg (SW, _("Scope of string exceeds line.")); + ss_get_char (&p); + if (!ss_get_until (&p, c, &s)) + msg (SW, _("Scope of string exceeds line.")); } else { - int is_num = isdigit ((unsigned char) *cp) || *cp == '.'; - - token->string = cp++; - while (cp < ls_end (&line) - && !isspace ((unsigned char) *cp) && *cp != ',' - && *cp != '-' && *cp != '+') - { - if (isdigit ((unsigned char) *cp)) - is_num = 1; - - if ((tolower ((unsigned char) *cp) == 'd' - || tolower ((unsigned char) *cp) == 'e') - && (cp[1] == '+' || cp[1] == '-')) - cp += 2; - else - cp++; - } + bool is_num = isdigit (c) || c == '.'; + const char *start = ss_data (p); - token->length = cp - token->string; - assert (token->length); + for (;;) + { + c = ss_first (p); + if (strchr (CC_SPACES ",-+", c) != NULL) + break; + + if (isdigit (c)) + is_num = true; + if (strchr ("deDE", c) && strchr ("+-", ss_at (p, 1))) + { + is_num = true; + ss_advance (&p, 2); + } + else + ss_advance (&p, 1); + } + s = ss_buffer (start, ss_data (p) - start); if (is_num) { struct data_in di; - di.s = token->string; - di.e = token->string + token->length; + di.s = ss_data (s); + di.e = ss_end (s); di.v = (union value *) &token->number; - di.f1 = first_column; + di.f1 = dfm_get_column (reader, di.s); di.format = make_output_format (FMT_F, token->length, 0); - if (!data_in (&di)) - return 0; + data_in (&di); } else token->type = MSTR; } - - dfm_forward_columns (reader, cp - ls_c_str (&line)); + token->string = ss_data (s); + token->length = ss_length (s); + + dfm_reread_record (reader, dfm_get_column (reader, ss_end (s))); return 1; } @@ -910,18 +891,13 @@ static int static int force_eol (struct dfm_reader *reader, const char *content) { - struct fixed_string line; - const char *cp; + struct substring p; if (dfm_eof (reader)) return 0; - dfm_get_record (reader, &line); - cp = ls_c_str (&line); - while (isspace ((unsigned char) *cp) && cp < ls_end (&line)) - cp++; - - if (cp < ls_end (&line)) + p = dfm_get_record (reader); + if (ss_span (p, ss_cstr (CC_SPACES)) != ss_length (p)) { msg (SE, _("End of line expected %s while reading %s."), context (reader), content); @@ -936,6 +912,7 @@ force_eol (struct dfm_reader *reader, const char *content) struct nr_aux_data { + const struct dictionary *dict; /* The dictionary */ struct matrix_data_pgm *mx; /* MATRIX DATA program. */ double ***data; /* MATRIX DATA data. */ double *factor_values; /* Factor values. */ @@ -944,8 +921,8 @@ struct nr_aux_data double *split_values; /* SPLIT FILE variable values. */ }; -static int nr_read_splits (struct nr_aux_data *, int compare); -static int nr_read_factors (struct nr_aux_data *, int cell); +static bool nr_read_splits (struct nr_aux_data *, int compare); +static bool nr_read_factors (struct nr_aux_data *, int cell); static bool nr_output_data (struct nr_aux_data *, struct ccase *, write_case_func *, write_case_data); static bool matrix_data_read_without_rowtype (struct case_source *source, @@ -956,7 +933,7 @@ static bool matrix_data_read_without_rowtype (struct case_source *source, /* Read from the data file and write it to the active file. Returns true if successful, false if an I/O error occurred. */ static bool -read_matrices_without_rowtype (struct matrix_data_pgm *mx) +read_matrices_without_rowtype (struct dataset *ds, struct matrix_data_pgm *mx) { struct nr_aux_data nr; bool ok; @@ -965,16 +942,18 @@ read_matrices_without_rowtype (struct matrix_data_pgm *mx) mx->cells = 1; nr.mx = mx; + nr.dict = dataset_dict (ds); nr.data = NULL; nr.factor_values = xnmalloc (mx->n_factors * mx->cells, sizeof *nr.factor_values); nr.max_cell_idx = 0; - nr.split_values = xnmalloc (dict_get_split_cnt (default_dict), + nr.split_values = xnmalloc (dict_get_split_cnt (dataset_dict (ds)), sizeof *nr.split_values); - vfm_source = create_case_source (&matrix_data_without_rowtype_source_class, &nr); + proc_set_source (ds, create_case_source ( + &matrix_data_without_rowtype_source_class, &nr)); - ok = procedure (NULL, NULL); + ok = procedure (ds, NULL, NULL); free (nr.split_values); free (nr.factor_values); @@ -1091,16 +1070,14 @@ nr_read_data_lines (struct nr_aux_data *nr, n_cols = mx->n_continuous; break; default: - assert (0); - abort (); + NOT_REACHED (); } break; case 2: n_cols = 1; break; default: - assert (0); - abort (); + NOT_REACHED (); } { @@ -1114,7 +1091,7 @@ nr_read_data_lines (struct nr_aux_data *nr, if (token.type != MNUM) { msg (SE, _("expecting value for %s %s"), - dict_get_var (default_dict, j)->name, + dict_get_var (nr->dict, j)->name, context (mx->reader)); return 0; } @@ -1231,16 +1208,16 @@ matrix_data_read_without_rowtype (struct case_source *source, if (!nr_output_data (nr, c, write_case, wc_data)) return false; - if (dict_get_split_cnt (default_dict) == 0 + if (dict_get_split_cnt (nr->dict) == 0 || !another_token (mx->reader)) return true; } } /* Read the split file variables. If COMPARE is 1, compares the - values read to the last values read and returns 1 if they're equal, - 0 otherwise. */ -static int + values read to the last values read and returns true if they're equal, + false otherwise. */ +static bool nr_read_splits (struct nr_aux_data *nr, int compare) { struct matrix_data_pgm *mx = nr->mx; @@ -1251,36 +1228,36 @@ nr_read_splits (struct nr_aux_data *nr, int compare) if (compare && just_read) { just_read = 0; - return 1; + return true; } - if (dict_get_split_vars (default_dict) == NULL) - return 1; + if (dict_get_split_vars (nr->dict) == NULL) + return true; if (mx->single_split) { if (!compare) { - struct mxd_var *mv = dict_get_split_vars (default_dict)[0]->aux; + struct mxd_var *mv = dict_get_split_vars (nr->dict)[0]->aux; nr->split_values[0] = ++mv->sub_type; } - return 1; + return true; } if (!compare) just_read = 1; - split_cnt = dict_get_split_cnt (default_dict); + split_cnt = dict_get_split_cnt (nr->dict); for (i = 0; i < split_cnt; i++) { struct matrix_token token; if (!mget_token (&token, mx->reader)) - return 0; + return false; if (token.type != MNUM) { msg (SE, _("Syntax error expecting SPLIT FILE value %s."), context (mx->reader)); - return 0; + return false; } if (!compare) @@ -1289,32 +1266,32 @@ nr_read_splits (struct nr_aux_data *nr, int compare) { msg (SE, _("Expecting value %g for %s."), nr->split_values[i], - dict_get_split_vars (default_dict)[i]->name); - return 0; + dict_get_split_vars (nr->dict)[i]->name); + return false; } } - return 1; + return true; } /* Read the factors for cell CELL. If COMPARE is 1, compares the - values read to the last values read and returns 1 if they're equal, - 0 otherwise. */ -static int + values read to the last values read and returns true if they're equal, + false otherwise. */ +static bool nr_read_factors (struct nr_aux_data *nr, int cell) { struct matrix_data_pgm *mx = nr->mx; - int compare; + bool compare; if (mx->n_factors == 0) - return 1; + return true; assert (nr->max_cell_idx >= cell); if (cell != nr->max_cell_idx) - compare = 1; + compare = true; else { - compare = 0; + compare = false; nr->max_cell_idx++; } @@ -1325,12 +1302,12 @@ nr_read_factors (struct nr_aux_data *nr, int cell) { struct matrix_token token; if (!mget_token (&token, mx->reader)) - return 0; + return false; if (token.type != MNUM) { msg (SE, _("Syntax error expecting factor value %s."), context (mx->reader)); - return 0; + return false; } if (!compare) @@ -1340,19 +1317,20 @@ nr_read_factors (struct nr_aux_data *nr, int cell) msg (SE, _("Syntax error expecting value %g for %s %s."), nr->factor_values[i + mx->n_factors * cell], mx->factors[i]->name, context (mx->reader)); - return 0; + return false; } } } - return 1; + return true; } /* Write the contents of a cell having content type CONTENT and data CP to the active file. Returns true if successful, false if an I/O error occurred. */ static bool -dump_cell_content (struct matrix_data_pgm *mx, int content, double *cp, +dump_cell_content (const struct dictionary *dict, + struct matrix_data_pgm *mx, int content, double *cp, struct ccase *c, write_case_func *write_case, write_case_data wc_data) { @@ -1376,13 +1354,13 @@ dump_cell_content (struct matrix_data_pgm *mx, int content, double *cp, for (j = 0; j < mx->n_continuous; j++) { - int fv = dict_get_var (default_dict, mx->first_continuous + j)->fv; + int fv = dict_get_var (dict, mx->first_continuous + j)->fv; case_data_rw (c, fv)->f = *cp; cp++; } if (type == 1) buf_copy_str_rpad (case_data_rw (c, mx->varname_->fv)->s, 8, - dict_get_var (default_dict, + dict_get_var (dict, mx->first_continuous + i)->name); if (!write_case (wc_data)) return false; @@ -1403,8 +1381,8 @@ nr_output_data (struct nr_aux_data *nr, struct ccase *c, size_t split_cnt; size_t i; - split_cnt = dict_get_split_cnt (default_dict); - split = dict_get_split_vars (default_dict); + split_cnt = dict_get_split_cnt (nr->dict); + split = dict_get_split_vars (nr->dict); for (i = 0; i < split_cnt; i++) case_data_rw (c, split[i]->fv)->f = nr->split_values[i]; } @@ -1432,7 +1410,8 @@ nr_output_data (struct nr_aux_data *nr, struct ccase *c, assert (nr->data[content] != NULL && nr->data[content][cell] != NULL); - if (!dump_cell_content (mx, content, nr->data[content][cell], + if (!dump_cell_content (nr->dict, mx, + content, nr->data[content][cell], c, write_case, wc_data)) return false; } @@ -1453,7 +1432,7 @@ nr_output_data (struct nr_aux_data *nr, struct ccase *c, for (content = 0; content <= PROX; content++) if (!mx->is_per_factor[content] && nr->data[content] != NULL) { - if (!dump_cell_content (mx, content, nr->data[content][0], + if (!dump_cell_content (nr->dict, mx, content, nr->data[content][0], c, write_case, wc_data)) return false; } @@ -1476,6 +1455,7 @@ struct factor_data /* With ROWTYPE_ auxiliary data. */ struct wr_aux_data { + const struct dictionary *dict; /* The dictionary */ struct matrix_data_pgm *mx; /* MATRIX DATA program. */ int content; /* Type of current row. */ double *split_values; /* SPLIT FILE variable values. */ @@ -1483,14 +1463,14 @@ struct wr_aux_data struct factor_data *current; /* Current factor. */ }; -static int wr_read_splits (struct wr_aux_data *, struct ccase *, +static bool wr_read_splits (struct wr_aux_data *, struct ccase *, write_case_func *, write_case_data); static bool wr_output_data (struct wr_aux_data *, struct ccase *, write_case_func *, write_case_data); -static int wr_read_rowtype (struct wr_aux_data *, +static bool wr_read_rowtype (struct wr_aux_data *, const struct matrix_token *, struct dfm_reader *); -static int wr_read_factors (struct wr_aux_data *); -static int wr_read_indeps (struct wr_aux_data *); +static bool wr_read_factors (struct wr_aux_data *); +static bool wr_read_indeps (struct wr_aux_data *); static bool matrix_data_read_with_rowtype (struct case_source *, struct ccase *, write_case_func *, @@ -1500,7 +1480,7 @@ static bool matrix_data_read_with_rowtype (struct case_source *, them to the output file. Returns true if successful, false if an I/O error occurred. */ static bool -read_matrices_with_rowtype (struct matrix_data_pgm *mx) +read_matrices_with_rowtype (struct dataset *ds, struct matrix_data_pgm *mx) { struct wr_aux_data wr; bool ok; @@ -1510,11 +1490,13 @@ read_matrices_with_rowtype (struct matrix_data_pgm *mx) wr.split_values = NULL; wr.data = NULL; wr.current = NULL; + wr.dict = dataset_dict (ds); mx->cells = 0; - vfm_source = create_case_source (&matrix_data_with_rowtype_source_class, - &wr); - ok = procedure (NULL, NULL); + proc_set_source (ds, + create_case_source (&matrix_data_with_rowtype_source_class, + &wr)); + ok = procedure (ds, NULL, NULL); free (wr.split_values); return ok; @@ -1549,60 +1531,60 @@ matrix_data_read_with_rowtype (struct case_source *source, /* Read the split file variables. If they differ from the previous set of split variables then output the data. Returns success. */ -static int +static bool wr_read_splits (struct wr_aux_data *wr, struct ccase *c, write_case_func *write_case, write_case_data wc_data) { struct matrix_data_pgm *mx = wr->mx; - int compare; + bool compare; size_t split_cnt; - split_cnt = dict_get_split_cnt (default_dict); + split_cnt = dict_get_split_cnt (wr->dict); if (split_cnt == 0) - return 1; + return true; if (wr->split_values) - compare = 1; + compare = true; else { - compare = 0; + compare = false; wr->split_values = xnmalloc (split_cnt, sizeof *wr->split_values); } { - int different = 0; + bool different = false; int i; for (i = 0; i < split_cnt; i++) { struct matrix_token token; if (!mget_token (&token, mx->reader)) - return 0; + return false; if (token.type != MNUM) { msg (SE, _("Syntax error %s expecting SPLIT FILE value."), context (mx->reader)); - return 0; + return false; } if (compare && wr->split_values[i] != token.number && !different) { if (!wr_output_data (wr, c, write_case, wc_data)) return 0; - different = 1; + different = true; mx->cells = 0; } wr->split_values[i] = token.number; } } - return 1; + return true; } /* Compares doubles A and B, treating SYSMIS as greatest. */ static int -compare_doubles (const void *a_, const void *b_, void *aux UNUSED) +compare_doubles (const void *a_, const void *b_, const void *aux UNUSED) { const double *a = a_; const double *b = b_; @@ -1622,9 +1604,9 @@ compare_doubles (const void *a_, const void *b_, void *aux UNUSED) /* Return strcmp()-type comparison of the MX->n_factors factors at _A and _B. Sort missing values toward the end. */ static int -compare_factors (const void *a_, const void *b_, void *mx_) +compare_factors (const void *a_, const void *b_, const void *mx_) { - struct matrix_data_pgm *mx = mx_; + const struct matrix_data_pgm *mx = mx_; struct factor_data *const *pa = a_; struct factor_data *const *pb = b_; const double *a = (*pa)->factors; @@ -1652,8 +1634,8 @@ wr_output_data (struct wr_aux_data *wr, size_t split_cnt; size_t i; - split_cnt = dict_get_split_cnt (default_dict); - split = dict_get_split_vars (default_dict); + split_cnt = dict_get_split_cnt (wr->dict); + split = dict_get_split_vars (wr->dict); for (i = 0; i < split_cnt; i++) case_data_rw (c, split[i]->fv)->f = wr->split_values[i]; } @@ -1721,7 +1703,8 @@ wr_output_data (struct wr_aux_data *wr, fill_matrix (mx, content, iter->data[content]); - ok = dump_cell_content (mx, content, iter->data[content], + ok = dump_cell_content (wr->dict, mx, content, + iter->data[content], c, write_case, wc_data); if (!ok) break; @@ -1740,7 +1723,7 @@ wr_output_data (struct wr_aux_data *wr, /* Sets ROWTYPE_ based on the given TOKEN read from READER. Return success. */ -static int +static bool wr_read_rowtype (struct wr_aux_data *wr, const struct matrix_token *token, struct dfm_reader *reader) @@ -1748,13 +1731,13 @@ wr_read_rowtype (struct wr_aux_data *wr, if (wr->content != -1) { msg (SE, _("Multiply specified ROWTYPE_ %s."), context (reader)); - return 0; + return false; } if (token->type != MSTR) { msg (SE, _("Syntax error %s expecting ROWTYPE_ string."), context (reader)); - return 0; + return false; } { @@ -1776,12 +1759,12 @@ wr_read_rowtype (struct wr_aux_data *wr, return 0; } - return 1; + return true; } /* Read the factors for the current row. Select a set of factors and point wr_current to it. */ -static int +static bool wr_read_factors (struct wr_aux_data *wr) { struct matrix_data_pgm *mx = wr->mx; @@ -1885,15 +1868,15 @@ cache_miss: winnage: local_free (factor_values); - return 1; + return true; lossage: local_free (factor_values); - return 0; + return false; } /* Read the independent variables into wr->current. */ -static int +static bool wr_read_indeps (struct wr_aux_data *wr) { struct matrix_data_pgm *mx = wr->mx; @@ -1925,7 +1908,7 @@ wr_read_indeps (struct wr_aux_data *wr) { msg (SE, _("Duplicate specification for %s."), content_names[wr->content]); - return 0; + return false; } if (type == 0) n_cols = mx->n_continuous; @@ -1937,7 +1920,7 @@ wr_read_indeps (struct wr_aux_data *wr) { msg (SE, _("Too many rows of matrix data for %s."), content_names[wr->content]); - return 0; + return false; } switch (mx->section) @@ -1960,13 +1943,11 @@ wr_read_indeps (struct wr_aux_data *wr) n_cols = mx->n_continuous; break; default: - assert (0); - abort (); + NOT_REACHED (); } break; default: - assert (0); - abort (); + NOT_REACHED (); } c->n_rows[wr->content]++; @@ -1978,23 +1959,23 @@ wr_read_indeps (struct wr_aux_data *wr) { struct matrix_token token; if (!mget_token (&token, mx->reader)) - return 0; + return false; if (token.type != MNUM) { msg (SE, _("Syntax error expecting value for %s %s."), - dict_get_var (default_dict, mx->first_continuous + j)->name, + dict_get_var (wr->dict, mx->first_continuous + j)->name, context (mx->reader)); - return 0; + return false; } *cp++ = token.number; } if (mx->fmt != FREE && !force_eol (mx->reader, content_names[wr->content])) - return 0; + return false; } - return 1; + return true; } /* Matrix source. */