projects
/
pspp
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
lexer: Remove DUMP_TOKENS debugging feature.
[pspp]
/
src
/
language
/
stats
/
aggregate.c
diff --git
a/src/language/stats/aggregate.c
b/src/language/stats/aggregate.c
index 7625a6f1a76496803ee5b7e20bcb345252107009..50a98e483efeed4ede0fa6336692b2e57e0394c3 100644
(file)
--- a/
src/language/stats/aggregate.c
+++ b/
src/language/stats/aggregate.c
@@
-87,16
+87,6
@@
struct agr_var
struct casewriter *writer;
};
struct casewriter *writer;
};
-/* Aggregation functions. */
-enum
- {
- SUM, MEAN, MEDIAN, SD, MAX, MIN, PGT, PLT, PIN, POUT, FGT, FLT, FIN,
- FOUT, N, NU, NMISS, NUMISS, FIRST, LAST,
-
- FUNC = 0x1f, /* Function mask. */
- FSTRING = 1<<5, /* String function bit. */
- };
-
/* Attributes of aggregation functions. */
const struct agr_func agr_func_tab[] =
/* Attributes of aggregation functions. */
const struct agr_func agr_func_tab[] =
@@
-232,7
+222,7
@@
cmd_aggregate (struct lexer *lexer, struct dataset *ds)
lex_match (lexer, '=');
if (!lex_match_id (lexer, "COLUMNWISE"))
{
lex_match (lexer, '=');
if (!lex_match_id (lexer, "COLUMNWISE"))
{
- lex_error (lexer, _("
while expecting COLUMNWISE")
);
+ lex_error (lexer, _("
expecting %s"), "COLUMNWISE"
);
goto error;
}
agr.missing = COLUMNWISE;
goto error;
}
agr.missing = COLUMNWISE;
@@
-241,7
+231,7
@@
cmd_aggregate (struct lexer *lexer, struct dataset *ds)
copy_documents = true;
else if (lex_match_id (lexer, "PRESORTED"))
presorted = true;
copy_documents = true;
else if (lex_match_id (lexer, "PRESORTED"))
presorted = true;
- else if (lex_match_id (lexer, "BREAK"))
+ else if (lex_
force_
match_id (lexer, "BREAK"))
{
int i;
{
int i;
@@
-259,10
+249,8
@@
cmd_aggregate (struct lexer *lexer, struct dataset *ds)
break;
}
else
break;
}
else
- {
- lex_error (lexer, _("expecting BREAK"));
- goto error;
- }
+ goto error;
+
}
if (presorted && saw_direction)
msg (SW, _("When PRESORTED is specified, specifying sorting directions "
}
if (presorted && saw_direction)
msg (SW, _("When PRESORTED is specified, specifying sorting directions "
@@
-463,14
+451,8
@@
parse_aggregate_functions (struct lexer *lexer, const struct dictionary *dict,
goto error;
}
goto error;
}
- exclude = MV_ANY;
-
ds_assign_string (&function_name, lex_tokstr (lexer));
ds_assign_string (&function_name, lex_tokstr (lexer));
-
- ds_chomp (&function_name, '.');
-
- if (lex_tokid(lexer)[strlen (lex_tokid (lexer)) - 1] == '.')
- exclude = MV_SYSTEM;
+ exclude = ds_chomp (&function_name, '.') ? MV_SYSTEM : MV_ANY;
for (function = agr_func_tab; function->name; function++)
if (!strcasecmp (function->name, ds_cstr (&function_name)))
for (function = agr_func_tab; function->name; function++)
if (!strcasecmp (function->name, ds_cstr (&function_name)))
@@
-490,7
+472,7
@@
parse_aggregate_functions (struct lexer *lexer, const struct dictionary *dict,
{
if (function->src_vars == AGR_SV_YES)
{
{
if (function->src_vars == AGR_SV_YES)
{
-
lex_error (lexer, _("expecting `('")
);
+
lex_force_match (lexer, '('
);
goto error;
}
}
goto error;
}
}
@@
-546,11
+528,8
@@
parse_aggregate_functions (struct lexer *lexer, const struct dictionary *dict,
}
/* Trailing rparen. */
}
/* Trailing rparen. */
- if (!lex_match (lexer, ')'))
- {
- lex_error (lexer, _("expecting `)'"));
- goto error;
- }
+ if (!lex_force_match (lexer, ')'))
+ goto error;
/* Now check that the number of source variables match
the number of target variables. If we check earlier
/* Now check that the number of source variables match
the number of target variables. If we check earlier