po/automake.mk: Fixed typo in variable names.

This resulted in po/pspp.pot not getting updated when it should have been.
This is a very good demonstration of the reason why carefully desigined
languages should insist on a variable being declared before it is used.

Also make the file depend on Makefile, so that it gets rebuilt when the
PACKAGE VERSION changes.

Deal properly with EOF in the REPL, when built without readline

utitilites/pspp-output.c: Deal properly with plural forms

updated german translation de.po

doc: Describe running the "rc" file at pspp startup.

Bug #54376.
Thanks to Harry Thijssen for reporting this bug.

doc/manual: Remove gitignore file in otherwise empty directory.

I added this by mistake some time ago.  It's not useful.

tests: Add missing host.at tests.

Thanks to Friedrich Beckmann for reporting the problem.

encoding-guesser: Explain UTF-16, UTF-16BE, UTF-16LE distinctions.

u8-istream: Add some more tests for byte-order marks.

make-file: Pass O_BINARY or O_TEXT to open() call in replace_file_start().

It's hard to imagine why this would make a difference on a real system,
because the file descriptor obtained from open() is passed to fdopen(),
which is documented to determine whether the stream is text or binary
based on the mode string.  However, bug #40223 implies that it makes a
difference.  I hope that this commit fixes the problem (if any!).

Thanks to the anonymous submitter of bug #40223.

csv-file-writer: Fix typo in user message.

HOST: Reimplement and add support for TIMELIMIT subcommand.

Fixes bug #25816.
Thanks to John Darrington for reporting this bug.

tests: Add 'slow' tag to more tests.

This reduces the runtime of "make check TESTSUITEFLAGS='-k \!slow -j128'"
from 4.4 to 3.5 seconds on my computer.

spv-light-decoder: Avoid GCC 10.x warning in decode_spvlb_valign().

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

value: Suppress uninitialized tmp.s warning in value_resize().

This warning is wrong, but it should not cause a performance problem to
avoid it.  Encountered with GCC 10.x.

Makefile.am: Suppress GCC 10.x warning in gl/careadlinkat.c.

This suppresses an error in gl/careadlinkat.c that otherwise can't be
avoided in GCC 10.x:
  ../../gl/careadlinkat.c: In function ‘careadlinkat’:
  cc1: error: function may return address of local variable

GFUNC_COMPAT_CAST also for dialog-common.c

I missed that one - Thanks!

Work around bug in Automake 1.16.1

moved GFunc cast to macro GFUNC_COMPAT_CAST

I moved the GFunc cast for g_list_foreach to a macro with
some explaining comment. The macro is in helper.h

psppire-spread-sheetmodel.c: modified userdata increment using GPOINTER_TO_INT

po/pl.po - replaced strange character in comment

This results in sed on MacOS to report an error.

dummy.c: Conform to exact prototypes

Apparently certain toolchains complain if the symbols in dummy.c do not
exactly match their declarations.  This should not be a problem unless
the compiler has enabled a lot of warnings, and is treating those warnings
as errors, (which is inadvisable except under very special circumstances).
However, it appears that some people are doing that.

This change ensures that they are identical.

Prefer GTK+ style casts rather than C-style casts.

Replace '(GObjectClass*) class' with 'G_OBJECT_CLASS(class)'

Remove _base_init functions from custom widget definitions.

These served no purpose, and are in Gtk+ only for reasons of
legacy so far as I can make out.   Also use G_TYPE_DEFINE macros
instead of explicit *_get_type function definitions.

INSTALL: Remove reference to --with-gui-tools which no longer exists

switched off unused-paramter warning for lib-essential gui files

I disabled the unused-parameter warning as for the other
gui files. Same as before the introduction of the essential
library.

src/ui/gui/automake.mk: Add missing AM_CFLAGS parameter

Warnings: correct cast for psppire-spread-sheet model

This fixes the compiler warning:
warning: wrong type argument to increment [-Wpointer-arith]

Remove src/ui/gui/memorandum.txt

Almost all of the information in this file is out of date.

Add convenience library libwidgets-essential

This avoids having to compile large numbers of modules twice.

xml-parser-generator - iterate over sorted dict - #58975

I sort the dict before iteration because the normal
iterator over a dict is not reproducible and will generate
different code in each run. This will work but the
build is not reproducible.

