fix up test output

more improvement

Improve MGET output table.

Basic MGET tests work.

More MATRIX SAVE negative tests.

MATRIX SAVE negative tests

MATRIX SAVE inline positive tests.

MATRIX SAVE positive tests.

GET is well tested.

MATRIX GET works with the active file.

MATRIX GET positive test

READ and WRITE are tested.

format: Make fmt_check() easier to translate.

Test REREAD also.

More READ negative tests.

READ negative tests

Make tests pass again.

matrix docs

MATRIX: Implement HOLD (untested).

MATRIX: Factor out write_file, like read_file.

MATRIX: Fix leak on error path.

Add an example for MATRIX PRINT.

More matrix examples.

work on examples

MATRIX command documentation is complete (but needs examples)

Document more matrix stuff.

documentation

Improve matrix error messages.

MATRIX: Fix memory leaks.

Work on documentation

docs

function documentation

work on docs

MATRIX

dataset: Fix memory leak destroying a dataset that has a permanent_dict.

Found by Address Sanitizer.

pivot-table-test: Fix memory leak when table is not displayed.

Found by Address Sanitizer.

lexer: Fix memory leak merging tokens only some of which come from macros.

Found by Address Sanitizer.

macro: Fix memory leaks in error cases parsing function arguments.

Found by Address Sanitizer.

lexer: Fix memory leak when macro expands as empty.

Found by Address Sanitizer.

macro: Fix memory leak expanding !DO loop over list.

Found by Address Sanitizer.

macro: Fix memory leak with keyword "enclose" arguments.

The memory for the argument was being allocated two places, which caused
the first-allocated block to be leaked.

Found by Address Sanitizer.

segment: Fix read past end of buffer when input ends in '-'.

Thanks to John Darrington for reporting this.
Found by Address Sanitizer.

expressions: Parse multiple sets of parentheses for grouping together.

Fuzzers are fond of driving expression parsers to failure by exhausting
the stack in trivial ways.  This defeats the simplest attempts by
lining up thousands of left parentheses in a row.

