Adopt use of gnulib for portability.
[pspp-builds.git] / src / ChangeLog
index 498afac3fddf2623722899c94c10fe39027d3f99..f034dd45fddac7e36ac26449650128c752335336 100644 (file)
-       * var-display.c: New file.
+Sun Jul 31 14:09:57 2005  Ben Pfaff  <blp@gnu.org>
+
+       Adopt use of gnulib for portability.
+
+       * Make.build: Add $(top_srcdir)/gl and $(top_builddir)/gl to
+       include path.
+
+       * Makefile.am: Remove bool.h, stat.h and change getline.[ch] to
+       getl.[ch] in pspp_SOURCES.  Remove libmisc, add libgl in
+       pspp_LDADD.
+
+       * In many source files, added an explicit inclusion of gettext.h
+       and definition of _ macro.  These are no longer in pref.h because
+       it interfered with definitions in a few gnulib source files.
+
+       * In many source files, changed #include "bool.h" to #include
+       <stdbool.h>, which is provided by gnulib.
+
+       * alloc.c: Removed functions defined in gnulib:
+       (xmalloc) Removed.
+       (xcalloc) Removed.
+       (xrealloc) Removed.
+       (xstrdup) Removed.
+       (out_of_memory) Redefined as wrapper for xalloc_die().
+
+       * alloc.h: Replace prototypes by #include "xalloc.h".
+
+       * casefile.c: Use full_read() and full_write() from gnulib instead
+       of our home-grown versions.
+       (full_read) Removed.
+       (full_write) Removed.
+
+       * getline.c: Renamed getl.c.
+
+       * getline.h: Renamed getl.h, updated all references.
+
+       * filename.c: (fn_readlink) Change to wrapper around xreadlink()
+       from gnulib.
+
+       * glob.c: Just #include <time.h> instead of the crazy rigmarole
+       here before.
+       (init_glob) Call set_program_name() to initial gnulib progname
+       module.
+
+       * html.c: (postopen) Use getlogin_r(), gethostname() from gnulib.
+
+       * permissions.c: Use "stat-macros.h" from gnulib.
+
+       * postscript.c: Just #include <time.h> instead of the crazy
+       rigmarole here before.
+
+       * q2c.c: (main) Make generated code #include "gettext.h".
+
+       * str.h: Get rid of most explicit declarations of standard
+       functions, in favor of including gnulib header files.
+
+       * expressions/evaluate.c: Ditto.
+
+       * expressions/operations.h.pl: Make generated code #include
+       <stdbool.h>, not "bool.h".
+
+Sat Jul 30 23:13:17 2005  Ben Pfaff  <blp@gnu.org>
+
+       * expressions/parse.c: (validate_function_args) Fix two msg() bugs
+       found by -Wformat.
+
+Sat Jul 30 23:10:01 2005  Ben Pfaff  <blp@gnu.org>
+
+       * expressions/evaluate.c: (expr_debug_print_postfix) Don't pass
+       null pointer to printf for %.*s.
+
+Sat Jul 30 23:05:33 2005  Ben Pfaff  <blp@gnu.org>
+
+       * vars-atr.c: (var_is_valid_name) Fix three msg() bugs found by
+       -Wformat.
+
+Sat Jul 30 22:58:33 2005  Ben Pfaff  <blp@gnu.org>
+
+       * rank.q: (parse_rank_function) Fix msg() bug found by -Wformat.
+
+Sat Jul 30 22:56:12 2005  Ben Pfaff  <blp@gnu.org>
+
+       * postscript.c: (postopen) Cast `char' to `unsigned char' before
+       passing to isspace().
+
+Sat Jul 30 22:52:09 2005  Ben Pfaff  <blp@gnu.org>
+
+       * pfm-read.c: (read_variables) Fix msg() bug found by -Wformat.
+
+Sat Jul 30 22:50:57 2005  Ben Pfaff  <blp@gnu.org>
+
+       * histogram.c: Include <config.h>.
+
+Sat Jul 30 22:48:50 2005  Ben Pfaff  <blp@gnu.org>
+
+       * get.c: (cmd_match_files) Fix msg() bug found by -Wformat.
+
+Sat Jul 30 22:46:10 2005  Ben Pfaff  <blp@gnu.org>
+
+       * format.c: (check_common_specifier) Fix msg() bug found by
+       -Wformat.
+       (check_output_specifier) Ditto.
+
+Sat Jul 30 22:43:57 2005  Ben Pfaff  <blp@gnu.org>
+
+       * file-handle.q: (cmd_file_handle) Fix msg() bug found by
+       -Wformat.
+
+Sat Jul 30 22:41:44 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-in.c: (parse_Z) [WORDS_BIGENDIAN] Don't declare buf[], to
+       avoid "unused variable" warning.
+
+Sat Jul 30 22:38:46 2005  Ben Pfaff  <blp@gnu.org>
+
+       * command.c: (find_word) Cast `char' to `unsigned char' before
+       passing to isspace().
+
+Sat Jul 30 22:36:29 2005  Ben Pfaff  <blp@gnu.org>
+
+       * case.c: (case_compare) Implement as delegating to
+       case_compare_2dict().
+
+Sat Jul 30 22:34:18 2005  Ben Pfaff  <blp@gnu.org>
+
+       * algorithm.c: Inclusion of <alloca.h> is unneeded.
+
+Sat Jul 30 22:01:32 2005  Ben Pfaff  <blp@gnu.org>
+
+       * Make.build: Don't append -ansi to AM_CFLAGS for GCC.  Using
+       -ansi changes the behavior of header files significantly.  It
+       causes __STRICT_ANSI__ to be defined, and some headers interpret
+       that as cause to e.g. not use `long long' or __attribute__.  The
+       former example is bad when off_t is supposed to be `long long',
+       and the latter prevents -Wformat from working.
+
+Sun Jul 24 20:26:59 2005  Ben Pfaff  <blp@gnu.org>
+
+       Get rid of dependency on libgmp by writing our own routine for
+       floating-point base conversion.
+
+       * pfm-write.c: (write_float) Rewrote.
+       (write_int) Rewrote.
+       (pow30_nonnegative) New function.
+       (pow30) New function.
+       (trig_to_char) New function.
+       (format_trig_digits) New function.
+       (recurse_format_trig_int) New function.
+       (format_trig_int) New function.
+       (should_round_up) New function.
+       (try_round_up) New function.
+       (format_trig_double) New function.
+
+Sun Jul 24 18:49:20 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-in.c: (parse_numeric) Allow "1+23" even for F format, for
+       compatibility.
+
+Sun Jul 24 18:47:37 2005  Ben Pfaff  <blp@gnu.org>
+
+       * pfm-read.c: (read_version_data) Read and ignore author field.
+
+Wed Jul  6 20:44:27 2005  Ben Pfaff  <blp@gnu.org>
+
+       * get.c: (mtf_processing) Don't assume that
+       mtf_compare_BY_values() always returns -1, 0, or 1.  Actually, it
+       returns a negative, zero, or positive result.  Fixes MATCH FILES
+       bug on Mac OS X reported by "Marshall DeBerry" <mdb@radix.net>.
+
+Mon Jul  4 18:01:15 2005  Ben Pfaff  <blp@gnu.org>
+
+       * flip.c: [HAVE_SYS_TYPES_H] Really include <sys/types.h>.  The
+       preprocessor test for sys/types.h was accidentally inverted.  This
+       was bug 12789.
+
+Sun Jul  3 22:47:39 2005  Ben Pfaff  <blp@gnu.org>
+
+       * get.c: (cmd_match_files) Fix memory leak on `by' and on
+       `vfm_source'.
+
+       * getline.c: [HAVE_LIBREADLINE] (read_console) Fix memory leak on
+       `line'.
+
+       * vfm.c: (close_active_file) Remove unnecessary test.
+
+Sun Jul  3 21:45:32 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix NDEBUG compile errors.
+
+       * hash.h: Needed explicit #include <assert.h>.
+
+       * linked-list.c: (ll_next) First arg is UNUSED when NDEBUG is
+       defined.
+
+Sun Jun 12 23:44:38 2005  Ben Pfaff  <blp@gnu.org>
+
+       Implement embedding for PostScript driver.  Fixes bug 12970.
+
+       * ascii.c: Fix compiler warnings.
+
+       * html.c: Ditto.
+
+       * chart.h: Add `file' member.
+
+       * output.h: (struct outp_class) initialise_chart, finalise_chart
+       should take outp_driver *, not outp_class *.  Update all
+       references.
+
+       * plot-chart.c: (chart_create) Fix segfault when there are no
+       output drivers at all.
+       (chart_submit) Call d->class->finalise_chart.
+
+       * postscript.c: (ps_open_page) Set cp_y to 0.
+       (ps_submit) New function.
+       (ps_chart_initialise) Implement.
+       (ps_chart_finalise) Implement.
+       (static var postscript_class) Add ps_submit.
+       (static var epsf_class) Add ps_submit.
+       
+
+Sun Jun 12 14:54:40 2005  Ben Pfaff  <blp@gnu.org>
+
+       Did some more work on bug 12859 and then realized that a *good*
+       solution would require some fundamental restructuring.  For now,
+       I'm marking REPEATING DATA unimplemented, and then we can revisit
+       it post-0.4.0.
+       
+       * command.def: Make REPEATING DATA unimplemented.
+
+       * data-list.c: (cmd_repeating_data) Assume inline file is 80
+       characters wide.
+       (realize_value) Revert previous changes; no longer needed.
+       Updated all callers.
+
+       * error.c: (err_hcf) Set nfile_loc, mfile_loc to 0 after freeing
+       file_loc, to avoid bad references later.
+
+       * str.c: Fix typo.
+
+Tue Jun  7 00:14:09 2005  Ben Pfaff  <blp@gnu.org>
+
+       Make some code tolerant of reentry.  Should not be needed if other
+       code is correct but it is good to be generally tolerant.
+       
+       * error.c: (err_hcf) Set file_loc to null after free().
+
+       * output.c: (outp_done) Similar changes.
+
+       * str.c: (ds_destroy) Ditto.
+       
+Tue Jun  7 00:10:20 2005  Ben Pfaff  <blp@gnu.org>
+
+       Continue work on bug 12859, plus some code cleanup.
+       
+       * data-list.c: (cmd_repeating_data) Replace `seen' bitmap by
+       boolean variables.  Don't try to compute starts_end, cont_end for
+       inline file.  Calculate length only after parsing variable
+       specifications.  Add proper transformation to list.
+       (realize_value) If the rpd_num_or_var has no value, return new
+       DEFAULT_MEMBER argument (for use with inline file).
+       (repeating_data_trns_proc) Pass default values.
+
+       * dfm-read.c: (dfm_close_reader) Only skip data if *not* still
+       open, and only if we actually started reading data.
+
+Sun Jun  5 18:39:36 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix bug 11894.
+       
+       * output.c: (outp_read_devices) Fix message.
+
+Fri May 27 12:34:43 WST 2005 John Darrington <john@darrington.wattle.id.au>
+       
+       * sort-prs.[ch] (newfiles), aggregate.c, sort.[ch]: Separated the guts 
+       of the sort algorithm from the parser for the SORT command.
+       
+       * rank.q: Added the parser for the RANK command.
+       
+Thu May 26 12:29:21 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix bug 13192.
+
+       * sort.c: (sort_parse_criteria) Only set *saw_direction if
+       saw_direction is non-null.  Thanks to John Darrington for
+       reporting this bug.
+
+Tue May 24 21:52:55 2005  Ben Pfaff  <blp@gnu.org>
+
+       * get.c: (mtf_processing) Handle case of a lookup table as the
+       active file.  Thanks to John Darrington for reporting this bug.
+
+Wed May 25 10:27:02 WST 2005 John Darrington <john@darrington.wattle.id.au>
+       
+       * alloc.c alloc.h: (xcalloc) changed signature to imitate the
+       POSIX  calloc function.
+       
+       * crosstabs.q get.c vars-prs.c: Updated calls to xcalloc to
+       reflect new signature.
+
+       * sfm-read.c: Now much more robust in the face of badly formed
+       system files.
+
+Mon May 23 11:57:31 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       *sfm-read.c: Fixed some bugs regarding long string continuation
+       records, which the previous fix uncovered.
+
+Sat May 21 12:48:34 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * sfm-read.c, sfmP.h:  Allow reading of system files when the 
+       case_size value in the header is -1.  Also changed some Errors to 
+       Warnings when reading system files.
+
+Tue May 17 21:00:57 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-list.c: (data_list_trns_free) Don't free the argument
+       because cancel_transformations() will do that itself.
+       (data_list_source_destroy) Destroy the argument to
+       data_list_trns_free(), because it no longer does so itself.
+
+Tue May 17 18:29:35 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-out.c: (format_and_round) Don't output leading `-' if value
+       rounds to zero.
+
+Tue May 17 00:06:43 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix bug 11119.
+
+       * som.c: (output_encodings) If some cell in the table won't fit
+       with the horizontal or vertical headers, cancel those headers.
+
+       * som.h: (struct som_table_class) Add fits_width, fits_length,
+       set_headers members.
+
+       * tab.c: (tabi_fits_width) New function.
+       (tabi_fits_length) New function.
+       (tabi_set_headers) New function.
+       (global var tab_table_class) Add the new functions as appropriate
+       members.
+       
+Mon May 16 22:34:06 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix rest of bug 13054.
+
+       * format.def: Fix EDATE, SDATE, ADATE, JDATE, QYR, MOYR, WKYR,
+       DATETIME, TIME system/portable file values.
+
+Mon May 16 22:31:15 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-list.c: (parse_free) Use make_input_format().
+       
+       * data-out.c: (num_to_string) Use make_output_format().
+
+       * dictionary.c: (dict_create_var) Ditto.
+
+       * format.c: (global var f8_2) New var.
+       (make_input_format) New function.
+       (make_output_format) New function.
+
+       * get.c: (cmd_match_files) Use make_output_format().
+
+       * list.q: (cmd_list) Ditto.
+
+       * matrix-data.c: Ditto.
+
+       * sfm-read.c: (parse_format_spec) Check output specifier
+       thoroughly.
+
+       * tab.c: (tab_float) Use make_output_format().
+
+Sun May 15 23:38:10 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix more of bug 13054.
+       
+       * format.def: FMT_A should allow 255-character output.  FMT_AHEX
+       should allow 510-character input and output.
+
+       * data-out.c: (num_to_string) Get rid of NEW_STYLE option.
+       (convert_E) Handle non-finite values.
+       (try_F) Rewrite.
+       (format_and_round) New function.
+       (convert_infinite) New function used by try_F() and convert_E().
+
+Sun May 15 23:36:55 2005  Ben Pfaff  <blp@gnu.org>
+
+       Regularize string and buffer function names so that they make some
+       kind of sense.
+
+       * str.c: (mm_reverse) Rename buf_reverse().  Update all
+       references.
+       (mm_find_reverse) Rename buf_find_reverse().  Update all
+       references.
+       (mm_case_compare) Rename buf_compare_case().  Update all
+       references.
+       (st_compare_pad) Rename buf_compare_rpad().  Update all
+       references.
+       (str_compare_rpad) New function.
+       (st_bare_pad_copy) Rename buf_copy_str_rpad().  Update all
+       references.
+       (buf_copy_str_lpad) New function.
+       (st_bare_pad_len_copy) Rename buf_copy_rpad().  Update all
+       references.
+       (st_pad_copy) Rename str_copy_rpad().  Update all references.
+       (st_trim_copy) Rename str_copy_trunc().  Update all references.
+       (st_uppercase) Renamed str_uppercase().  Update all references.
+       
+Sat May 14 08:22:26 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * dfm-read.c: Fixed polarity of test in dfm-close-reader.  Closes 
+       Bug #13082
+
+Tue May 10 20:08:18 2005  Ben Pfaff  <blp@gnu.org>
+
+       * data-in.c: (data_in) Add assertion to check input specifier.
+
+       * data-out.c: (data_out) Add assertion to check output specifier.
+
+Tue May 10 19:56:35 2005  Ben Pfaff  <blp@gnu.org>
+
+       Start to fix bug 13054.
+
+       * format.c: (check_input_specifier) Improve error message.
+       (check_input_specifier) Check F, COMMA, and DOLLAR formats for
+       valid decimal places.
+       (check_output_specifier) Ditto (but different criteria).
+       (convert_fmt_ItoO) Assert valid input and output specifiers.
+       Also, if input specifier has *any* decimal places, make the output
+       specifier 1 place wider.
+
+Mon May  9 07:14:29 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * sysfile-info.c: Fixed bug [# 13024 ]
+
+Sun May  8 13:52:12 2005  Ben Pfaff  <blp@gnu.org>
+
+       "Fix" bug 13021 by disabling FILE TYPE.  Eventually, we should
+       actually implement it.
+
+       * command.c: (FILE_TYPE_okay) Always return 1.
+
+       * command.def: Change FILE TYPE, END FILE TYPE into UNIMPL.
+
+       * file-type.c: Add prototypes to get rid of warnings.
+
+Sun May  8 08:08:07 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * barchart.c box-whisker.c cartesian.c piechart.c plot-hist.c: Fixed 
+        more ISO/IEC 9899:1990 conformance issues.
+
+Wed May  4 23:54:02 2005  Ben Pfaff  <blp@gnu.org>
+
+       Fix bug 12948.  See also new test in
+       tests/bugs/match-file-scratch.sh.
+       
+       * get.c: (mtf_merge_dictionary) Don't compact dictionary because
+       that deletes scratch variables that someone else might be using,
+       and because we can't reassign our sources' value indexes.
+       Instead, simply don't copy scratch variables into the master
+       dictionary.
+
+       * dictionary.c: (dict_compact_values) Delete variables from the
+       dictionary passed in, not from default_dict (!).
+
+Tue May  3 22:25:17 2005  Ben Pfaff  <blp@gnu.org>
+
+       Improve hash.c comments, error-checking.
+       
+       * hash.c: (struct hsh_table) [NDEBUG] Add hash_ordered member.
+       (hsh_create) size == 0 should *not* return NULL!  Set
+       hash_ordered.
+       (hsh_clear) Set hash_ordered.
+       (locate_matching_entry) Check hash_ordered.
+       (hsh_rehash) Rename rehash().  Add assertion.  Set hash_ordered.
+       (hsh_data) Set hash_ordered.  Add const-ness to return value and
+       update all callers.
+       (hsh_sort) Ditto.       
+
+Wed May  4 08:50:11 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * casefile.c: Removed unnecessary #include <valgrind/valgrind.h>
+
+Tue May  3 19:14:48 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * copyleft.c: Updated copyright date.
+
+       * Makefile.am: Removed erroneous explicit "-lplot"
+
+       * examine.q oneway.q: Made these files conform to ISO/IEC 9899:1990
+
+Tue May  3 16:20:31 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * command.c command.def: Added description string for unimplemented commands.
+
+       * oneway.q: Sorted the hash tables before shipping out the results. Closes 
+       bug [#12931].
+
+Mon May  2 23:45:01 2005  Ben Pfaff  <blp@gnu.org>
+
+       Code improvements.
+       
+       * data-list.c:  (parse_fixed) Use lex_end_of_command().
+       (parse_free) Ditto.
+       (cmd_repeating_data) Set cont_end.num in right situations.
+       (parse_repeating_data) Remove redundant test.
+
+Mon May  2 23:37:19 2005  Ben Pfaff  <blp@gnu.org>
+
+       Partial fix for bug 12859.
+       
+       * data-list.c: (cmd_data_list) Add transformation properly in
+       vfm_source == NULL case.
+
+Mon May  2 23:27:28 2005  Ben Pfaff  <blp@gnu.org>
+
+       * lexer.c: (lex_error) Improve error messages.
+
+Mon May  2 22:28:17 2005  Ben Pfaff  <blp@gnu.org>
+
+       * get.c: (cmd_match_files) Check token type before trying to match
+       tokid.  Fixes bug 12923.
+
+Mon May  2 22:16:51 2005  Ben Pfaff  <blp@gnu.org>
+
+       * flip.c: [HAVE_SYS_TYPES_H] Include <sys/types.h>.  Fixes bug
+       12789.
+
+Mon May  2 22:02:52 2005  Ben Pfaff  <blp@gnu.org>
+
+       * expressions/generate.pl: (get_token) Make use of /g
+       backward-compatible with Perl 5.6.1.
+
+Sun May  1 23:00:19 2005  Ben Pfaff  <blp@gnu.org>
+
+       * var-display.c: (cmd_variable_alignment) Fix memory leak.
+       (cmd_variable_level) Ditto.
+
+Sun May  1 22:49:04 2005  Ben Pfaff  <blp@gnu.org>
+
+       Hash table had buggy deletion function.  The fix required changing
+       other functions to do probing in the required order.
+
+       * hash.c: (locate_matching_entry) Rewrite and change interface.
+       (hsh_rehash) Rewrite to use locate_matching_entry().
+       (hsh_probe) Ditto.
+       (hsh_find) Ditto.
+       (hsh_delete) Ditto.  Also, fix stupid bugs.
+
+Sun May  1 22:24:58 2005  Ben Pfaff  <blp@gnu.org>
+
+       * dictionary.c: (dict_clone) Properly copy vectors.
+
+Sun May  1 22:07:58 2005  Ben Pfaff  <blp@gnu.org>
+
+       New implementation of long variable names.  Each variable has a
+       "normal" name, which may be up to 64 bytes long and which is used
+       for all normal operations.  Variables may have a "short" name,
+       which is limited to 8 bytes and used only for system and portable
+       file input and output.
+       
+       Make tokid case-preserving.  Update most uses of tokid to treat it
+       case-insensitively.
+
+       Update many commands to deal with long variable names.
+
+       * autorecode.c: (cmd_autorecode) Use strcasecmp() instead of strcmp().
+
+       * command.c: (cmd_parse) Ditto.
+       (match_strings) Use toupper() before comparing characters.
+       (conflicting_3char_prefixes) Use mm_case_compare() instead of
+       memcmp().
+       (cmd_match_words) Ditto.
+
+       * compute.c: (lvalue_parse) Use st_trim_copy() instead of
+       strncpy().
+
+       * count.c: (struct cnt_var_info) Change n[] to fit long var name.
+       Use st_trim_copy() instead of strcpy().
+
+       * data-in.c: (parse_enum) Use mm_case_compare() instead of
+       memcmp().
+
+       * data-list.c: (struct dls_var_spec) Change name[] to fit long var
+       name.
+       (parse_free) Use st_trim_copy() instead of strcpy().
+
+       * descript.c: (struct dsc_var) Change z_name[] to fit long var
+       name.
+       (try_name) Use strcasecmp() instead of strcmp().
+       (generate_z_varname) Use st_trim_copy() instead of strcpy().
+       (descriptives_compare_dsc_vars) Use strcasecmp() instead of
+       strcmp().
+
+       * dictionary.c: (struct dictionary) Removed `long_name_tab'
+       member.
+       (compare_long_names) Removed.
+       (hash_long_name) Removed.
+       (dict_create) Don't initialize `long_name_tab' member.
+       (dict_clone) Copy short names into new dictionary. 
+       (dict_clear) Don't clear `long_name_tab' member.
+       (dict_get_varname_block) Removed.
+       (dict_add_longvar_entry) Removed.
+       (free_nte) Removed.
+       (dict_destroy) Don't destroy `long_name_tab' member.
+       (dict_create_var_from_short) Removed.
+       (dict_create_var_x) Removed.
+       (dict_create_var) Get rid of longname handling.
+       Clear short name.
+       (dict_clone_var) Get rid of longname parameter and longname
+       handling.
+       (dict_lookup_var) Get rid of longname handling.
+       (dict_reorder_var) New function.
+       (dict_rename_var) Clear short name.
+       (dict_rename_vars) Get rid of longname handling.  Clear short
+       names.
+       (dict_create_vector) Support long vector names.
+       (dict_lookup_vector) Use strcasecmp() instead of strcmp().
+       (quasi_base27) Removed.
+       (make_short_name) Removed.
+       (compare_strings) New function.
+       (hash_string) New function.
+       (dict_assign_short_names) New function.
+
+       * file-handle.q: (get_handle_with_name) Use strcasecmp() instead
+       of strcmp().
+       (get_handle_for_filename) Support long handle names.
+
+       * file-type.c: (struct col_spec) Make `name' fit long var names.
+       (cmd_file_type) Use strcasecmp() instead of strcmp().
+
+       * flip.c: (make_new_var) Rewrite.
+       (flip_sink_write) Use st_trim_copy() instead of strncpy().
+
+       * format.c: (parse_format_specifier_name) Use mm_case_compare()
+       instead of memcmp().
+
+       * get.c: (cmd_save_internal) Rephrase.
+       (rename_variables) Drop test for identical variable name.
+       (struct mtf_proc) Change `first', `last' to fit long var name.
+
+       * hash.c: (hsh_hash_case_string) New function for case-insensitive
+       string hashing.
+
+       * lexer.c: (restore_token) Use st_trim_copy() instead of
+       strncpy().
+       (lex_get) Don't uppercase string when copying into tokid.
+       (lex_put_back_id) Use st_trim_copy() instead of
+       strncpy().
+
+       * list.q: (determine_layout) Consider length of variable names in
+       choosing vertical layout.
+
+       * matrix-data.c: (cmd_matrix_data) Use strcasecmp() instead of
+       strcmp().
+       (string_to_content_type) Ditto.
+
+       * modify-vars.c: (compare_variables_given_ordering) Ditto.
+       (struct var_renaming) Change `new_name' to fit long var name.
+       (compare_var_renaming_by_new_name) Use strcasecmp() instead of
+       strcmp().
+
+       * pfm-read.c: (read_variables) Disallow system variables in system
+       files.
+       (write_variables) Call dict_assign_short_names() and use
+       short_name[] members.
+
+       * repeat.c: (internal_cmd_do_repeat) Use strcasecmp() instead of
+       strcmp().
+
+       * sfm-read.c: (sfm_open_reader) Rewrite code for long variable
+       map.  Reorder variables into same order as long variable map.
+       (read_variables) Set short name.
+
+       * sfm-write.c: (sfm_open_writer) Call dict_assign_short_names().
+       (write_variable) Use st_bare_pad_copy().
+       (write_longvar_table) Rewrite.
+
+       * str.c: (mm_case_compare) New function.
+
+       * sysfile-info.c: (compare_vectors_by_name) Use strcasecmp()
+       instead of strcmp().
+
+       * t-test.q: (tts_custom_groups) Remove redundant test.
+       (tts_custom_pairs) Ditto.
+
+       * var.h: (struct variable) Change `name' to fit long var names.
+       Remove `longname'.  Add `short_name' member.  Reorder some
+       variables.
+       (struct name_table_entry) Removed.
+       (struct vector) Change `name' to fit long vector names.
+
+       * vars-atr.c: (var_is_valid_name) Allow long var names.
+       (compare_var_names) Use strcasecmp() instead of strcmp().
+       (compare_var_ptr_names) Ditto.
+       (hash_var_name) Use hsh_hash_case_string().
+       (hash_var_ptr_name) Ditto.
+       (var_set_short_name) New function.
+       (var_clear_short_name) New function.
+       (var_set_short_name_suffix) New function.
+
+       * vars-prs.c: (parse_DATA_LIST_vars) Support long names.
+       Use strcasecmp() instead of strcmp().
+       (struct array_var_set) Removed `longname_tab'.
+       (array_var_set_lookup_var_idx) Drop longname_tab support.
+       (array_var_set_destroy) Don't destroy `longname_tab'.
+       (var_set_create_from_array) Don't create `longname_tab'.
+
+       * vector.c: (cmd_vector) Use strcasecmp() instead of strcmp().
+       Support long names.
+
+       * expressions/parse.c: (word_matches) Use mm_case_compare()
+       instead of memcmp().
+       (compare_strings) New function.
+       (lookup_function) Use compare_strings() instead of strcmp().
+       
+Sun May  1 22:07:43 2005  Ben Pfaff  <blp@gnu.org>
+
+       * algorithm.c: (move_element) New function.
+
+Sun May  1 22:05:35 2005  Ben Pfaff  <blp@gnu.org>
+
+       * aggregate.c: (parse_aggregate_functions) Always initialize
+       destvar.
+
+Sun May  1 22:03:47 2005  Ben Pfaff  <blp@gnu.org>
+
+       * aggregate.c: (cmd_aggregate) Use dict_clone_var_assert().
+
+       * dictionary.c: (dict_clone) Ditto.
+       (dict_clone_var_assert) New function.
+
+       * get.c: (mtf_merge_dictionary) Use dict_clone_var_assert().
+
+Sun May  1 15:05:54 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * error.c: Added a string for the compiler version to the
+       request_bug_report_and_abort function.
+
+       * groff_font.c, font.c: Removed manpage(1) style references from 
+       comments, because RMS frowns upon them.
+
+Thu Apr 28 18:52:06 2005  Ben Pfaff  <blp@gnu.org>
+
+       * expressions/parse.c: Improve previous fix for bug 12858 (LAG).
+
+Fri Apr 29 09:28:00 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * expressions/parse.c: Added handler for OP_LAG_Vn and OP_LAG_Vs.  
+       Fixed bug [#12858] .
+
+Wed Apr 27 12:42:34 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * loop.c recode.c repeat.c: Fixed a couple of instances of SHORT_NAME_LEN 
+       which should be LONG_NAME_LEN
+
+Wed Apr 27 07:43:50 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * command.def echo.c:  Added the ECHO command.
+
+Mon Apr 25 22:55:59 2005  Ben Pfaff  <blp@gnu.org>
+
+       Finish fixing MATCH FILES (bug 11677).
+
+       * get.c: (trim_dictionary) Rewrite in terms of drop_variables(),
+       keep_variables(), rename_variables().
+       (drop_variables) New function.
+       (keep_variables) New function.
+       (struct mtf_file) Rename `in' to `in_name'.  Add `in_var'.
+       (cmd_match_files) Deal with in_var.  Use drop_variables(),
+       keep_variables().  When IN is specified, require BY.  Set master
+       variables after master dictionary is complete.  Add IN variables
+       after master dictionary is complete.
+       (mtf_free_file) Free `in_name'.
+       (mtf_delete_file_in_place) Set in_var value to 0.
+       (mtf_read_nonactive_records)  Rephrase.
+       (mtf_processing) Support IN.  Rephrase.  Fix bugs.
+       (mtf_merge_dictionary) Don't set master variables; we do that
+       later now.
+       (get_master) Don't insist that there's a master variable.
+       
+Mon Apr 25 22:55:22 2005  Ben Pfaff  <blp@gnu.org>
+
+       Kluge to make some variable renaming sort of work.
+       Needs real fix.
+
+       * dictionary.c: (dict_rename_var) Call dict_add_longvar_entry().
+
+Mon Apr 25 22:52:28 2005  Ben Pfaff  <blp@gnu.org>
+
+       Add functions for comparing sets of variables between cases.
+       Use the functions.
+
+       * case.c: (case_compare) New function.
+       (case_compare_2dict) New function.
+       
+       * aggregate.c: (struct agr_proc) Remove `prev_break' member.  Add
+       `break_case'.
+       (cmd_aggregate) Nullify break_case.  Don't call
+       initialize_aggregate_info().
+       (agr_destroy) Destroy break_case.
+       (aggregate_single_case) Rewrite.  Use case_compare().
+       (dump_aggregate_info) Copy from break_case into output.
+       (initialize_aggregate_info) Copy break_case from input.
+
+       * get.c: (mtf_compare_BY_values) Use case_compare_2dict().
+
+       * vfm.c: (equal_splits) Use case_compare().
+
+Sat Apr 23 17:01:04 WST 2005 John Darrington <john@darrington.wattle.id.au>
+
+       * dictionary.c vars-prs.c sfm-write.c: Fixed some memory leaks
+
+Sun Apr 17 23:08:15 2005  Ben Pfaff  <blp@gnu.org>
+
+       Start work on fixing MATCH FILES.
+
+       * get.c: (enum operation) Remove OP_MATCH.
+       (trim_dictionary) Change return value to bool.  Don't support
+       OP_MATCH.
+       (struct mtf_file) Remove `first', `last' members.
+       (struct mtf_proc) Add `first', `last' members.  Change mtf_case
+       from `struct ccase *' to `struct ccase'.  Remove `by' member.
+       (cmd_match_files) Essentially rewrite.
+       (mtf_free) Don't free `by' member.  Destroy `mtf_case' member.
+       (mtf_read_nonactive_records) mtf_ parameter is not unused.
+       (mtf_processing) Ditto.  Also rephrase some code.
+       (mtf_merge_dictionary) Rewrite for easy comprehension.  
+
+Sun Apr 17 23:06:00 2005  Ben Pfaff  <blp@gnu.org>
+
+       * matrix-data.c: (wr_output_data) [DEBUGGING] Fix compilation
+       error.
+
+       * q2c.c: (dump_token) [DEBUGGING] Fix compilation error.
+       
+Thu Apr 14 2005 John Darrington
+
+        * var-display.c: New file.
 
        * format.h var.h sfm-read.c sfm-write.c dictionary.c :  Added 
          display_width, measure and alignment parameters to variables.
 
        * aggregate.c command.def compute.c count.c data-list.c descript.c
-         dictionary.c dictionary.h expr-prs.c file-type.c flip.c get.c lexer.c
-         lexer.h loop.c modify-vars.c pfm-read.c recode.c repeat.c sfm-read.c
-         sfm-write.c sfm-write.h sfmP.h val-labs.c val.h var.h vars-prs.c
-         vector.c :  
-           - Replaced literal constants representing maximum variable name 
-             length with macro definitions. 
-           - Added support for long variable names.
-           - Changed lexer such that it no longer makes tokens upper case, but
-             relies upon case insensitive behaviour of commands.
+         dictionary.c dictionary.h expr-prs.c file-type.c flip.c get.c 
+         lexer.c lexer.h loop.c modify-vars.c pfm-read.c recode.c repeat.c 
+         sfm-read.c sfm-write.c sfm-write.h sfmP.h val-labs.c val.h var.h 
+         vars-prs.c vector.c :  
+             - Replaced literal constants representing maximum variable name 
+               length with macro definitions. 
+             - Added support for long variable names.
+             - Changed lexer such that it no longer makes tokens upper
+               case, but relies upon case insensitive behaviour of commands.
 
 Mon Apr  4 22:27:34 2005  Ben Pfaff  <blp@gnu.org>
 
        * aggregate.c: (parse_aggregate_functions) If dict_create_var()
-       fails, don't dereference the resulting null pointer (PR 12427).
+       fails, don't dereference the resulting null pointer (bug 12427).
        Also, fix double free error.
 
 Sat Mar 19 23:06:02 2005  Ben Pfaff  <blp@gnu.org>
@@ -39,7 +876,7 @@ Mon Mar 14 21:52:34 2005  Ben Pfaff  <blp@gnu.org>
 
 Mon Mar 14 21:07:23 2005  Ben Pfaff  <blp@gnu.org>
 
-       Make sort stable (PR 12313).
+       Make sort stable (bug 12313).
        
        * sort.c: Don't need to include some headers anymore.
        (static var min_buffers) New variable.
@@ -87,7 +924,6 @@ Mon Mar 14 21:05:13 2005  Ben Pfaff  <blp@gnu.org>
 
        * flip.c: (cmd_flip) Ditto.
 
->>>>>>> 1.161
 Sun Mar 13 22:52:05 2005  Ben Pfaff  <blp@gnu.org>
 
        * file-handle.q: (struct file_handle) `open_mode' should not be
@@ -95,7 +931,7 @@ Sun Mar 13 22:52:05 2005  Ben Pfaff  <blp@gnu.org>
        
 Sun Mar 13 22:40:54 2005  Ben Pfaff  <blp@gnu.org>
 
-       First phase of making SORT CASES stable (PR 12313).
+       First phase of making SORT CASES stable (bug 12313).
 
        * sort.c: (struct indexed_case) New structure.
        (do_internal_sort) Rewrite to make internal sorting stable.
@@ -138,7 +974,6 @@ Sun Mar 13 22:35:55 2005  Ben Pfaff  <blp@gnu.org>
        (initialize_aggregate_info) Fix initialization for MIN, MAX for
        strings.
 
->>>>>>> 1.160
 Sat Mar 12 23:26:21 2005  Ben Pfaff  <blp@gnu.org>
 
        Start work on testing and debugging AGGREGATE.
@@ -164,11 +999,11 @@ Sun Mar 13 14:54:27 WST 2005 John Darrington <john@darrington.wattle.id.au>
 Sat Mar 12 13:29:21 2005  Ben Pfaff  <blp@gnu.org>
 
        * split-file.c: (cmd_split_file) Ignore LAYERED and SEPARATE
-       keywords (PR 11628).
+       keywords (bug 11628).
 
 Sat Mar 12 13:17:12 2005  Ben Pfaff  <blp@gnu.org>
 
-       * vfm.c: (procedure_with_splits) Fix PR 11492: end_func() must be
+       * vfm.c: (procedure_with_splits) Fix bug 11492: end_func() must be
        called *before* close_active_file().
 
 Sat Mar 12 12:20:57 2005  Ben Pfaff  <blp@gnu.org>
@@ -252,7 +1087,7 @@ Sun Mar  6 23:25:40 2005  Ben Pfaff  <blp@gnu.org>
 Sun Mar  6 19:52:22 2005  Ben Pfaff  <blp@gnu.org>
 
        DATA LIST with free-field formats should not have implied decimal
-       places (PR 12035).  Also clean up data-in.c a bit.
+       places (bug 12035).  Also clean up data-in.c a bit.
 
        * data-in.h: (enum) Add DI_IMPLIED_DECIMALS.
 
@@ -457,7 +1292,7 @@ Fri Feb 25 21:11:35 WST 2005 John Darrington <john@darrington.wattle.id.au>
 
 Sun Feb 13 16:11:13 2005  Ben Pfaff  <blp@gnu.org>
 
-       Fix PR 11955.
+       Fix bug 11955.
 
        * aggregate.c: (parse_aggregate_functions) Code cleanup.
        Important part: get rid of spurious copying of function->format to
@@ -465,7 +1300,7 @@ Sun Feb 13 16:11:13 2005  Ben Pfaff  <blp@gnu.org>
 
 Fri Feb 11 00:08:36 2005  Ben Pfaff  <blp@gnu.org>
 
-       Fix PR 11916, which was confusing a variable's `index' member
+       Fix bug 11916, which was confusing a variable's `index' member
        with the variable's position in a var_set.  Although these are
        usually the same, they are not for array `var_set's.
        
@@ -658,7 +1493,7 @@ Wed Jan  5 08:30:48 WST 2005 John Darrington <john@darrington.wattle.id.au>
 Mon Jan  3 17:44:37 2005  Ben Pfaff  <blp@gnu.org>
 
        * pfm-read.c: (read_variables) Remove direct manipulation of
-       v->aux, which is no longer needed.  Fixes PR 11483.
+       v->aux, which is no longer needed.  Fixes bug 11483.
 
 Sat Jan  1 19:01:16 WST 2005 John Darrington <john@darrington.wattle.id.au>