Closes: https://savannah.gnu.org/bugs/index.php?58975

src/ui/gui/psppire-conf.c: Use GError instead of strerror.

Apparently the use of strerror was causing problems on certain
platforms.  In this case, since the function being checked provides
a GError argument it makes sense to use that instead.

Closes bug #59034

replace finite with isfinite

finite function is deprecated on MacOS.

Warning: added missing initializer for closure_callback in GSourceFuncs

Warning: "static const" instead of "const static"

Remove warning:
 ‘static’ is not at beginning of declaration

Warning: avoid warning that weekday may be used uninitialized

Warning: adhere to const qualifier for initializations and assignments

This commit avoids the warnings:
warning: initialization discards ‘const’ qualifier from pointer target type
warning: assignment discards ‘const’ qualifier from pointer target type

Warnings: fix missing function prototypes

In most cases I added the static qualifier which was
missing. Some functions are never called so I either
removed them or I put them in #if DEBUG

Warnings: removed fallthrough in case statements

I guess the fallthroughs in the case statements were not intentional.
This avoids the warning:
warning: this statement may fall through

Warnings: added cast for enumaration types

This avoids the warning:
warning: implicit conversion from ‘enum <anonymous>’ to ...

Added tooltip for statistics in dialogs

This removed the warning:

warning: missing initializer for field ‘tooltip’

Warnings: function type cast for g_list_foreach

I added a cast for the functions used in g_list_foreach. Without
the cast, the warning
warning: cast between incompatible function types
is reported.

Warnings: missing initializer for value_tables and function type cast (GObject)

I fixed the warnings:
warning: missing initializer for field ‘value_table’ of ‘GTypeInfo’
warning: cast between incompatible function types
by replacing the handwritten type info code with G_DEFINE_TYPE...
macros (preferred). When this was not possible due to usage of
base init functions I added casts and the initializer.

Warnings: add AM_CFLAGS also for the psppire build

With the new setup the psppire and glade build honours the
AM_CFLAGS setup. This includes the Wextra warnings which
show some more warnings. The two checks which are disabled are

-Wno-sign-compare     : disabled also for core pspp
-Wno-unused-parameter : many callbacks result in unused parameters

Fixed bug which caused a crash when using the Find feature

src/ui/gui/dummy.c: Eliminate warnings when building with gcc-10.2.0

Updated translations from translationproject.org

src/ui/gui/automake.mk: Remove yelp-check rule

psppire-marshaller.c: Include the header file

html.c: Reimplement as HTML5

get_language: New function

Issue error message if failing to change directory

barchart.c: Sort the categories before displaying them.

Partial fix for bug #58968

limit the number of variables in barchart from crosstabs #58968

The barchart was created with more than two variables from
the crosstabs subcommand. I added an assertion and limit
the number of variables to two. This bug resulted in a
regression failure (crash) for test

493: CROSSTABS barchart

on i386 architecture.

see: https://savannah.gnu.org/bugs/?58968

fixed range check in parse_record_placement (32Bit/64Bit issue) #58968

The regression failed for test "lexer crash due to overflow" (403)
on i386 architecture. The reason is that long is 64Bit on amd64 while
it is 32Bit on i386. The code used long as type that can hold larger
values than int which does not work on 32Bit systems.

The change keeps the value as double as long as the range checks are done.

see: https://savannah.gnu.org/bugs/?58968

fixed git distribution check for .gitattributes added in previous commit

added .gitattributes file such that lexer.at shows diff

factor.c: added translation hint

Removed stray newline in acinclude.m4

Reported-by: Michel Boaventura

Fixed error running 'make distcheck'

Fixed some errors in format and orthography of printed strings.

Reported-by: Anonymous https://lists.gnu.org/archive/html/bug-gnu-pspp/2020-08/msg00004.html

pspp-convert.1 - fixed a syntax typo

The debian build checks reported a bug in the man page:

troff: <standard input>:129: '\-' is not allowed in an escape name

The man page can be checked with:

LC_ALL=en_US.UTF-8 MANROFFSEQ='' MANWIDTH=80 \
man --warnings -E UTF-8 -l -Tutf8 -Z ./pspp-convert.1 >/dev/null

Revive psppire's glade compatibility