I am a bit curious whether the fuzzer will now invent something more
sophisticated, such as nested function calls or non-empty expressions like
1+(1+(1+(1+(1+....

This fixes bug #61286.
Thanks to Irfan Ariq for reporting the bug.

DATA LIST: Fix assertion when RECORDS given twice with decreasing value.

Fixes bug #61285.
Thanks to Irfan Ariq for reporting this bug.

dictionary: Allow dict_set_documents() argument to reference old documents.

merge_dictionary() in combine-file.c includes the old documents in the new
ones by just copying pointers.  dict_set_documents() didn't handle this
properly.  This fixes the problem.

Fixes bug #61258.
Thanks to Irfan Ariq for reporting the problem.

encoding-guesser: Avoid reading uninitialized data for zero-length files.

Found while investigating bug #61254.
Thanks to Irfan Ariq for reporting this bug.

segment: Fix 1-byte read past initialized data when file ends in CR.

Fixes bug #61253.
Thanks to Irfan Ariq for reporting this bug.

Remove some unnecessary gettext macro definitions

Fix compiler warning

Fix memory leak in tokenize_string_segment

Fix memory leak in MCONVERT

Fix memory leak upon failure to create matrix reader

More ASAN_OPTIONS

Use XCALLOC / XZALLOC macros where reasonable

Fix possible segfault when running RANK with bad syntax

* src/language/stats/rank.c (): Initialise the vars member

Fixes bug #61257

Replace numerous instances of xzalloc with XZALLOC

Matrix readers - fix memory leaks

* src/language/data-io/matrix-reader.c (matrix_reader_destroy): free members cvar, svars and fvars

Ignore sanitizer ODR warnings, during testing with -fsanitize=address

* tests/libpspp/sparse-xarray-test.at: Set detect_odr_violation=0

Implement the MCONVERT command.

lexer: Fix use-after-free error in lex_source_get_lookahead().

This code used local variable 'out' as if its value stayed the same from
one iteration of the loop to the next, but in fact its scope meant that
it became indeterminate on each new iteration.  This commit fixes the
problem by moving its declaration to an outer scope.

Thanks to John Darrington for reporting the problem.

Avoid numerical problems with missing weights on non-linear cases.

* src/math/order-stats.c (order_stats_accumulate_idx): Ignore cases
  with missing weight values.

Fix possible incorrect assertion when creating unique casereaders.

* src/data/casereader-translator.c (uniquify): Force dir to be an element of  {0, 1, -1}

Remove unused module src/math/extrema

* src/math/extrema.c: Delete
* src/math/extrema.h: Delete
* src/math/automake.mk: Remove entries for src/math/extrema.c and
  src/math/extrema.h

Options dialog: add checkbox for startup tips

sys-file-encoding: Automatically generate the .c file at build time.

Suggested by John Darrington.

Windows/build-dependencies: Add -fstack-protector flags

For some reason this flag seems to be necessary with the latest
x86_64_w64-mingw32 toolchain.

Windows/build-dependencies: Use correct logical OR operator

Windows/build-dependencies: New flag --no-clean

Add some missing #include directives

* src/language/stats/crosstabs.c: Add missing #include directive.
* src/libpspp/pool.c: Add missing #include directive.

MATRIX DATA: Fully implement.

This command had a partial implementation for correlation matrices that
left out some of the language features.  This commit adds those features.

case: Introduce new functions for numbers and substrings in cases.

Use the case_num_*() functions everywhere in the tree for clarity and
brevity.

sys-file-encoding: Put the buffer-read-only declaration at the very top.

This accidentally got pushed down when license notices were added to a
bunch of files en masse.  It only works if it's at the top.

Remove unneeded Emacs declarations that say that a .c file is in C.

These were there because they were formerly .q files.

DEFINE: Properly support redefining a macro.

Redefining a macro didn't work in simple cases because the macro name was
being expanded in the DEFINE command.

Thanks to Frans Houweling for reporting the bug.

macro: Fix crash for !QUOTE of a quoted string.

This revealed that the tests didn't include anything for !QUOTE and
!UNQUOTE, even though the manual had lots of examples.  I added a test for
these and for !NULL, which had also been forgotten.

Thanks to Frans Houweling for reporting the bug.

Avoid GtkCritical on startup

TeX tests: Use the shell instead of wc to test for maximum line length.

When testing for maximum line length use the shell rather than
relying on wc -L  : The -L flag is not present on some systems.

tests/tex.at: Remove dependence on wc
configure.ac: Remove test for wc -L

Fixes bug #59859

Fix import of ods files with repeated column data.

Fixes bug #61078

Reported-by: Elias Tsolis

Fix crash when double clicking on variable sheet cells when no variable is defined.

Reported-by: Maruthi Pathapati

DEFINE: Only expand macro functions when the name is followed by '('.

Frans Houweling reported that PSPP was flagging an error for !EVAL(!len)
when !len was the name of a defined macro.  This was because !len is short
for the !LENGTH macro function.  This commit fixes the problem.

Tolerate pathnames with spaces

Reported by: Vivien Kraus

macro: Continue expanding macro even in face of errors in call.

In practice, it was more confusing not to expand it than to expand it.

macro: Allow positional arguments to be empty.

It wasn't clear before that this was allowed, but it seems that it should
be for compatibility.

Reported by Frans Houweling.

macro: Make ARG_CHAREND and ARG_ENCLOSE more uniform in struct macro_param.

A few pieces of code want to find the end of a parameter and it's easier
if the "end" token is always the same member.

identifier: Make T_STOP always 0.

This is more sensible default if a token is zero-initialized.

macro: Allow macro A to use its arguments as part of call to macro B.

I overlooked this possibility before.  This implements it.

Thanks to Frans Houweling for reporting the issue.

Turn !* into a single token, for macro expansion purposes.

DEFINE: Equals sign is optional for both positional and keyword parameters.

Thanks to Frans Houweling for reporting this bug.

macro: Properly parse !ENCLOSE keyword arguments.

The opening delimiter was being included in the argument.

Thanks to Frans Houweling for reporting this bug.

DEFINE: Don't use PSPP_CHECK_MACRO_EXPANSION macro in tests.

I found that this macro just obscured things, so I expanded it in each
case and removed the macro itself.

DEFINE: Allow !DEFAULT to follow the argument type declaration.

Frans Houweling reported that SPSS allows either order.

lexer: Change the pipeline to allow more flexible use of macros.

Frans Houweling reported that a use of macros in the following way:

    DEFINE !dir() "/directory/to/my/work" !ENDDEFINE.
    GET FILE=!dir + "/filename.sav".

did not work properly with the newly implemented PSPP macro facility.
Indeed, PSPP has until now implemented string concatenation early in the
lexical pipeline, before macro expansion, so that the above could
not work.  This commit reworks it so that string concatenation happens
as the last stage in lexical analysis.  It allows the above syntax
to work as expected.

str: New function ss_swap().

segment: Make negative numbers into single segments.

configure: Enable GCC warnings to report use of C2x features.

One C2x feature in GCC 11 is the reason that the previous commit
afda22462a88 ("Fix broken build due to missing braces") was needed by
some developers.

Fix broken build due to missing braces

DEFINE: New command.

lexer: Move lex_ellipsize() into string module, as str_ellipsize().

token: Update functional interface and add token_copy(), token_equal().

These will have users in an upcoming commit.

segment: Ignore !ENDDEFINE in /*comments*/ and "strings".