projects
/
pspp
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Work on docs
[pspp]
/
src
/
language
/
command.c
diff --git
a/src/language/command.c
b/src/language/command.c
index 8580a18374feeb38569834f9ad5cdb36bab69571..068ae0ea631a7f19da70898336a2414052205c55 100644
(file)
--- a/
src/language/command.c
+++ b/
src/language/command.c
@@
-1,5
+1,5
@@
/* PSPP - a program for statistical analysis.
/* PSPP - a program for statistical analysis.
- Copyright (C) 1997-9, 2000, 2009, 2010, 2011 Free Software Foundation, Inc.
+ Copyright (C) 1997-9, 2000, 2009, 2010, 2011
, 2012, 2013, 2014
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
@@
-36,9
+36,9
@@
#include "libpspp/i18n.h"
#include "libpspp/message.h"
#include "libpspp/str.h"
#include "libpspp/i18n.h"
#include "libpspp/message.h"
#include "libpspp/str.h"
-#include "output/text-item.h"
+#include "output/driver.h"
+#include "output/output-item.h"
-#include "xalloc.h"
#include "xmalloca.h"
#include "gettext.h"
#include "xmalloca.h"
#include "gettext.h"
@@
-50,10
+50,22
@@
static inline bool
cmd_result_is_valid (enum cmd_result result)
{
static inline bool
cmd_result_is_valid (enum cmd_result result)
{
- return (result == CMD_SUCCESS || result == CMD_EOF || result == CMD_FINISH
- || (result >= CMD_PRIVATE_FIRST && result <= CMD_PRIVATE_LAST)
- || result == CMD_FAILURE || result == CMD_NOT_IMPLEMENTED
- || result == CMD_CASCADING_FAILURE);
+ switch (result)
+ {
+ case CMD_SUCCESS:
+ case CMD_EOF:
+ case CMD_FINISH:
+ case CMD_DATA_LIST:
+ case CMD_END_CASE:
+ case CMD_END_FILE:
+ case CMD_FAILURE:
+ case CMD_NOT_IMPLEMENTED:
+ case CMD_CASCADING_FAILURE:
+ return true;
+
+ default:
+ return false;
+ }
}
/* Returns true if RESULT indicates success,
}
/* Returns true if RESULT indicates success,
@@
-138,7
+150,6
@@
cmd_parse_in_state (struct lexer *lexer, struct dataset *ds,
ds = session_active_dataset (session);
assert (!proc_is_open (ds));
unset_cmd_algorithm ();
ds = session_active_dataset (session);
assert (!proc_is_open (ds));
unset_cmd_algorithm ();
- dict_clear_aux (dataset_dict (ds));
if (!dataset_end_of_command (ds))
result = CMD_CASCADING_FAILURE;
if (!dataset_end_of_command (ds))
result = CMD_CASCADING_FAILURE;
@@
-163,8
+174,8
@@
do_parse_command (struct lexer *lexer,
struct dataset *ds, enum cmd_state state)
{
const struct command *command = NULL;
struct dataset *ds, enum cmd_state state)
{
const struct command *command = NULL;
+ size_t nesting_level = SIZE_MAX;
enum cmd_result result;
enum cmd_result result;
- bool opened = false;
int n_tokens;
/* Read the command's first token. */
int n_tokens;
/* Read the command's first token. */
@@
-188,8
+199,10
@@
do_parse_command (struct lexer *lexer,
result = CMD_FAILURE;
goto finish;
}
result = CMD_FAILURE;
goto finish;
}
- text_item_submit (text_item_create (TEXT_ITEM_COMMAND_OPEN, command->name));
- opened = true;
+
+ nesting_level = output_open_group (group_item_create_nocopy (
+ utf8_to_title (command->name),
+ utf8_to_title (command->name)));
if (command->function == NULL)
{
if (command->function == NULL)
{
@@
-231,12
+244,12
@@
finish:
result = lex_end_of_command (lexer);
lex_discard_rest_of_command (lexer);
result = lex_end_of_command (lexer);
lex_discard_rest_of_command (lexer);
- while (lex_token (lexer) == T_ENDCMD)
- lex_get (lexer);
+ if (result != CMD_EOF && result != CMD_FINISH)
+ while (lex_token (lexer) == T_ENDCMD)
+ lex_get (lexer);
- if (opened)
- text_item_submit (text_item_create (TEXT_ITEM_COMMAND_CLOSE,
- command->name));
+ if (nesting_level != SIZE_MAX)
+ output_close_groups (nesting_level);
return result;
}
return result;
}
@@
-362,7
+375,7
@@
report_state_mismatch (const struct command *command, enum cmd_state state)
assert (!in_correct_state (command, state));
if (state == CMD_STATE_INITIAL || state == CMD_STATE_DATA)
{
assert (!in_correct_state (command, state));
if (state == CMD_STATE_INITIAL || state == CMD_STATE_DATA)
{
- switch (command->states)
+ switch (
(int)
command->states)
{
/* One allowed state. */
case S_INITIAL:
{
/* One allowed state. */
case S_INITIAL:
@@
-374,11
+387,11
@@
report_state_mismatch (const struct command *command, enum cmd_state state)
"been defined."), command->name);
break;
case S_INPUT_PROGRAM:
"been defined."), command->name);
break;
case S_INPUT_PROGRAM:
- msg (SE, _("%s is allowed only inside
INPUT PROGRAM
."),
- command->name);
+ msg (SE, _("%s is allowed only inside
%s
."),
+ command->name
, "INPUT PROGRAM"
);
break;
case S_FILE_TYPE:
break;
case S_FILE_TYPE:
- msg (SE, _("%s is allowed only inside
FILE TYPE."), command->name
);
+ msg (SE, _("%s is allowed only inside
%s."), command->name, "FILE TYPE"
);
break;
/* Two allowed states. */
break;
/* Two allowed states. */
@@
-401,7
+414,8
@@
report_state_mismatch (const struct command *command, enum cmd_state state)
command->name, "FILE TYPE");
break;
case S_INPUT_PROGRAM | S_FILE_TYPE:
command->name, "FILE TYPE");
break;
case S_INPUT_PROGRAM | S_FILE_TYPE:
- msg (SE, _("%s is allowed only inside INPUT PROGRAM or inside FILE TYPE."), command->name);
+ msg (SE, _("%s is allowed only inside %s or inside %s."), command->name,
+ "INPUT PROGRAM", "FILE TYPE");
break;
/* Three allowed states. */
break;
/* Three allowed states. */
@@
-430,7
+444,7
@@
report_state_mismatch (const struct command *command, enum cmd_state state)
}
else if (state == CMD_STATE_INPUT_PROGRAM)
msg (SE, _("%s is not allowed inside %s."),
}
else if (state == CMD_STATE_INPUT_PROGRAM)
msg (SE, _("%s is not allowed inside %s."),
- command->name, "INPUT PROGRAM"
);
+ command->name, "INPUT PROGRAM");
else if (state == CMD_STATE_FILE_TYPE)
msg (SE, _("%s is not allowed inside %s."), command->name, "FILE TYPE");
else if (state == CMD_STATE_FILE_TYPE)
msg (SE, _("%s is not allowed inside %s."), command->name, "FILE TYPE");
@@
-482,15
+496,12
@@
cmd_finish (struct lexer *lexer UNUSED, struct dataset *ds UNUSED)
int
cmd_n_of_cases (struct lexer *lexer, struct dataset *ds)
{
int
cmd_n_of_cases (struct lexer *lexer, struct dataset *ds)
{
- /* Value for N. */
- int x;
-
- if (!lex_force_int (lexer))
+ if (!lex_force_int_range (lexer, "N OF CASES", 1, LONG_MAX))
return CMD_FAILURE;
return CMD_FAILURE;
-
x
= lex_integer (lexer);
+
long n
= lex_integer (lexer);
lex_get (lexer);
if (!lex_match_id (lexer, "ESTIMATED"))
lex_get (lexer);
if (!lex_match_id (lexer, "ESTIMATED"))
- dict_set_case_limit (dataset_dict (ds),
x
);
+ dict_set_case_limit (dataset_dict (ds),
n
);
return CMD_SUCCESS;
}
return CMD_SUCCESS;
}
@@
-514,7
+525,7
@@
cmd_erase (struct lexer *lexer, struct dataset *ds UNUSED)
if (settings_get_safer_mode ())
{
if (settings_get_safer_mode ())
{
- msg (SE, _("This command not allowed when the
SAFER option is set.")
);
+ msg (SE, _("This command not allowed when the
%s option is set."), "SAFER"
);
return CMD_FAILURE;
}
return CMD_FAILURE;
}