DEFINE: Don't label a declaration, for C99 compatibility.

Thanks to Friedrich Beckmann for reporting the problem.

Fixes: 36fc62cea799 ("macro: Coding style improvement in macro_tokens_from_string().")Fixes: b8f7ae6610f0 ("DEFINE: Improve error messages for parsing macro body.")

AGGREGATE: Improve stats accumulation data structure.

str: Replace ss_alloc_substring() by ss_clone() and similarly with a pool.

This makes it possible to use in an initializer.

AGGREGATE: Use array instead of linked list.

AGGREGATE: Remove write-only struct member.

macro: Coding style improvement in macro_tokens_from_string().

value: Update comments.

These used to be true but they no longer are, so remove them.

segment: Comment fix.

DEFINE: Improve error messages for parsing macro body.

Without this change, errors always got reported following the macro body,
not for the particular token at issue.

Improve error messages for missing subcommands.

format: Cite variable names in messages reporting incompatible width.

Cite tokens when reporting invalid identifiers.

These functions weren't able to cite a specific token in cases where that
was appropriate.  This fixes the problem.

Improve error messages by citing syntax in more of them.

A while back, PSPP gained the ability to cite particular tokens in
diagnostics, but few of the existing messages were enhanced to do
so.  This commit adds this to several more of them, in particular
all of the messages that used lex_error() and variations.

CTABLES: Allow CAPTIONS, CORNERS, TITLES in TITLES subcommand.

Matthias Faeth reported that SPSS accepts these spellings.

pspp-dump-sav: Do not install by default.

People keep fuzzing pspp-dump-sav and reporting vulnerabilities against it,
but hardly anyone uses it (maybe just me).  It's not worth the trouble.

Fixes: #62977.

CTABLES: Make ID values reproducible when category labels are moved.

ID values were assigned in an arbitrary order when category labels were
moved, because the pivot table leaf index for the category dimension was
not included in the sort.  This fixes the problem.

This also factors finding the category leaf index out of the inner loop in
ctables_table_output().  This index doesn't vary across that inner loop and
I simply hadn't noticed while writing the code.

Implement CTABLES procedure.

lexer: New function lex_ofs_error().

message: New function msg_location_merged().

FREQUENCIES: Implement LAYERED option on SPLIT FILE.

pivot-table: Introduce infrastructure for handling split files.

This will be used for the first time in FREQUENCIES in an upcoming commit.

pivot-table: Allow all pivot_value formatting functions to use defaults.

dictionary: Limit split file variables to 8, for compatibility.

dictionary: Introduce SPLIT_NONE for dictionaries without split variables.

This also implements dict_get_split_type(), which previously had a
prototype but no implementation.

dictionary: New function var_force_valid_weight().

Automatically infer variables' measurement level from format and data.

Use rsvg-convert instead of GIMP for converting SVG to PNG.

rsvg-convert is much faster and more commonly installed.

str: New functions for case-matching of substrings, prefixes, and suffixes.

str: New function ds_put_substring_multiple().

lexer: Fix negative 'n0' and 'n1' in lex_source_contains_macro_call().

lexer: Support more options in lex_error_expecting_array().

pivot-table: Improve comment.

pivot-table: Avoid making PIVOT_N_AXES an enum member.

Otherwise the compiler will complain about it not appearing in a switch.

format-parser: Comment and style fixes.

doc: Improve documentation for macro support.

po: Remove reference to $(all_q_sources), which doesn't exist anymore.

Fixes: 0a18cc59b123 ("q2c: Remove, since it is no longer used.")

variable: Also complain about negative weights in var_force_valid_weight().

FREQUENCIES: Use the smallest mode if there are multiple.

This matches the documentation.

render: Coding style fix.

doc: Improve description of extension subtype 5.

pspp-output: Improve usage message.

FREQUENCIES: Fix memory leak with percentiles and split file.

FREQUENCIES: Fix use-after-free error.

syntax-gen: Test the category properly.

This code was testing the format type against a collection of format type
categories, which doesn't make any sense.  This commit fixes the problem.

Found by inspection.

g_memdup2: added src/ui/gui/glibfix.h to the distribution file list

I forgot to include the new header file for the previous fix in the
list of files for the distribution.

supply g_memdup2() for glib before 2.68.

The previous commit did not work on debian. I followed the ideas from

https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1927

and

https://github.com/mdbtools/gmdb2/commit/62ce520a9c5

So this commit provides g_memdup2 for glib versions which do not
provide g_memdup2.

