transformations: Fix memory error in trns_chain_prepend().

This only worked if the transformation chain had one element.

Thanks to Zhou Geng for reporting this bug as poc30 in the report here:
https://lists.gnu.org/archive/html/bug-gnu-pspp/2024-03/msg00015.html

LIST: Fix bug listing duplicate variables.

Thanks to Zhou Geng for reporting this bug as poc28 in the report here:
https://lists.gnu.org/archive/html/bug-gnu-pspp/2024-03/msg00015.html

SHOW: Fix memory leak on error path.

SHOW: Fix crash in SHOW TITLE and SHOW SUBTITLE if there is no (sub)title.

Thanks to Zhou Geng for reporting this bug as poc14 in the report here:
https://lists.gnu.org/archive/html/bug-gnu-pspp/2024-03/msg00015.html

po: Update translations from translationproject.org.

u8-line: Fix handling of double-width characters in u8_line_reserve().

This code was terribly confusing.  This adds a little redundancy but it is
easier to understand.

This is difficult logic and more bugs may lurk.

Thanks to Zhou Geng for reporting this bug as poc13 in the report here:
https://lists.gnu.org/archive/html/bug-gnu-pspp/2024-03/msg00015.html

u8-line: Fix read past end of buffer in u8_line_find_pos().

Fix bug found by fuzzer.

Thanks to Zhou Geng for reporting this bug as poc9 in the report here:
https://lists.gnu.org/archive/html/bug-gnu-pspp/2024-03/msg00015.html

po: Update translations from translationproject.org.

po: Update translations from translationproject.org.

po: Update translations from translationproject.org.

po: Update translations from translationproject.org.

po: Update translations from translationproject.org.

gui: Translate names of aggregation functions in bar chart dialog.

Thanks to Mindaugus for reporting the problem.

Release version 2.0.1.

doc: Document how to run command-line PSPP from the Flatpak.

README.Git: Update to latest version of Gnulib branch stable-202307.

REPEATING DATA: Remove documentation for unimplemented command.

This command once had a buggy implementation, which was removed, but the
documentation remained.

Thanks to Frans Houweling for reporting the inconsistency.

gui: Mark "Show tips at startup" for translation.

Thanks to Mindaugus for reporting that it was missing.

po: Update translations from translationproject.org

DO REPEAT: Accept duplicate names as substitution variables.

Thanks to Frans Houweling for reporting the bug.

CROSSTABS: Fix tabular output when columns or rows are empty.

Thanks to Andreas Hammer for reporting the bug.

AUTORECODE: Fix crash if scratch variables created before source variables.

Thanks to Frans Houweling for reporting this bug.

dictionary: Remove unused functions and prototypes.

psppire: Sort by name or label case-insensitively in dictionary view.

Thanks to Elio Spinello for reporting the bug.

SORT VARIABLES: Add test case for case sensitivity.

CTABLES: Fix crash with single nesting level, no label, and moving labels.

This fixes a crash for the following input:

DATA LIST LIST NOTABLE /year (F4.0).
BEGIN DATA
2020
2021
2022
2020
2020
END DATA.

CTABLES
  /VLABELS VARIABLES=year DISPLAY=NONE
  /TABLE year [C]
  /CLABELS ROWLABELS=OPPOSITE.

Thanks to Frans Houweling for reporting this bug and providing the test
case.

tests: Remove two tests that proved problematic cross-OS and cross-arch.

The test "journal enabled by default interactively" failed in Debian
testing on s390x, powerpc, and ppc64.  I can't reproduce the problem on
x86_64.  This commit removes it.

The test "interactive output appears immediately" failed in Debian testing
on s390x, hppa, powerpc, and ppc64.  It also previously failed on Mac OS.
I can reproduce the problem on x86_64 if readline isn't available, but not
if it is.  (I think it would always be available for Debian).  This commit
removes it.

Thanks to Friedrich Beckmann for reporting the Debian build problems.

doc: set LANG=C for building the tutorial images

The build process fails if the shell has set a different language
setting the us. This patch sets the language during the generation
of the documentation images.

Closes: https://savannah.gnu.org/bugs/?65124

Update releases in appstream metadata to make flathub happy.

Release version 2.0.0.

MacOS: Fix skipped test "convert invalid UTF-8 to ISO-8859-1"

The skipping worked, but the non-skipped test failed on linux
systems.

Revert "Revert "MacOS: skip test "convert invalid UTF-8 to ISO-8859-1"""

This reverts commit bcc19d9457129559b836a0cc6cd81e2c71d42bb5.
Just letting Mac OS crash on invalid conversions seems like the
most reasonable path for now.  Maybe Apple will fix their buggy
iconv.

Revert "i18n.c: added EOPNOTSUPP error code for MacOS"

This reverts commit 1799abc8539bd1a49c72c5723d1fa89ff4463e3f.
Handling EOPNOTSUPP like EILSEQ produced a hang rather than
the expected results, so this was a failed experiment.

Upgrade to stable Gnulib branch stable-202307.

Suggested by Bruno Haible.

i18n.c: added EOPNOTSUPP error code for MacOS

