remove stray spaces from po/automake.mk

Remove #include <assert.h> from files which don't need it

Delete trailing whitespace at line endings.

Fix typos in comments

* src/ui/gui/memorandum.txt: Fix typo.
* tests/language/command.at: Remove doubled word.

Documentation: Fix various typos

* doc/statistics.texi: Fix typos.
* doc/utilities.texi: filesystem -> file system

Remove stray semicolon

* src/ui/gui/psppire-dialog-action-select.c: Remove extra ;

acinclude.m4: Quote arguments to AC_DEFINE

Update perl-module to GPLv3+

* perl-module/PSPP.xs:
* perl-module/README:

LOOP: Limit number of iterations when IF clauses present.

The LOOP command is limited by the MXLOOPS setting in some cirumstances.
Until now, if an IF clause was present, MXLOOPS was disregarded.
However, this behavior does not match SPSS behavior.  This commit makes
LOOP honor MXLOOPS even when IF is present, for compatibility.

With this commit, only the presence of an index clause makes LOOP
disregard MXLOOPS.

Thanks to Frans Houweling for reporting this bug.

Fixed incorrect syntax generated by the Select Random Cases dialog.

Reported by: Eric E Graig

pivot-table: Use ctime_r() instead of ctime(), for thread safety.

ascii: Avoid buffer overread outputting page longer than allocated lines.

This does seem like an odd case but I triggered it at one point and it
costs little to check for it.

pivot-table: Fix memory leaks in pivot_table_dump().

# Conflicts:
# src/output/pivot-table.c

pivot-table: Fix memory leak in pivot_value_destroy().

driver: Fix memory leak in output_submit().

perl-module/automake.mk (perl_module_tarball): Make build dir writable.

It seems that recent perl tries to write them.

po/Makevars: Add copyright and licence notices

zip-writer: New functions for adding members.

These will have additional users in an upcoming commit.

zip-reader: Improve error messages.

With this change, the zip-reader's error messages always include the zip
file name and, when relevant, the member name.  Otherwise it wasn't always
clear what a message liek "No such file or directory" was talking about.

zip-reader: New function zip_reader_get_member_name().

This allows the caller to obtain a list of .zip members.

zip-reader: Don't keep .zip file open for lifetime of zip_reader.

The zip_reader no longer uses the .zip file after it opens it, so close it
when it's no longer needed.

zip-reader: Read the whole central directory at .zip open time.

By reading the whole central directory when we open the .zip file, rather
than one entry at a time, we make it easier to get a list of entries and
make it available to the client.  (This will be implemented in an upcoming
commit.)

This commit also fixes memory leaks in the error case in zip_member_open().

This commit also changes the usage model.  Previously, if the caller opened
a single member more than once, all the members shared a single copy of
the member and read each other's data.  Now, each caller that opens a
member gets a separate copy and a separate file pointer.  Callers now need
to call zip_member_finish() when they're done with a member.

With this commit, the code only keeps an open FILE for a member if the
caller opened that member.  Previously, it kept an open FILE for any
member that had been encountered.

zip-reader: Fix memory leak on error path in zip_reader_create().

zip-reader: Better hide implementation details from clients.

By integrating the "inflate" code with the zip reader code, it is no
longer necessary to expose the zip_member or decompressor details.

output: Move text_item and group_item usage closer to the SPV model.

output: Reimplement table_from_string in terms of tab.

With this change, there is only a single table_class in the tree.

output: Remove table_unshare.

It was unused.

output: Remove table_select and table_paste.

table_select() was unused, and it was in turn the last user of
table_paste().

render: Drop use of table_select().

This is the last use of table_select() in the tree.

render: Modernize coding style.

Update gitignore.

doc: Update SPV file format documentation.

pivot-table: Improve comments.

AUTORECODE: Implement PRINT subcommand.

AUTORECODE: Use minimum width output format for new variables.

The target variables don't need any decimals, since they are always
integers, and only need to be as wide as their largest value, which is
known in advance.

AUTORECODE: Style update.

AUTORECODE: Properly handle variable labels.

AUTORECODE: Properly handle value labels.

Value labels are supposed to be copied from source variables, where they
are present.  Also, it's better to format the source values using the
print format rather than just assuming that %g is the right format.

variable: New function var_append_value_name__().

This variation on var_append_value_name() allows the caller to pass in the
parameter specifying how to show the value, which will be useful in an
upcoming commit.

variable: Make var_append_value_name() omit trailing white space.

This seems to be desirable in the cases where it's actually used.  It only
affects one test case in the current testsuite, but it's an improvement for
an upcoming change too.

AUTORECODE: Fix whitespace.

A previous commit introduced tab indentation.  Oops.

AUTORECODE: Fix treatment of missing values.

User-missing values are supposed to be recoded to a separate range above
the non-missing values.  The system-missing value is supposed to not be
recoded at all.  This fixes and documents that behavior.

INSTALL: Mention that Python 3 is supported.

It worked fine but the INSTALL doc claimed that it didn't.

AUTORECODE: Do not create value labels when they would be empty.

AUTORECODE: Code style update.

AUTORECODE: Make GROUP option work properly with multi-length strings.

