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")

Allow /RENAME target to use the TO keyword.

Allow code such as
 SAVE OUTFILE = "foo.sav" /RENAME= (A B C = v1 TO v2).

Closes bug: #57752

main.c: set "register-session" property for gtk_application

To open an application by clicking an associated file like
grid.sps in the Finder on Macos requires that the property
"register-session" is set for gtk_application. I did not
find a negative impact on Debian bullseye so I hope this
works also on other platforms. The gtk support for this
will be in the next gtk-3-24 release.

See: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/2098

desktop file: added Keywords and removed Education category

During debian packaging of 1.2.0-5 the missing Keywords entry
was mentionend. I removed the Education category because this
was mentioned also. Education and Science are both toplevel
categories. I added also two more mimetypes for .sps and .spv
files.

metainfo.xml: changed id to org.fsf.pspp, added more information

I read the documentation about the metainfo.xml content

https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html

and changed/added some information

  - changed id from pspp.desktop to org.fsf.pspp
  - added categories
  - added releases
  - added provides

desktop: changed translations to msgfmt and xgettext

During the build of the debian package 1.2.0-5 it turned out
that the pspp.appdata.xml file has a wrong structure with
respect to the translations. I did the following things

  - renamed pspp.appdata.xml to org.fsf.pspp.metainfo.xml
  - added gettext translation support for org.fsf.metainfo.xml
  - renamed pspp.desktop to org.fsf.pspp.desktop
  - replaced gen-det-desktop.sh based translation with gettext

gettext has now support for translating .desktop and .metainfo.xml
files. I took the naming conventions from the freedesktop and gnome
documentation, i.e. metainfo instead of appdata and the use of
an organization name.

Add a test for bug #58591

psppire-dict.c: Avoid compiler warning

Avoid negative overflow when the text output driver tries to print
random binary data.

libpspp/u8-line.c (u8_line_find_pos): Deal properly with the case
where the target is not found in the search string.

libpspp/u8-line.c (u8_line_reserve): Add some assertions to prevent
negative overflow into lower function calls.

Reported by: Andrea Fioraldi

Fixes bug: #58591

libpspp/str.c (ds_splice_uninit): Add assertion

Fixed a use after free error when manipulating datasets.

Reported by: Andrea Fioraldi

Fixes bug: #58598

spv.c: Avoid warnings about uninitialised variables

find-dialog.c: Replace exp10 with our own integer version

A previous commit introduced the use of exp10 from the standard
math library.  However the use cases here don't need floating
point operands, and it's smaller and faster to implement our
own integer version.

Revert "exp10 replaced with pow when exp10 is not available"

This reverts commit 5f1ab307d380c4d60410b911a272b6870b65c2d8.
See the following commit for explanation.

Fix buffer overflow in lex_ellipsize__

Reported by: Andrea Fioraldi

Fixes bug: #58586