With MacOS 14 the iconv library was changed by Apple. pspp uses
the native iconv library. There is a different error code
returned when an illegal 0xc0 UTF-8 code is converted. This
results in a failed "convert invalid UTF-8 to ISO-8859-1" test in
the regression. This patch now handles also EOPNOTSUPP.

Revert "MacOS: skip test "convert invalid UTF-8 to ISO-8859-1""

This reverts commit 579295f8b15068b121807168e84330c1f99361e9. Ben
has an idea how to fix this in libpspp to handle the different
error code on MacOS.

MacOS: skip test "convert invalid UTF-8 to ISO-8859-1"

Starting with MacOS 14 Sonoma Apple has changed the behaviour
of the iconv library. The test tests the behaviour for invalid
UTF-8 sequence 0xC0. The native Apple iconv library responds with
a different error number than expected. This patch skips the
test on MacOS until this is hopefully fixed on MacOS.

Unconditionally replace g_string_free_and_steal().

It's a persistent problem and I don't understand why.  I hope this fixes
the issue for good.

Thanks to Ali Alhaidary for reporting this bug.

Bug #64848.

Release version 2.0.0-pre3.

tests: Fall back to running the interactive output test on GNU/Linux only.

It fails on Mac OS for unclear reasons.

tests: Refine the last change.

The remaining failures might be attributable to journal file creation
errors.

tests: Try another strategy to check for immediate output.

The previous strategy hung on some machines and failed on others.  The
failures are sometimes sensitive to whether libreadline is available.  I
hope that this avoids the problem.

tests: Stop using squish-pty.

squish-pty seems to break easily, so switch to using another way to
simulate interactive behavior.  This way requires a small source code
change to PSPP itself but it should be more portable.

Thanks to Friedrich Beckmann for reporting the problem.  The problem that
Friedrich reported was a hang in the "interactive output appears
immediately" test on Debian sid.  I couldn't reproduce that problem on
Fedora or Debian testing, but I did get failures on Debian testing.  I
guess it's best to just avoid ptys at all.

Fix typo in user string.

Thanks to Benno Schulenberg for reporting this bug.

po: Update translations from translationproject.org.

Release version 2.0.0-pre2.

Enable journaling by default and add GUI feature for configuring it.

MATCH FILES: Fix crash in string variable resizing.

