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.

Fix memory leak in the value labels dialog

src/ui/gui/org.fsf.pspp.desktop.in: Add translator hint

Add an instruction to not translate the Icon name.   Recent versions of
xgettext (correctly) ignore this entry.  But earlier versions erroneously
include it in the list of strings to translate.

psppire-data-window.c: Remove ssw api protection

SpreadLevel Plot: Don't try to take the logarithm of a negative

src/output/cairo-chart.h (xrchart_write_scale): Check the range and deal with the consequences.

If a chart was requested where either scale had an range of zero
(the minimum value and the maximum value were equal), then all
sorts of memory errors could occur later in the chart's rendering.

This change checks for this condition early and refuses to draw the
chart in this case.

psppire-data-window.c (on_cut): Do not append trailing whitespace to clip.

psppire-data-window.c (on_cut): Deal properly with string variables.

Do not attempt to use the output driver while it's being destroyed.

src/ui/gui/automake.mk: Remove OBSOLETE

This definition is indeed obsolete.

#undef close which was causing problems on w32

Revert "Conditional Compilation per GCS 3.5"

This reverts commit 002a30f274ced97769ebc3373c493a8e5329c2bc.

Revert "Keep systems happy which do not have sys/resource.h"

This reverts commit f4ba0d4b24301f389dfb76a66094b093125fac9b.

Histogram Dialog: Allow only numeric variables to be selected

Reduce the length of the icon names.

This was causing problems for the autobuilder.

po/metainfo.its: New file.

Add a ITS file for metainfo.xml format, and use it when extracting strings for
for translation.

gui: cut from editable to clipboard when cell is focused

When a cell is focused, then the cut operation
must operate on the selection in the cell editable
and not on the complete cell, i.e. the store content.

gui: trim displayed string for string variables (spaces)

The current gui always renders strings in the datasheet view
with the full width by adding spaces to the end of a string.
This patch renders the strings in the same way as they are rendered
in the status edit box above the cells array, i.e. trailing whitespace
is removed. Editing strings is straightforward.

gui: render empty string for numeric system missing value

Numeric cells which contain the system missing value are currently rendered
as a number of spaces, a dot and a number of spaces depending on the
current variable width and decimal places settings. When the cell is
edited, then this string is pretty confusing.
With the patch system missing values for numeric variables are rendered
as empty strings. Editing the cell is then straightforward and starts
with an empty cell.

Keep systems happy which do not have sys/resource.h

configure.ac: Select ustar format for distribution.

This format allows for longer filenames, which is needed for some of the
long icon names used by PSPPIRE.

Document /*...*/ style comments.

icons: added psppire- prefix to action items #58708, #50026

The edit-find action icon is already available in the default
gnome and kde icon themes. Therefore the default icon is always found
first. I added the psppire- prefix to the action icon names such that
the icons are unique.

Closes: https://savannah.gnu.org/bugs/?58708
Closes: https://savannah.gnu.org/bugs/?50026

Remove pspp-mode.el

This is now distributed at elpa.gnu.org/packages/pspp-mode.html

HTML output driver: Correct typo

VALIGN was incorrectly typed as ALIGN

Autorecode Dialog: Use correct dictionary pointer

psppire_box_pack_start_defaults: Change to static inline

This is a simple, one line wrapper around another function.
So having it inline makes sense.

src/ui/gui/helper.c: Remove unused functions

Remove the default help function.

This function is no longer reachable.

Audit and cleanup dispose methods of classes which have them.

Ensure that all non-trivial dispose methods use the dispose_has_run
flag and that only unreffing takes place within dispose.

Mitigate bug #48050

This bug caused an assertion failure in src/data/datasheet.c  The change here
traps the faulty condition earlier and raises a G_CRITICAL rather than an
assertion.

Fix memory leak displaying string values in the data sheet

Value variant: Don't over allocate string values

pspp-mode.el: Make all keywords case insensitive

pspp-mode.el: Make indentation closer to what lisp people like.

Add finalize methods to some classes which don't have them.

These may not be strictly necessary, but can make tracking
for leaks easier by placing a diagnostic in the method.

Unref variables after fetching from treeview model.

Commit e511c5a34a8dd5863c5f71c820609c3c0e363cc9 added a reference
to boxed variable pointers.   This means they have to be unreffed
after fetching one with gtk_tree_model_get.

gtk.in.h: Remove redundant macro replacements.

Today configure.ac insists upon Gtk+ 3.22 so working around older
Gtk+ versions is no longer required.

lib/ptukey.c: Use the standard macro INFINITY from the math library

Change all instances of "help_page" to "help-page".

Signal and property names are documented as treating
'-' and '_' as equivalent.  However it helps when grepping
files if we use just one orthography.

Conditional Compilation per GCS 3.5

Deal with conditional compilation using the way recommended by
the GNU Coding Standards Section 3.5

https://www.gnu.org/prep/standards/html_node/Conditional-Compilation.html#Conditional-Compilation

Conform to GNU nomenclature.

Fix memory leak introduced in commit e2039bc0694b89d26065899d9ed20f9bb42426f4

Follow-up to bugfix #58664

If the width of a variable changes, then implicitly so does its
missing values --- even if the number of missing values is zero.
This is because the variable's width is cached in the missing_value
struct itself.

Fix bug #58664

trim.c: avoid zero size xnrealloc memory allocation

Without the patch the regression fails on test

GET with /KEEP=ALL crashes -- uncompressed

when I compile with -fsanitize=address on MacOS. The reason
is that xnrealloc is called with a zero size allocation. I
simply avoid this, although it might be perfectly o.k. to do so.

desktop: open a file via xdg-open without additional output window

Currently there is a difference between opening an additional data
file via the Edit->Open menu or by opening a file via xdg-open (same
for doubleclick on Filebrowser). In the latter case the on_open
callback is called which runs an additional post_initialize. That
results in an additional output window.

I changed the behavior such that post_initialize is only called
when there is no psppire window yet. That happens when psppire
is started from a terminal with the file as argument on linux.
Further I check if there is an empty datawindow and replace that
in the same way as it is done via the Edit->Open menu.

Fix bug #54784 (again).

Certain GUI entities hold pointers to variables.  To ensure that
the variables survive the lifetime of the entity which holds the
pointer, the entity must take a reference.

This is particularly important where a transformation which involves
TEMPORARY of a filter is involved.

Introduce reference counting for variables.

Necessary for upcomming commit.

psppire-output-view.c (psppire_output_view_destroy): Fix memory leak.

dictionary.c: Don't emit callbacks when the dictionary is destroyed.

Also reorder some functions in order of dependence.

data_parser:  Keep a reference to the dictionary

Revert "Fixed a use after free error when manipulating datasets."

This reverts commit 1d06ba6d498b9446d5fb8c432d24efb2bee713a9
which caused memory leaks.

typo in title - Lable -> Label

switched to portable control key <Primary> for accellerators

gtk supports the "Primary" key which translates to the "CMD" key
on MacOS and "Ctrl" on linux. So this is a portable way to
say the "Copy" via key.