PsppireSelector: Don't attempt to unref a null pointer

Update GtkBuilder files to mention gtk+ version 3.22

src/language/stats/chart-category.h: Add #include directive

PsppireVarView: Use g_realloc instead of xrealloc

src/ui/gui/automake.mk: Sort the manifest

Rename PsppireButtonbox to PsppireButtonBox

This is more consistent with other names.

src/ui/gui/psppire-conf.c: On error, give a reason for failure to create file

Avoid warnings about ! being passed a non-boolean

PsppireImportAssistant: Proceed to next page when a file is double clicked

Remove/Replace some obsolete gnulib modules

Smake: create build-aux/config.rpath

Update to more recent GNULIB

Bump version to 1.5.0

Release PSPP version 1.4.0.

sys-file-reader: get_text_token did not set delimiter - bug #58943

parse_mrsets used loop local variables which were not initialized.
The regression worked with low compiler optimization levels because
the value for "delimiter" was kept over the loop iterations.
get_text_token did not set the delimiter value when only delimiters
are left in the string. I set the delimiter value to the last value
in the string in that case. I initialized the variables to see the
bug also in -O0.

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

render-test: Initialize i18n to make tests pass on mingw.

On mingw, filename recoding requires i18n to be initialized.

tests: Tolerate 3-digit exponents in output for chart tick format.

This makes the test pass on mingw.

tests: Ignore Pango warnings when generating charts.

On mingw, Pango complains about font names.  It's harmless.  I don't know
a way to suppress the warning.

placement-parser: Properly handle LONG_MAX != INT_MAX.

Also, delete spurious null bytes from the related test.  The null
bytes spurred a test failure on mingw although I am not sure why.

tests: More character encoding issues on command line.

Unix-like systems don't have a command-line encoding, so we can pass
whatever bytes we want.  mingw uses some particular encoding for
command line arguments, so this screws up trying to pass UTF-8 in.  This
commit side-steps the problem by passing only ASCII characters on the
command line and making the test program decode the bytes.

This fixes several test failures under mingw.

tests: Use AUTOTEST_PATH to find chart test programs.

Otherwise the test wrappers feature can't help to find it on mingw, so
the chart geometry tests fail.

zip-test: Open files in binary mode.

Fixes a test failure on mingw.

tests: Avoid dealing with character encoding issues on command line.

Unix-like systems don't have a command-line encoding, so we can pass
whatever bytes we want.  mingw uses some particular encoding for
command line arguments, so this screws up trying to pass UTF-8 in.  This
commit side-steps the problem by passing only ASCII characters on the
command line and making the test program decode the bytes.

This fixes several test failures under mingw.

tests: Tolerate carriage-returns in output for SAMPLE test.

This fixes a test failure with mingw.

tests: Always set a paper size.

Normally PSPP can find a default paper size but this isn't necessarily true
inside, e.g., a build container, so set a default to avoid test failures.

tests: Don't output to PDF without a good reason.

PDF output can cause spurious error messages from Pango to appear in the
output (which would be nice to fix but I'm not sure how) and it's not
always configured in, so omit it in MRSETS where it's not really needed.

tests: Fix synthetic system file reader tests on Windows.

The system files were being written as text files, so any \n character was
getting translated into \r\n, corrupting the output.

Smake: added 16x16, 32x32 and 256x256 pspp.png icons

Recent spread-sheet-widget to version 0.6

Fix typo in po/automake.mk

Recent spread-sheet-widget to 0.5 or later

Import Assistant Sheet Page: Set the sheet selector sentitivity

gui: normalize clipboard cut selection rectangle (#58756)

With this patch the selection rectangle for the clipboard cut
operation is normalize, i.e. it does not matter in which corner
you start the selection.

Closes: https://savannah.gnu.org/bugs/index.php?58756

gui: clipboard CUT and COPY store same data with/without label

clipboard store data with or without value labels depending on
the setting in the data view. clipboard cut was always storing
the raw data.
With this patch copy and cut both store raw data or labels
depending on the view setting.

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

gui: Pasting also considers labels as valid data

With this patch pasting from clipboard also interprets labels as
valid data. First it is checked if the pasted data is a label for
the destination variable. Then the corresponding data is stored.

psppire-import-assistant (struct _PsppireImportAssistant): Remove unused member.