Revert "Add header wrapper for glib to supply g_memdup2() for glib before 2.68."

This reverts commit 652517f6e99bca918bf2275003f5d51fe5f2f0c6.

Add header wrapper for glib to supply g_memdup2() for glib before 2.68.

Friedrich Beckmann reported that some of our build targets have glib as
old as 2.58.

SPLIT FILE: Add LAYERED vs. SEPARATE distinction to the dictionary.

attributes: New macro ATTRSET_INITIALIZER.

gui: Use g_memdup2() instead of deprecated g_memdup().

configure: Update autoconf and libtool macro use to fix warnings.

Fix use-after-free errors.

Found by GCC 12.1.1.

Suppress GCC 12.1.1 warnings.

I don't think these warnings are for real problems, so just suppress them.

I don't understand why the change to datasheet-test.c suppresses a warning,
but it does.

Release version 1.6.2.

gui: Fix icon names.

Using icon-name "org.gnu.pspp" makes the window manager show the PSPP
icon instead of a generic one.

Reported by Friedrich Beckmann.

tests/atlocal.in - switched from which to command -v

Sam James reported a possible future build problem due to the
usage of the "which" command:

https://savannah.gnu.org/bugs/?62675

I replace "which" with "command -v" as proposed. Thanks for your
support.

Closes: 62675

pspp specific icon not shown in about menu

The changes of the pspp icon name in

commit 52ba15f3dce

were missing in the help about menu such that the icons were not
shown.

DESCRIPTIVES: Fix bad error message.

Smake: Fix icon rendering by using correct application ID.

PSPP-specific icons were not showing up properly in the GUI because
the compiled-in resources were looking in gnu.pspp whereas the name
is now org.gnu.pspp.  This fixes the problem.

Fixes: a421d8d9f61b ("Change application ID from org.fsf.pspp to org.gnu.pspp.")

Release version 1.6.1.

Smake: Fix dependencies for src/ui/gui/resources.xml.

This file is a list of all the icon files, but generating it doesn't use
icon files themselves, so depending on them isn't useful.  On the other
hand, the file does need to be regenerated if the icon list changes, and
that icon list is in Smake itself, so this target does need to depend on
the Makefiles.

po: Update Hungarian translation.

Thanks to Meskó Balázs for updating this translation.

Change application ID from org.fsf.pspp to org.gnu.pspp.

This includes changing the argument of gtk_application_new() from
"gnu.pspp" to "org.gnu.pspp" for consistency.

Suggested by Hubert Figuière at https://github.com/flathub/flathub/pull/3275.

perl: remove references to Memory::Usage module for perl module tests

The perl memory test was removed with commits

https://git.savannah.gnu.org/cgit/pspp.git/commit/?id=20934a68593edbd52577f1ba0ea93c74eb018af5

and

https://git.savannah.gnu.org/cgit/pspp.git/commit/?id=05297509d237dc99ba7504ccf260e995586bc3b4

This commit removes the remaining references to the perl memory test.
The configure step still asked for optional Memory::Usage perl module
which is not used without the perl memory test.

gui: make import assistant window a modal window

I changed the import assistant window to a modal window such that
the other windows cannot be changed while the import is ongoing.
There are problems when the main window is closed during the import.

Closing: https://savannah.gnu.org/bugs/?62448

Implement SET LEADZERO.

configure: Increase version number to 1.6.1.

po: Update ca,es,de,nl,uk translations from translationproject.org.

Many thanks to Francesc Josep Miguel Quesada, Jaap Verhage,
Yuri Chornoivan and the other members of the translation team!

perl-module: Remove loading of memory:usage module

The memory leakage/usage test was removed with commit
https://git.savannah.gnu.org/cgit/pspp.git/commit/?id=20934a68593edbd52577f1ba0ea93c74eb018af5

but the memory module was still loaded. On debian this
memory module requires an additional dependency
libmemory-usage-perl which is not required. As the
memory module is not used inside the test I removed it.

Update to newest Gnulib.

line-reader: Avoid undefined behavior in printf utility in test.

The test used
    printf '\"'
assuming that it would print just a double-quote, but on at least one
system it printed a backslash followed by a double-quote.  This commit
changes it to
    print '"'
which should be reliable everywhere.

Thanks to Michael Orlitzky <michael@orlitzky.com> for reporting this bug.

tests: Add test for doubled parentheses.

I can't reproduce the problem reported in a bug report, but it seems
worthwhile to add a test for it anyway.

