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.

Revert "Fix crash in gui, when T-TEST or ONEWAY while a filter is set."

This reverts commit d01a4986ffc8f5afa5ee7d29dd7233143b6de7ec
which caused memory leaks.

Improve behaviour when importing trivial csv files.

This is a follow up to commit 5312cd16f3d7226d4e62a847d64849c57d4b4843
Don't activate any check button, if there is no obvious candidate.

Add a test for bug #58598

src/dictionary.c: Move dict_ref next to dict_unref

This make it easier for me to hack when I'm tracking
down memory (de)allocation errors.

PsppireDataWindow: Fix use after free error

PSPPIRE: Fix memory leaks in dialogs

PSPPIRE: fix leaks of GtkAccelGroup

Replace some calls to xcalloc with XCALLOC

src/ui/gui/windows-menu.c (repopulate_windows_menu): Avoid memory leak

Fix memory leak in src/output/table.c

PSPPIRE: Fix memory leaks when creating windows

PSPPIRE: Fix crash when cleaning after bad text import.

The wrong pointer was being freed after cleaning up when
attempting to import text files with very long lines.

PSPPIRE: Avoid memory leaks from popup memnus

GUI: Fix memory leak on startup

psppire-import: fixed bug with short csv file import

The import crashed with a short csv file.
Closes: https://savannah.gnu.org/bugs/?58644

pspp-mode.el: update elpa autoload comment

The pspp-mode is not automagically active in the 1.2.0-5 debian
release which install the emacs pspp mode. The reason was this
wrong elpa autoload comment.

de.po: Fixed wrong translation

added title to variable sheet dialogs

help: added help page info to variable sheet dialogs

help: removed yelp support

The yelp tool is broken for three years and cannot show uris with
a tag. John wrote a bug report three years ago but it is not fixed
yet.

https://gitlab.gnome.org/GNOME/yelp/-/issues/116

Once yelp works again, we can revert this commit.

automake.mk - added 96x96 size

artwork: categories icons moved according to new strategy

artwork: actions icons according to new strategy

artwork: apps - pspp.svg - removed all but scalable version from artwork

icons: create scaled bitmaps from one svg source

Using file-svg-load in gimp allows to scale an svg to
an arbitrary size. The artwork directory only needs
to contain svg files which differ in layout.

gui: added help in recode.ui subdialog

metainfo.xml - modified after validation

desktop: add mimetypes application/x-spss-sps and application/x-spss-spv

I added an xml mime package which relates the files .sps and .spv
to a mime type. Via that mime type the files can then be openend
by clicking in the file browser.

desktop: reversed mimetype to application/x-spss-sps

If the mimetype is of type text, then the icons are not found
or displayed in gnome Files. No application except apt seems
to define text based mime types. And for apt it does not work
either. So I reverse.

Fixed some calls to xcalloc where the quantity and size parameters had been transposed.

recode dialog: set "old and new values" sentivitity as appropriate.

The "Old and New Values" button should only be clickable when a row
is selected in the corresponding treeview.  This was not the
case.

Fixes bug: #58557

matrix-reader: Fix possibly-zero second argument to xcalloc().

xcalloc() is documented to disallow zero for its second argument:

  /* Allocate zeroed memory for N elements of S bytes, with error
     checking.  S must be nonzero.  */
  void *
  xcalloc (size_t n, size_t s)

This code didn't always call it properly.

Found with AFL++.
Thanks to Andrea Fioraldi for reporting the problem.
Bug #58600.

build-aux: Ignore more files.

These entries were added automatically by running Smake.

sys-file-reader: Tolerate duplicate document records.

sys-file-reader: Tolerate invalid variable names.

This should make PSPP better at reading system files that have encoding
issues.

sys-file-reader: Accept bad variable indexes, with warnings.

The SPSS file writer software at
https://packagist.org/packages/tiamo/spss sometimes generates malformed
variable indexes for weighting variables or value label records.  PSPP
until now has rejected these files entirely.  This commit should make
it issue a warning instead.

Thanks to István Ignácz for reporting this issue at
https://lists.gnu.org/archive/html/pspp-users/2020-06/msg00012.html

gui: Fix compatibility with older gettext.

Older gettext does not recognize the metainfo.xml extension, only
appdata.xml.  This commit allows msgfmt to work with such older setups.

CC: Friedrich Beckmann <friedrich.beckmann@gmx.de>
Fixes: 86c5d611b3f2 ("metainfo.xml: changed id to org.fsf.pspp, added more information")