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