Completely rewrite src/data/format.[ch], to achieve better
[pspp-builds.git] / src / language / data-io / ChangeLog
1 Tue Oct 31 20:04:06 2006  Ben Pfaff  <blp@gnu.org>
2
3         * placement-parser.c: (PRS_TYPE_T) Now that struct fmt_spec uses
4         an enum fmt_type for its type member, we can't depend on the
5         ability to put negative values into that member as out-of-band
6         values, because enum fmt_type might be an unsigned type.  So use
7         values around SCHAR_MAX instead, because we know that SCHAR_MAX
8         will fit into any type, signed or unsigned, and there aren't
9         nearly that many format types.
10         (parse_var_placements) Add for_input parameter to specify whether
11         we're parsing input or output formats.  Update all callers.
12         (fixed_parse_columns) Ditto.
13         (fixed_parse_fortran) Ditto.
14         
15
16 Tue Oct 31 18:21:48 2006  Ben Pfaff  <blp@gnu.org>
17
18         * print-space.c (print_space_trns_proc): Let dfm_put_record add
19         the new-line character, to match dfm_put_record change below.
20
21 Sat Oct 28 11:57:19 2006  Ben Pfaff  <blp@gnu.org>
22
23         * data-writer.c (struct dfm_writer): Removed `bounce' member, and
24         all references to it.
25         (dfm_put_record) Change semantics so that it adds formatting
26         itself, such as new-line characters, instead of putting that
27         responsibility on the caller.  Also, pad binary records with
28         spaces instead of zeros, for compatibility.
29
30         * print.c (struct prt_out_spec) New member `sysmis_as_spaces'.
31         (struct print_trns) Remove `omit_new_lines' and all references,
32         since dfm_put_record() is taking care of that.  Add
33         `include_prefix'.
34         (internal_cmd_print) Allow an empty set of data to print.  Set
35         include_prefix.
36         (parse_specs) Allow an empty set of data to print.
37         (parse_variable_argument) Only add space with PRINT command.  Set
38         sysmis_as_spaces.
39         (print_trns_proc) Indent records if include_prefix is set, for
40         compatibility.  Output SYSMIS as spaces if sysmis_as_spaces is
41         set.  Put "1" in first column if PRINT EJECT is used with an
42         external output file.
43         (flush_records) Ditto.
44
45 Sat Oct 28 16:19:57 WST 2006 John Darrington <john@darrington.wattle.id.au>
46
47         * data-reader.c: Eliminated references to extern variable getl_buf
48
49 Sat Aug  5 08:25:07 2006  Ben Pfaff  <blp@gnu.org>
50
51         Fix bug #17329 in REREAD parsing, reported by John Darrington.
52
53         * inpt-pgm.c (cmd_reread): Fix file handle parsing.
54
55 Mon Jul 31 10:32:31 2006  Ben Pfaff  <blp@gnu.org>
56
57         * print.c (parse_specs): Allow a comma between specifications.
58
59 Sun Jul 16 19:57:10 2006  Ben Pfaff  <blp@gnu.org>
60
61         * automake.mk: (src_language_data_io_libdata_io_a_SOURCE) Add
62         print-space.c, placement-parser.c, placement-parser.h.
63
64         * data-list.c: Basically rewrote the whole thing.  Broke out a lot
65         of code into placement-parser.c.  Code is much cleaner now.
66
67         * placement-parser.c: New file.
68
69         * placement-parser.h: New file.
70
71         * print.c: Basically rewrote the whole thing.  Broke out PRINT
72         SPACE into print-space.c.  Code is much cleaner now.
73
74         * print-space.c: New file.
75
76 Sat Jul  1 17:39:40 2006  Ben Pfaff  <blp@gnu.org>
77
78         Fix bug #11612, "q2c documentation does not agree with code".
79         
80         * list.q: Audit use of q2c "+" prefixes that indicate that a
81         command may appear multiple times.
82
83 Sat Jul  1 20:44:22 2006  Ben Pfaff  <blp@gnu.org>
84
85         Fix bug #15786: System File Creation crashes if directoy is
86         nonexistent.
87         
88         * get.c (parse_write_command): Check that the any_writer open
89         succeeds.
90
91 Tue Jun 27 22:44:28 2006  Ben Pfaff  <blp@gnu.org>
92
93         Fix regression in command name completion reported by John
94         Darrington.  Now completion is again state-dependent and occurs
95         only on the first line of a command.
96         
97         * inpt-pgm.c: (cmd_input_program) Reading of first token in
98         command moved into cmd_parse.
99
100 Fri Jun  9 13:56:00 2006  Ben Pfaff  <blp@gnu.org>
101
102         Reform string library.
103         
104         * matrix-data.c (context): Use dynamic string.
105         (another_token) Deal with changed dfm_get_record() interface.
106         (mget_token) Ditto.
107         (force_eol) Ditto.
108
109         * data-list.c (struct data_list_pgm) Delete delims, delim_cnt
110         members, replacing them by struct string delims.  Update all
111         references to use struct string functions.
112         (cut_field) Change interface to avoid needing "end_blank", by
113         getting the data-reader to remember that state for us.  Change
114         internals to use substring.  Update both callers.
115
116         * data-reader.c (read_file_record): Use ds_read_stream().
117         (dfm_get_record) Change interface to return substring.  Updated
118         all callers.
119         (dfm_expand_tabs) Use ds_find_char().  Now maintain position
120         relative to end-of-line.  Use ds_swap().
121         (dfm_reread_record) Don't limit position by line length.
122         (dfm_column_start) Make parameter const.
123         (dfm_columns_past_end) New function.
124         (dfm_get_column) New function.
125
126 Thu May 25 18:26:26 WST 2006 John Darrington <john@darrington.wattle.id.au>
127
128         * print.c (print_trns_free): Made the code agree with the comment, by
129         not freeing PRT.  Has the side effect that the command no longer
130         crashes on invalid syntax.
131
132 Tue May  9 20:55:46 2006  Ben Pfaff  <blp@gnu.org>
133
134         * get.c (cmd_match_files): Fix memory leak replacing default_dict.
135
136 Sat May  6 22:25:09 2006  Ben Pfaff  <blp@gnu.org>
137
138         Fix segfault.
139         
140         * list.q (write_fallback_headers): (write_fallback_headers)
141         Properly record width of leader and pass it to write_varname().
142
143 Sat May  6 19:03:13 2006  Ben Pfaff  <blp@gnu.org>
144
145         * get.c: (mtf_merge_dictionary) Fix value label memory leak.
146
147 Sat May  6 13:51:16 2006  Ben Pfaff  <blp@gnu.org>
148
149         Use a casefile, instead of a case sink, for MATCH FILES output.
150         It's more straightforward, although it has the same effect.
151         
152         * get.c: (struct mtf_proc) Replace `sink' case sink member by
153         `output' casefile member.
154         (cmd_match_files) Work with casefile instead of sink.
155         (mtf_processing) Add case to casefile instead of sink.
156
157 Sat May  6 10:43:07 2006  Ben Pfaff  <blp@gnu.org>
158
159         Continue reforming procedure execution.  In this phase, get rid of
160         the output code for SPLIT FILE groups in procedure.c, which really
161         shouldn't be doing any output.  Move it into the individual
162         procedures instead.  This also adds some flexibility.
163
164         * list.q (write_all_headers): Call output_split_file_values().
165
166 Wed May  3 23:00:17 2006  Ben Pfaff  <blp@gnu.org>
167
168         Continue reforming procedure execution.  In this phase, get rid of
169         many global variables, consolidating procedure execution in
170         procedure.c.  Encapsulate transformations in new "struct
171         trns_chain".  Also, change implementation of N OF CASES, FILTER,
172         and PROCESS IF from special cases to transformations.
173          
174         * data-list.c: (data_list_trns_proc) Return TRNS_END_FILE at end
175         of file.  (Why didn't we do this before?)
176         (cmd_match_files) Direct procedure output to null sink.
177         Use discard_variables() instead of indirect version.
178
179         * inpt-pgm.c: Use transformation chain.
180         (struct input_program_pgm) Add trns_chain member.
181         (cmd_input_program) Initialize trns_chain member and capture
182         transformations with proc_capture_transformations().
183         (input_program_source_read) Use trns_chain_execute().
184         (destroy_input_program) Destroy input chain.
185
186 Tue May  2 10:39:56 WST 2006 John Darrington <john@darrington.wattle.id.au>
187
188         * list.q Changed from using fixed length char buffers to struct 
189         string so that any length variables can be used.
190
191 Mon May  1 18:21:19 2006  Ben Pfaff  <blp@gnu.org>
192
193         Further clean up the CMD_* command result codes.
194         
195         * (enum cmd_result_extensions) New.  Add CMD_END_INPUT_PROGRAM and
196         CMD_END_CASE result codes.
197         (struct input_program_pgm) Added case_nr, write_case, wc_data
198         members for use by END CASE transformation.
199         (emit_END_CASE) New function.
200         (cmd_input_program) Interpret CMD_END_CASE by outputting an END
201         CASE transformation.  If none is output by the input program
202         itself, add one automatically at the end.  Change lack of
203         variables from warning to error.
204         (cmd_end_input_program) Return CMD_END_INPUT_PROGRAM instead of
205         CMD_END_SUBLOOP.
206         (input_program_source_read) No longer any need to special-case END
207         CASE.  Handle TRNS_DROP_CASE properly.  Initialize new members in
208         inp for use by END CASE transformation.
209         (destroy_input_program) New function.
210         (input_program_source_destroy) Just call destroy_input_program().
211         (cmd_end_case) Just return CMD_END_CASE.
212         (end_case_trns_proc) No longer a stub handled by
213         input_program_source_read().  Actually output the case and
214         increment the case number.
215
216 Mon May  1 16:06:30 2006  Ben Pfaff  <blp@gnu.org>
217
218         Remove vestiges of REPEATING DATA support.
219         
220         * data-list.c: (struct rpd_num_or_var) Removed.
221         (struct repeating_data_trns) Removed.
222         (cmd_repeating_data) Removed.
223         (find_variable_input_spec) Removed.
224         (parse_num_or_var) Removed.
225         (parse_repeating_data) Removed.
226         (realize_value) Removed.
227         (struct rpd_parse_info) Removed.
228         (rpd_parse_record) Removed.
229         (repeating_data_trns_proc) Removed.
230         (repeating_data_trns_free) Removed.
231         (repeating_data_set_write_case) Removed.
232         (rpd_msg) Removed.
233
234         * inpt-pgm.c: (input_program_source_read) Don't deal with
235         REPEATING DATA.
236
237         * data-list.h: Removed.
238
239         * automake.mk (src_language_data_io_libdata_io_a_SOURCES): Removed
240         data-list.h.
241
242 Mon May  1 15:58:28 2006  Ben Pfaff  <blp@gnu.org>
243
244         Remove vestiges of FILE TYPE support. 
245         
246         * data-list.c: (cmd_data_list) Don't check for FILE TYPE.
247         (cmd_repeating_data) Ditto.
248         
249         * automake.mk (src_language_data_io_libdata_io_a_SOURCES): Remove
250         file-type.c, file-type.h.
251
252         * file-type.c: Removed.
253         
254         * file-type.h: Removed.
255
256 Wed Apr 26 13:16:28 2006  Ben Pfaff  <blp@gnu.org>
257
258         Improve the way we handle the various parsing "states".  Until now
259         we've hard-coded the state transitions in the command definition
260         file, but that's error-prone and, worse, it's redundant--we can
261         figure out what state we're in anyhow.  We can cleanly handle
262         INPUT PROGRAM and FILE TYPE with a nested command-processing loop.
263         
264         * data-list.c: (cmd_data_list) Use in_file_type() or
265         in_input_program() in place of case_source_is_class() or
266         case_source_is_complex().
267
268         * file-type.c: NB: Not really fixed except minimally to compile,
269         because it doesn't work anyway.
270         (in_file_type) New function.
271         (cmd_record_type) No need to check that we're in FILE TYPE.
272         (cmd_end_file_type) Ditto.
273         (var file_type_source_class) Make static.
274
275         * get.c: (cmd_match_files) Check vfm_source instead of pgm_state.
276
277         * inpt-pgm.c: (in_input_program) New function.
278         (cmd_input_program) Rewrite to include nested command processing
279         loop.
280         (cmd_end_input_program) Just return CMD_END_SUBLOOP.
281         (var input_program_source_class) Make static.
282         (cmd_end_case) No need to check that we're in INPUT PROGRAM.
283         (cmd_end_file) Ditto.
284         
285         * automake.mk (src_language_data_io_libdata_io_a_SOURCES): Add
286         file-type.h, inpt-pgm.h.
287
288         * file-type.h: New file.
289
290         * inpt-pgm.h: New file.
291
292 Tue Apr 25 13:11:55 2006  Ben Pfaff  <blp@gnu.org>
293
294         * print.c: Don't special-case MS-DOS line terminators.
295         (macro LINE_END_WIDTH) Removed.
296         (alloc_line) Line ends are 1 byte.
297         (print_trns_proc) Just output \n for line end.
298
299 Sun Apr 23 22:05:58 2006  Ben Pfaff  <blp@gnu.org>
300
301         Continue reforming error message support.  In this phase, get rid
302         of message "titles" and put the message text in `struct error'.
303         Now `struct error' encapsulates a message more properly.
304         
305         * data-list.c: (macro RPD_ERR) Removed.
306         (rpd_msg) New function.  Updated all references to tmsg() to call
307         this function instead.
308
309 Sat Apr 15 19:38:13 2006  Ben Pfaff  <blp@gnu.org>
310
311         Remove last users of struct file_ext so we can get rid of it
312         entirely.
313         
314         * data-reader.c: (struct dfm_reader) Change file member from
315         struct file_ext to FILE *.  Updated all references.
316         (dfm_close_reader) Close file with fn_close() instead of
317         fn_close_ext().  Also, make a copy of the file name from the file
318         handle before closing it, because we can't extract it after we
319         close the file.
320         (dfm_open_reader) Open file with fn_open() instead of
321         fn_open_ext().
322
323         * data-writer.c: (struct dfm_writer) Change file member 
324         struct file_ext to FILE *.  Updated all references.
325         (dfm_close_writer) Close file with fn_close() instead of
326         fn_close_ext().  Also, make a copy of the file name from the file
327         handle before closing it, because we can't extract it after we
328         close the file.
329         (dfm_open_writer) Open file with fn_open() instead of
330         fn_open_ext().
331
332 Sat Apr 15 18:00:32 2006  Ben Pfaff  <blp@gnu.org>
333
334         * data-list.c: Add prototype to suppress warning for
335         cmd_repeating_data().
336         
337 Thu Mar  2 08:40:33 WST 2006 John Darrington <john@darrington.wattle.id.au>
338         
339         * Moved files from src directory