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.

po: Update translations from translationproject.org.

org.fsf.pspp.metainfo: changed links to https / added 1.5.4 release

The debian checker mentionend the http links for the version info.
So I changed them to https. And I added the 1.5.4 release.

po: Update German translation from translationproject.org.

Thanks to the german translation team at translationproject.org

start message to translators with TRANSLATORS

All comments which start with TRANSLATORS are visible for the
translators. The previous message is only visible if the translator
looks into the source code.

MATRIX: Correct error message.

Thanks to Friedrich Beckmann for reporting the mistake.

po: Update Spanish translation from translationproject.org.

Thanks to Francesc Josep Miguel Quesada for the updates!

pivot-table: Avoid null dereference in "pspp-output dump" in corner cases.

CROSSTABS: Fix computation of adjusted standardized residual.

Bug #60982.

psppire-output-view: Export correctly to drivers that don't handle groups.

Thanks to Elias Tsolis for reporting this bug.
Bug #61549.

Yuri Chornoivan contributed some typo fixes

Closes: https://savannah.gnu.org/bugs/?62245
Thank you for your contribution!

po: Update Ukrainian translation from translationproject.org.

Thanks to Yuri Chornoivan for translating.

Release version 1.5.4.

sys-file-encoding: Clean up .c file on distclean.

Found by "make distcheck".

icons: Fix uninstall.

The uninstall target for the icons looked for filenames that began with
"pspp", but the new names for the application icons don't start that way.

Fixes: 25e030629aed ("Rename icon files to match full application ID.")

bug #62189: Fixed clipped numbers in histogram legend

I modified the legend rendering to avoid the clipping of mean and
standard deviation in the legend of the histogram plot. In addition
I changed the number print format to also show reasonable numbers
for small values.

fixed: https://savannah.gnu.org/bugs/index.php?62198

Windows: Native build failed due to missing gnulib dependencies

The windows build failed because the gnulib header files were not
build before the compilation of c files for the native build.
The gnulib headers are
listed in the BUILT_SOURCES but that must be the first dependency
before the object files belonging to the executable. This is the
case for the default make target but not for an individual
executable like src/ui/terminal/pspp. I adapted the make file
to use the default make target.
See: https://www.gnu.org/software/automake/manual/html_node/Sources.html

Windows build: Adapted icon names to org.fsf.pspp naming

Ben changed the icon names to a naming scheme using org.fsf.xxx
The changes were missing in the windows build resulting in a
build failure for the windows target.

svg2png: Use rsvg-convert if GIMP is not available.

The GNOME flatpak SDK has rsvg-convert, which does just fine, but not GIMP,
and I have no interest in building GIMP as a step in building PSPP.

Rename pspp.xml to org.fsf.pspp.xml.

This reflects the convention expected by flatpak and seems to reflect the
overall expected convention.

Rename icon files to match full application ID.

This seems to be the convention.  It makes building a flatpak easier.

org.fsf.pspp.metainfo.xml.in: Fix release dates.

These dates were in YYYY-DD-MM format, but they should be ISO 8601
compliant YYYY-MM-DD format.

configure: Also support gtksourceview 4.x.

This doesn't require any source code changes because 4.x and 3.x are
source compatible for our purposes.

Makefile: Fix running "make dist" before "make all".

Otherwise, there's a race between generating and using config.h, which
causes build failures.  This happens particularly with the PSPP autobuilder.

Switch from recursive to non-recursive use of Gnulib.

This allows for better use of dependencies in Makefiles.  It causes a new
warning from Automake, which I hope to fix through a change to Gnulib
sometime soon.  I'm consulting the other Gnulib maintainers on that:
https://lists.gnu.org/archive/html/bug-gnulib/2022-03/msg00044.html

Gnulib needs Automake 1.14 or later.

tests: Update testsuite results to accomodate new Gnulib.

It seems that libunistring now does line wrapping slightly differently,
perhaps due to Unicode updates.

Update to newest gnulib.

This required a dependency on Automake 1.11 or later.  It also required
dropping C2x compatibility warnings because Gnulib now triggers them.  The
Gnulib tempname module requires linking against $(LIB_GETRANDOM), so this
adds that in each case where the build links against libgl.

spv-light-decoder: Avoid GCC 11.x warning in decode_spvlb_halign().

It seems to be wrong, but GCC can't tell and it's hard to blame it.

expressions: Fix value of $TIME variable and add a test.

Thanks to Frans Houweling for reporting this bug.
Bug #62003.

psppire-dialog-action-crosstabs: Do not add PIVOT or NOPIVOT to command.

The CROSSTABS command does not support /FORMAT=PIVOT or /FORMAT=NOPIVOT.
I can't find any evidence that it ever did.  Perhaps q2c ignored keywords
it didn't recognize?  SPSS does not document support for any such format
keywords, so it's better to remove them from the emitted syntax than to
add support to CROSSTABS.

Thanks to Alexandra Younger for reporting the issue.

lexer: New functions for parsing real numbers in specified ranges.

These will be more useful in the future.

format-parser: Improve comment.

FREQUENCIES: Fix percentiles and median calculation for multiple variables.

The calculations here were using the same space each time for
calculating percentiles and median, but different areas are needed.  This
commit fixes the problem.

Bug #60728.
Thanks to elias tsolis for reporting the problem.

FREQUENCIES: Remove unused pool.