Bug #62557.

lexer: New function lex_ofs_representation().

This interface is more convenient for situations where it's easier to
consume tokens before getting their representations.

mode: New order statistic.

math: Make 'accumulate' a feature of order statistics, not all stats.

subcase: Rename subcase_destroy() to subcase_uninit().

This reflects the convention that "destroy" frees the pointer passed in
while "uninit" only frees the internals.

math: Coding style updates in some order-stat implementations.

I generally find structure assignments clearer than individual assignments
to each of their members these days.

math: Improve comments.

This includes some changes to parameter names and coding style in
order-stats that I think clarify the code and modernize it.

math: Avoid unneeded extra allocations for fixed-size data structures.

This seems generally a bit cleaner to me.

spv-legacy-decoder: Always give the pivot table a title.

A pivot table must have a title.  Some parts of PSPP rely on that.  The
legacy SPV decoder didn't ensure this, and one SPV in my corpus didn't have
one, which led to null pointer dereferences, so this commit fixes the
problem.

output-item: Add missing new-line printing tables in output_item_dump().

This only affects "pspp-output dump".

output-item: Fix memory leak in output_item_dump().

pivot-table: New functions for setting captions, etc.

pivot-table: Fix pivot_table_dump() null pointer dereference in special case.

This function only gets called from "pspp-output dump", so it's not a very
important bug.

Release version 1.6.0.

perl-module: Copy Perl module from srcdir to builddir before distributing.

I ran into an issue where the Perl tests were failing in distcheck, but
not in check.  I changed the sources for the Perl module, but distcheck
still used the old ones.  It turns out that "make dist" uses files from
the builddir, which "make dist" wasn't updating from the srcdir because
that only happened when the Perl module was actually built, which "make
dist" and "make distcheck" don't do.

This commit fixes the problem by ensuring that the Perl module files get
updated in the builddir on "make dist".

perl-module: Remove test for memory usage.

This test only works if you're lucky.  It doesn't work with Address
Sanitizer and it doesn't seem to work with current versions of Perl.
(It might be that there's a real leak, but I don't know how to debug
it in that case.)

po: Update translations from translationproject.org.

Many thanks to our translators!

gui: For text data import, use the same parser for preview as for import.

There were odd discrepancies between the preview code in the GUI for
text data import and the code in GET DATA /TYPE=TXT that implemented the
eventual import.  This commit unifies the two parsers, dropping the one
that was used only for preview.  This should prevent further discrepancies.

Bug #61809.
Thanks to Friedrich Beckmann for reporting this bug.

data-parser: Make data parser not require a dictionary.

An upcoming commit will make the GUI's text import use this code too, in
a context where a dictionary isn't yet available.  The data parser only
uses the dictionary for one specific purpose that isn't relevant for the
GUI, so remove the need for it.

gui: Make text import assistant accept only one quote character.

This aligns with what SPSS does.

gui: Always use the most comma text import settings.

The heuristic did not perform well in practice according to user reports,
and most files use the defaults.

gui: Honor quotes in the text data import dialog.

The text data import dialog's preview of the data to be read
didn't honor the quotes but always treated " and ' and (weirdly)
parentheses as quotes.  This makes it honor the user's settings.

Bug #61809.
Thanks for Friedrich Beckmann for reporting this bug.

New command SHOW SYSTEM to easily print information useful in bug reports.

po: Update translations from translationproject.org.

po: Update de translation from translationproject.org

Release version 1.5.5.

po: Update translations from translationproject.org.

expressions: Convert SYSMIS into int as INT_MIN during optimization too.

In an expression like LPAD('abc', $sysmis), the LPAD function is evaluated
using a C function that takes an 'int' as its second argument.  During
expression evaluation, out-of-range values get converted into SYSMIS by
the NUM_TO_INTEGER operator that gets automatically inserted to ensure
that, and then SYSMIS gets converted into INT_MIN to pass to the C
function.  This worked OK for runtime evaluation, but the optimizer failed
to do the conversion of SYSMIS to INT_MIN, which yielded undefined
behavior.  Some platforms (e.g. GCC on x86) converted SYSMIS (which is
the second-smallest 'double' value) into INT_MIN, so no problem showed up
there, but other platforms yielded INT_MAX, causing unexpected failures.

This commit fixes the problem by explicitly converting SYSMIS into INT_MIN
during optimization.

Thanks to Friedrich Beckmann for reporting and uncovering the problem.
Bug #62267.