When GROUP is used, all of the variables are supposed to be put into a
single table.  The implementation assumed in practice that, for string
variables, all of the strings were the same width, which caused problems.
This commit fixes the problem and updates the test to match.

/RENAME subcommand: Allow quoted strings in destination variables.

Allow syntax like /RENAME = old = "new var" old2 = "another new var".

Fixes bug #56771

perl-module: Add copyright and licence notices to all files

Descriptives: Constness

Pivot Table: Fix compiler warning.

Revert part of commit 9bd8bde378997bf189e6aceffe3da5d321d0a837 .
This was ill construed, because the footnote is owned by the pivot table.

pivot_table: Constness

perl-module: Add copyright and licence notices

doc/dev/*.texi: Add copyright and licence notices

doc/automake.mk: Add copyright and licence notice

Add copyright and licence notice to Makefile.am

value: Get rid of value_str(), value_str_rw(), value_num().

These can now all be replaced now by plain ->s or ->f, which is
shorter and clearer anyway.

value: Drop distinction between long and short string values.

The time has passed when this is a super valuable distinction.  There is
basically no distinction in the language itself anymore, the efficiency
gains sound pretty marginal, and the differences lead to bugs, both
correctness- and memory leak-wise, that are hard to test for (because the
developer must make sure to test both short and long strings everywhere).
This commit changes the code so that both short and long strings are
represented the way that long strings were previously.

t-test-parser: Re-fix memory leak.

I managed to screw this up in untabification.

Fixes: 759791b6dee8 ("t-test-parser: Untabify.")

perl-module: Init and destroy for long strings in set_missing_values().

Otherwise long strings segfault.

t-test-parser: Untabify.

This file was inconsistent on tabs versus spaces.  Usually we use spaces
in PSPP.

t-test-parser: Force values to be specified with a string vars on GROUPS.

t-test-parser: Fix memory leak with GROUPS and long strings.

categoricals: Fix memory leak allocating long string categoricals.

shapiro-wilk: Fix memory leak.

CC: John Darrington <john@darrington.wattle.id.au>
Fixes: 0b7e8882ce9b ("EXAMINE: Implement the Shapiro-Wilk Test.")

EXAMINE: Fix leak in Shapiro-Wilk test.

CC: John Darrington <john@darrington.wattle.id.au>
Fixes: 0b7e8882ce9b ("EXAMINE: Implement the Shapiro-Wilk Test.")

AUTORECODE: Declare index variables inside 'for' loops.

Just a style update.

ABOUT-NLS: Update to most recent version from gettext-0.20

AUTORECODE: Always report parse errors.

The code didn't always report syntax errors that it found.

lexer: Make lex_error_expecting() easier to use.

With this change, the caller doesn't have to explicitly supply the null
sentinel since the macro does it automatically.

pivot-table: Properly allocate pivot_table's current_layer.

The default current_layer should be all-zeros, but the code wasn't
allocating current_layer at all.  This didn't cause much of a problem
because nothing in the current tree allocates layers, but it will as
layers are added.

Thanks to John Darrington for reporting the bug.

system-file-format.texi: Correct typo.

Bug #56547.

README.Git: Advance to Gnulib commit 9ce9be2ef0cb ("autoupdate").

A recent commit to PSPP depended on a more recent version of Gnulib that
includes Gnulib commit 3cd207e1d0fd ("fatal-signal: Pass the signal number
to the action.") but the PSPP commit did not actually advance the Gnulib
dependency to that commit.  This commit does that (actually it advances to
Gnulib tip of master).

Fixes: 21d2449f801c ("unlink_replace_files: Change signature to avoid compiler warning.")

Remove UNUSED attributes where they were inappropriate.

These parameters were marked UNUSED, when they were in fact,
used.

Recent to GTK+3.22

means-calc.c: Add comments where needed.

Apply Dictionary: Fix memory leak.

Added test for the APPLY DICTIONARY command.

There appeared to be no existing test for this command.

PsppireOutputView.c: Fix memory leak.

Constness

PSPP.xs: Remove unused variables.

terminal-reader.c: Remove unused variable.

PsppireOutputViewer.c: Remove non-functional debug feature.

Use non-deprecated gtk things

unlink_replace_files: Change signature to avoid compiler warning.

MEANS: Don't compile debugging functions

PsppireDict: Constness

Descriptives: Constness

TEST Examine unrepresentable graphs: Ignore stderr

MEANS: Fix output with unusual formats

MEANS: Change the statistic formats to follow their respective variables

Don't crash when inserting variables into an empty sheet.

Fixes bug #56392.

Add a comment

MEANS: Fix behaviour when splits is active.

MEANS: Fix memory leak

Re-implement MEANS.

Fixes bug #44264.

pivot-table: Better document how to hide empty rows and columns.

Suggested by John Darrington.

quick-cluster.c Fix uninitialised variables.

Gcc 7.4 issued a warning about this.

doc: Better describe the effect of OUTFILE on PRINT.

Reported by Nolan Void <nolvoa@hanakami.com>.

Revert "Restart of new means implementation"

This reverts commit ad7ae8bb105c925b951e241fd6a3d1cb93d614a0.
I inadvertently pushed this!

Revert "Remove bad tests"

This reverts commit b9a1a1c32d5589e208ea751911d35177b7751720.
I inadvertently pushed this!