Commmit 638bf380e550 ("combine-files: Allow matching string
variables to have different widths.") introduced a bug in the
case where the variables in data files being combined didn't
have the same variable order.  This is kind of an
embarrassing bug since the input files having the same order
of variables is an exceptional case anyhow.  This commit
should fix the problem.

The test as updated will crash if the bug regresses.

Thanks to Frans Houweling for reporting the bug.

settings: Make PRESERVE, RESTORE, and SHOW handle TLOOK.

Thanks to Frans Houweling for reporting the issue.

psppire-import-textfile: Default to using column names from imported file.

We speculate that users will find this less surprising.

Suggested by Alan Mead.

ONEWAY: Improve handling of missing values in ONEWAY multiple comparisons.

Thanks to Andreas Hammer for the bug report and example data.

pspp-dump-sav: Number variables in cases starting from 1.

The variables are numbered starting from 1 elsewhere in the output, so they
should be in the cases as well.

Bug #64621.
Thanks to Daniel for reporting this bug.

Fixed an assertion failure in psppire-dump-sav when reading a corrupt file.

Closes bug #62980

Simplify drag-drop target settings in CTABLES dialog

html: Include generic font-family name "sans-serif" as a fallback.

https://developer.mozilla.org/en-US/docs/Web/CSS/font-family recommends
that a font-family specification should always include one of the generic
font-family names to serve as a fallback.  For PSPP, sans-serif is a
reasonable fixed fallback.

This addresses bug #63020, which asks for "Sans Serif" to mean sans-serif.
It obviously doesn't generalize to other font name translations, but it
does address the case that the bug filer complained about.  If we need a
more general mechanism, we can invent it later.

Thanks to Frans Houweling for reporting this issue.

Add localisations for ar and ta domains

Remove Find and Close from the list of default button box buttons

Commit e1a2cf577fac791c79cd531e688a48efab9b7c0c inappropiratly added the FIND
and CLOSE buttons to the default list of buttons to be shown.  This change
fixes that error.

Whitespace changes only.

Fix the cleanup of the pivot table in the CTABLES dialog

Properly dispose of psppire_dialog_action objects when the data editor is closed

Fixed a crash when trying to insert variables into the datasheet

Fix cut-and-paste error in comment

Change TAB characters to spaces

Most of our code is indented with four spaces.  However some files had been
using tabs.  Some code used both tabs and spaces in the same file!.  This
change replaces all tabs with eight spaces.

Added a new dialog to generate CTABLES

This dialog provides just a small subset of what ought to be available. In
particular, nested tables cannot be specified and only the default summary
functions work.  Other caveats are also present, but it generally provides
a start and proof of concept.

Update pofiles from translationproject.org

Remove trailing whitespace from files

Expose pivot_category_dump to global scope.

This function is useful for debugging during development of users of
pivot tables.

Warn about MacOS

output_item_unref: Avoid a level of indentation

Add a callback from PsppireDialog on GtkDrawingArea draw signals.

If a PsppireDialog contains a GtkDrawingArea, potentially any change
to the drawn surface could indicate a change in state.  Therefore,
we need to notify interested parties of this change.

Make the page breaking in the TeX driver a little better

Revert recent changes to configure.ac

Commit e1a2cf577fac791c79cd531e688a48efab9b7c0c recented gtksourceview to
version 4.8  - However this turned out to be unnecessary, so this change
reverts that detail.

psppire-output-view: remove set but unused variable

Fix clipboard paste data negotiation

Fixed a bug which caused pasting of output to programs such as libreoffice to
get pasted as images instead of editable tables.

Re-implemented the search dialog for the syntax editor

This new dialog offers a lot more features, such as case insensitivity,
backward searching etc.   This change also removes the option to build
against gtksourceview-3 - gtksourceview-4 is now required.

Implement the SET SUMMARY command

Update copyright dates of documentation

Fix possible divide by zero error

Fix memory leak

Fix minor English grammar errors, mostly "a" versus "an".

I spotted one of these and then found the rest via "grep".

lexer: New function for other kind of half-open range.

The existing function lex_force_num_range_halfopen() handled ranges of the
form [low,high).  There is a new need for (low,high] ranges.  This commit
adds that new function and renames the existing one so that it's obvious
which kind it handles.

dictionary: Properly size string buffer in make_hinted_name().

This causes a test failure on 32-bit platforms, where
INT_BUFSIZE_BOUND(unsigned long) is smaller than F26ADIC_STRLEN_MAX + 1.

Fixes: 2ba8dc2955f8 ("spreadsheet-reader: Merge duplicate code with libpspp/str.h.")
Thanks to Friedrich Beckmann for reporting the problem.

MEANS: Allow string variables to be selected as the categorical variable.

The gui did not allow string variables to be the subject of any MEANS command.
However it is entirely appropriate for them to be the independent (categorical)
variable.

spreadsheet-reader: Merge duplicate code with libpspp/str.h.

The tree had two functions for 26-adic spreadsheet number formatting.  This
eliminates the duplication and moves the corresponding parsing function
into str.h as well.

tests: Fix PostgreSQL test.

Portability/Regression: Check OS version to skip test first (1598)

The test

1598: interactive output appears immediately

does not work on MacOS and the plan is to skip that test. I changed
the testcode to first test the OS and skip immediately instead of
first checking the python shell. Without that the test does not
stop on MacOS and runs forever.

Portability: Variable initialization not at beginning of block

Recent changes in segment.c result in compilation errors on older
compilers like on debian buster or MacOS.

../pspp/src/language/lexer/segment.c:1255:7:
error: a label can only be part of a statement
and a declaration is not a statement
       int eol = at_end_of_line (input, n, eof, 1);

I changed the code such that it compiles on MacOS again to
a more traditional way to avoid the compile error.

ods-reader: Support more than a single piece of text in a cell.

This code read the first text content in the first <p> it found in a cell.
This meant that anything more sophisticated than just <p>text<p> broke,
including <p>text<span>some more</span></p>.  This commit fixes it.

The code here was really complicated and I didn't understand it.  I
simplified it a lot to use xmlTextReaderExpand(), which is a lot easier to
use than a state machine at the level we need it.  I also couldn't figure
out how to make the existing caching logic work quite right, so I replaced
it by something simpler which I think will be just fine.

This updates the test by adding some bold text to a cell.

Thanks to elias tsolis for reporting this bug.
Bug #63443.

spreadsheet-test: Fix usage message.

ods-reader: Integrate xmlTextReaderRead() calls into process_node().

This will allow using xmlTextReaderNext() instead of xmlTextReaderRead()
for some kinds of nodes.

tests: Add some keywords for ODS and gnumeric tests.

gui: Update how the Run menu works in the syntax window.

Until now, the Run menu had both "Selection", which ran the current
selection, and "Current line", which ran the current line.  This commit
replaces that by a single menu item "Selection", which runs the current
command if there is no selection and all the selected or partially selected
commands if there is a selection.  This behavior matches SPSS.

Requested by Matthias Faeth.

command-segmenter: New library for dividing syntax into individual commands.

This is a building block for allowing the GUI to run whole commands instead
of just lines.

deque: Replace deque_init_null() by DEQUE_EMPTY_INITIALIZER.

identifier: Fix typo in comment.

CTABLES: Avoid causing nhtsa.sav to appear modified after running tests.

Adding a hard link makes Git think that nhtsa.sav has been modified, before
it checks for real changes, which is sometimes annoying in the interface.
Using a symlink instead avoids the problem.

CTABLES: Refine treatment for applying a summary to nested variables.

In the previous commit, applying a summary to a nest operator caused an
error, which I assumed was the correct behvior.  This commit changes it
to apply the summary to the inner variable.

Thanks to Matthias Faeth for reporting the correct behavior.

CTABLES: Don't crash for attempting to apply a summary to nested variables.

Thanks to Matthias Faeth for reporting this bug.

CTABLES: Stop parsing after a summary spec error.