projects
/
pspp
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Terminal interface: Remove support for the ncurses library.
[pspp]
/
src
/
language
/
stats
/
t-test-parser.c
diff --git
a/src/language/stats/t-test-parser.c
b/src/language/stats/t-test-parser.c
index 7c8d8a548a5ea42b7343e5fe8b5b716c744bc37a..9a0043aa45761db7771a25a96a2f7db02b046e39 100644
(file)
--- a/
src/language/stats/t-test-parser.c
+++ b/
src/language/stats/t-test-parser.c
@@
-1,5
+1,5
@@
/* PSPP - a program for statistical analysis.
/* PSPP - a program for statistical analysis.
- Copyright (C) 2011 Free Software Foundation, Inc.
+ Copyright (C) 2011
, 2015
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
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
@@
-44,22
+44,22
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
/* Variables pertaining to the paired mode */
const struct variable **v1 = NULL;
/* Variables pertaining to the paired mode */
const struct variable **v1 = NULL;
- size_t n_v1;
+ size_t n_v1
= 0
;
const struct variable **v2 = NULL;
const struct variable **v2 = NULL;
- size_t n_v2;
+ size_t n_v2
= 0
;
- size_t n_pairs;
+ size_t n_pairs
= 0
;
vp *pairs = NULL;
/* One sample mode */
vp *pairs = NULL;
/* One sample mode */
- double testval;
+ double testval
= SYSMIS
;
/* Independent samples mode */
const struct variable *gvar;
union value gval0;
union value gval1;
/* Independent samples mode */
const struct variable *gvar;
union value gval0;
union value gval1;
- bool cut;
+ bool cut
= false
;
tt.wv = dict_get_weight (dict);
tt.dict = dict;
tt.wv = dict_get_weight (dict);
tt.dict = dict;
@@
-80,7
+80,8
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
mode_count++;
tt.mode = MODE_SINGLE;
lex_match (lexer, T_EQUALS);
mode_count++;
tt.mode = MODE_SINGLE;
lex_match (lexer, T_EQUALS);
- lex_force_num (lexer);
+ if (!lex_force_num (lexer))
+ goto parse_failed;
testval = lex_number (lexer);
lex_get (lexer);
}
testval = lex_number (lexer);
lex_get (lexer);
}
@@
-100,14
+101,16
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
value_init (&gval0, var_get_width (gvar));
parse_value (lexer, &gval0, gvar);
cut = true;
value_init (&gval0, var_get_width (gvar));
parse_value (lexer, &gval0, gvar);
cut = true;
- if (lex_
match (lexer, T_COMMA)
)
+ if (lex_
token (lexer) != T_RPAREN
)
{
{
+ lex_match (lexer, T_COMMA);
value_init (&gval1, var_get_width (gvar));
parse_value (lexer, &gval1, gvar);
cut = false;
}
value_init (&gval1, var_get_width (gvar));
parse_value (lexer, &gval1, gvar);
cut = false;
}
- lex_force_match (lexer, T_RPAREN);
+ if (! lex_force_match (lexer, T_RPAREN))
+ goto parse_failed;
}
else
{
}
else
{
@@
-120,8
+123,8
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
if ( cut == true && var_is_alpha (gvar))
{
if ( cut == true && var_is_alpha (gvar))
{
- msg (SE, _("When applying
GROUPS
to a string variable, two "
- "values must be specified."));
+ msg (SE, _("When applying
%s
to a string variable, two "
+ "values must be specified.")
, "GROUPS"
);
goto parse_failed;
}
}
goto parse_failed;
}
}
@@
-132,7
+135,7
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
if (tt.n_vars > 0)
{
if (tt.n_vars > 0)
{
- msg (SE, _("
VARIABLES subcommand may not be used with PAIRS.")
);
+ msg (SE, _("
%s subcommand may not be used with %s."), "VARIABLES", "PAIRS"
);
goto parse_failed;
}
goto parse_failed;
}
@@
-178,8
+181,7
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
else
n_pairs = n_v1 * n_v2;
else
n_pairs = n_v1 * n_v2;
- pairs = xcalloc (sizeof *pairs, n_pairs);
-
+ pairs = xcalloc (n_pairs, sizeof *pairs);
if ( with)
{
if ( with)
{
@@
-229,7
+231,7
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
{
if ( tt.mode == MODE_PAIRED)
{
{
if ( tt.mode == MODE_PAIRED)
{
- msg (SE, _("
VARIABLES subcommand may not be used with PAIRS.")
);
+ msg (SE, _("
%s subcommand may not be used with %s."), "VARIABLES", "PAIRS"
);
goto parse_failed;
}
goto parse_failed;
}
@@
-273,13
+275,15
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
else if (lex_match_id (lexer, "CRITERIA"))
{
lex_match (lexer, T_EQUALS);
else if (lex_match_id (lexer, "CRITERIA"))
{
lex_match (lexer, T_EQUALS);
- if ( lex_
force_match_id (lexer, "CIN
"))
+ if ( lex_
match_id (lexer, "CIN") || lex_force_match_id (lexer, "CI
"))
if ( lex_force_match (lexer, T_LPAREN))
{
if ( lex_force_match (lexer, T_LPAREN))
{
- lex_force_num (lexer);
+ if (!lex_force_num (lexer))
+ goto parse_failed;
tt.confidence = lex_number (lexer);
lex_get (lexer);
tt.confidence = lex_number (lexer);
lex_get (lexer);
- lex_force_match (lexer, T_RPAREN);
+ if (! lex_force_match (lexer, T_RPAREN))
+ goto parse_failed;
}
}
else
}
}
else
@@
-298,7
+302,7
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
if (tt.n_vars == 0 && tt.mode != MODE_PAIRED)
{
if (tt.n_vars == 0 && tt.mode != MODE_PAIRED)
{
-
msg (SE, _("One or more VARIABLES must be specified.")
);
+
lex_sbc_missing ("VARIABLES"
);
goto parse_failed;
}
goto parse_failed;
}
@@
-363,12
+367,17
@@
cmd_t_test (struct lexer *lexer, struct dataset *ds)
free (pairs);
free (v1);
free (v2);
free (pairs);
free (v1);
free (v2);
-
free (tt.vars);
return ok ? CMD_SUCCESS : CMD_FAILURE;
parse_failed:
free (tt.vars);
return ok ? CMD_SUCCESS : CMD_FAILURE;
parse_failed:
+
+ free (pairs);
+ free (v1);
+ free (v2);
+ free (tt.vars);
+
return CMD_FAILURE;
}
return CMD_FAILURE;
}