Make distcheck pass

Integrated the application icons

Added names of new artwork contributors to about dialog

Added mimetype icons

Added some icons to represent pspp file types in file managers etc.

Use "auto" mode for parsing syntax in the GUI (and elsewhere).

This should be less surprising, I hope, than always parsing syntax in
"interactive" mode.

Reported by Ronald Crichton <Ronald.Crichton@cit.edu.au>.

Change "variable_changed" to "variable-changed" in signal name

This makes it consistent with usage elsewhere.

po: Update Dutch translation from translationproject.org.

Fix crash when changing variable formats in data import dialog.

Recent changes altered the CHANGE_VARIABLE signal handler signature,
but this handler had not been updated.

Reported-by: dr soumalya ray <drsoumalya-ray@yahoo.co.in>

Update version number to 0.8.0.

Updated localisations from translation-project.org

REGRESSION: Fix issues found by by review

Ben reported following potential problems:

I think that if the DEPENDENT subcommand is given twice (a user error), then
the variables specified the first time will be silently leaked.

cmd_regression() has two local variables named 'i'.

I don't understand why "struct workspace" has a member 'psw' that is a
dynamically allocated array, because it looks like each element of the array
is used only during processing a single split file group.  That is, I think
that 'psw' could be just a singleton, instead of an array.

I think that fill_all_vars() is wrong: if an absent var follows a not-absent
var, then vars[] will be filled with a gap and there will be a write into
unallocated memory.

Reported-by: Ben Pfaff
Avoids shadowing variable in lower scope.  suggested by Ben Pfaff

configure.ac: Increase version number to 0.7.11.

ONEWAY: Fix memory leak in run_oneway.

Bug #39070.

tower-test: Use memmove(), not memcpy(), for overlapping memory move.

Found by valgrind.

sparse-xarray: Fix memory leak in dump_sparse_xarray_to_disk().

Bug #39070.

model-checker: Fix memory leak in finish_mc().

This memory leak was visible only when the model checker was configured
with a particular path to follow.

constness

Fixed incorrect behaviour of REGRESSION when multiple dependent variables are entered

The REGRESSION command behaved badly when more than one dependent variable was entered.
The cause was that the unnormalised covariance matrix returned from the covariance
module was mutated for each variable.  Consequently, each variable after the first
was wrong.  This change fixes that by changing the ownership semantics of the returned
matrix (and thereby its constness).

This change also adds some comments, attends to constness and fixes the remaining
memory leaks associated with regression.

Fix problems associated with LINEAR REGRESSION and splits

Added test to check that REGRESSION works with multiple dependent variables

Added a test to ensure that SPLITS vs REGRESSION/SAVE works ok

Rename tests from REGRESSION to LINEAR REGRESSION

This way I can run just the linear regression tests without the
logistic regression tests.

Avoid unexpected failures in check-valgrind due to XTERM_LOCALE

Partial fix for regression vs. splits issue

Split the struct regression into information which is
parsed from the command, and data which is calculated during
execution of the command.

Bug #39070

linreg.c: Remove superfluous cast

range-tower: Fix memory leaks.

Bug #39070.

CROSSTABS: Fix memory leaks.

Bug #39070.

linreg: Remove some unused features

session: Fix two memory leaks.

Bug #39070.

LOGISTIC REGRESSION: Fix memory leak in error path

datasheet: Avoid reading a given row more than necessary in source_read().

The datasheet casereader uses datasheet_get_row(), which calls rw_case(),
which until this commit called source_read() for every column.  The latter
was expensive if casereader_peek() actually copied data for a whole large
row.  For a datasheet with N columns, all from a single backing source,
the upshot was that datasheet_get_row() took O(N**2) time.

This commit fixes the problem in the common case by reading as many columns
as possible from a given source in a single call to source_read().

Bug #33260.

variable.c: New function var_set_width_and_formats

Added a new function to set the width and formats of a variable as
an atomic operation. This is necessary to allow values to re-interpreted
without going through an intermediate format and thereby perhaps loosing
information.

Closes bug #39252

variable.c: (var_set_width)  traits other than width may also be set

Split VAR_TRAIT_FORMAT into PRINT and WRITE variants

Fix and simplify the code for converting variables between types

Before this commit, changing a variable from string to numeric or
vici-versa would cause all values of that variable to become SYSMIS.
This commit fixes this problem at least partially.

Partial fix for bug #39252

Fix constness of datasheet_resize_column arguments

The aux variable should not be changed, so make it const

dictionary.c: Remove the var_resized and var_display_width_changed callbacks

Remove the resized and display width changed callbacks, and
transfer their actions to the variable_changed callback.

Changed the signature of the VARIABLE_CHANGED signal to take the WHAT and OLDVAR parameters

Re-implement signal handlers of the variable-changed signal to PsppireDict to take
the new WHAT and OLDVAR parameters.

Added the WHAT and OLDVAR parameters to the var_changed dictionary callback

Propagate the newly implemented WHAT and OLDVAR parameters in the PsppireDict
class.

dictionary.c: Added a oldvar parameter to the var_changed callback

The var_changed callback needs a copy of the old (unchanged) variable,
so that implementations can compare and act accordingly.

dictionary.c: Added a WHAT argument to dict_var_changed callback

Defined some bitwise constants to indicate which aspect of a variable has
changed.  Added an argument to the dict_var_changed callback taking an
argument which contains a combination of these types.

This is used in later commits to avoid the callback implementation having
to guess what exactly changed.

variable.c: Add _quiet versions of the var_set_ methods and use in var_clone

var_clone doesn't need to provoke callbacks, since the new variable does not
(yet) have a dictionary  hence the callbacks can do nothing.  Also the callbacks
cause problems for later commits, since they themselves could call var_clone.

This change therefore defines _quiet versions of all the var_set methods, which
do not generate any callbacks, and uses those versions in the implementation
of var_clone.

Minor additions and updates to the meta documentation

Remove generation of GSignal marshallers which are  no longer required

psppire_var_type_dialog: Remove global functions which are not used

INPUT PROGRAM: Use a separate dataset for the input program.

Without this change, INPUT PROGRAM uses the same dataset as the following
syntax.  This usually is fine, but there's at least one case where it
doesn't work: if the INPUT PROGRAM defines a vector, then a following
proc_execute() without any intervening transformations causes
proc_execute() to follow a "fast path" where it skips running the procedure
but instead just does a few things, like clearing the set of vectors.
Unfortunately, this means that later when the input program is really run,
  it doesn't have the vectors and accesses freed memory.

This commit switches to a different tactic.  INPUT PROGRAM creates a
special session and dataset that are used only for the input program.  This
keeps the dataset and the transformations used for the input program
separate from those used after the input program.

Bug #39097.
Reported by John Darrington.

command.def: Disallow DATASET commands inside INPUT PROGRAM, FILE TYPE.

It doesn't make sense to try to switch datasets within INPUT PROGRAM, and
so we shouldn't allow it.

Correct typo in documentation of SAVE TRANSLATE

Text Import Dialog: Fix initialisation of pages which had got broken

manual: REGRESSION and DESCRIPTIVES make temp transformations permanent.

Ideally, this should not be the behavior, but it is better than the crash
we had before.

Suggested by John Darrington.

Bug #38787.
Bug #38786.

INPUT PROGRAM: Avoid infinite loop for trivially empty input program.

Bug #38782.
Reported by John Darrington.

Avoid letting data fields overflow in output and GUI.

When the output for data_out() is too big for the field width, it produces
output that consists of just asterisks: ******.  This is OK when the
output is really going into a fixed-width space, such as the output for
PRINT and WRITE.  But it is obnoxious if the output is going into PSPP
output or the GUI.  This commit introduces a new function that typically
does not do this, and starts using it in output and the GUI.

Bug #35829.
Reported by John Darrington.
Bug #30731.
Reported by lavila <la@jump23.com>.

Whitespace changes only

Fix memory leak in error path of RECODE

Fix memory leak in error path of LOGISTIC REGRESSION

Fix memory leak in LOGISTIC command

Fixed memory leak in examine command

Fix leak check in oneway command

Remove "slash" property from psppire_button_editable, which is not used

Linear Regression Test: Provide more decimal places in test results.

This test previously had only 2 decimal places displayed, which
limited its ability to detect numerical errors.  This change increases
the decimal places to 3.

The results for this test were checked against a pattern provided
by Olaf Nöhring.

Regression: Use alternative formula for adjusted R-square

This is the formula given in SPSS Statistical Algorithms Ed 20.

Corrected the Degrees of Freedom in the Regression Coefficient Significance

Reported and Tested by: Olaf Nöhring.

Open imported files in new data window instead of overwriting the old

Add hint paramater to Window Load method

Smake: Use glibtoolize instead of libtoolize, if it is available.

Requested by Teppo Jussmäki <jussmaki@fastmail.fm>.
Tested by Teppo Jussmäki <jussmaki@fastmail.fm>.

DELETE VARIABLES: Fix crash with FILTER.

FILTER has the surprising property that it introduces a temporary
transformation, even if proc_make_temporary_transformations_permanent()
was previously called.  This causes a crash if the procedure implementation
really relies on there being no temporary transformations.

This commit fixes the problem in the procedures I was able to identify as
having it, adds a comment to proc_make_temporary_transformations_permanent()
to make it a little less likely to happen again, and adds a test
specifically to check for regression on DELETE VARIABLES.

Reported by John Darrington.
Bug #38843.

REGRESSION: Avoid use-after-free error with TEMPORARY and SAVE.

This should fix the use-after-free error, although I am not sure whether it
makes REGRESSION do exactly what it should do.

Bug #38787.

DESCRIPTIVES: Avoid use-after-free with TEMPORARY and Z scores.

This is not an ideal fix, but it avoids the use-after-free error that we
have had until now.

Bug #38786.

FLIP: Fix use-after-free and double frees with temporary transformations.

I originally thought this was a bug in FLIP, but it turned out that in
fact we just hadn't ever really tested
proc_make_temporary_transformations_permanent() properly with actual
transformations, and especially with adding a transformation after calling
proc_make_temporary_transformations_permanent().

The modified test should avoid regression for the bug.

Bug #38832.

DESCRIPTIVES: Fix a pair of memory leaks.

Work around change in signature of the switch page callback in GtkNoteBook

Fix constness problem in spreadsheet reader dialog

version.c: Make the build date/time independent of locale and timezone

Remove unused function

Implement import dialog to import spreadsheet files.

Reviewed-by: Ben Pfaff

Sort items in datasheets dropdown box.

Before this change, the values in the data sheets dropdown
box appeared in a non-deterministic order (the order in
the underlying hash), which was inconvenient to users.
This change produces the values in sorted order.

Closes bug #38921

Documentation: Explain more emphatically that UPDATE/MATCH/ADD do not mutate input files.

Closes bug #38682.

Cairo output: Add a bit of horizontal between table columns.

Fixes bug #34798

Spreadsheet Import Dialog: Implement reset button

Merge remote branch 'origin/master' into import-gui

Conflicts:
src/ui/gui/page-assistant.c

Import Assistant: Fix broken reset buttons

Import Assistant: Fix broken reset buttons

Merge remote branch 'origin/master' into import-gui

Avoid use of deprecated GtkNotebookPage

PsppireButtonBox: Do not use the GtkBoxChild structure.

Avoid using Gtk features which are (soon to be) deprecated.

Merge remote branch 'origin/master' into import-gui

PsppSheetViewColumn: Remove inappropriate #ifndef

Replaced some obsolete GTK symbols with the recommended modern ones

GtkXPaned: Whitespace changes only

Avoid compiler warning

Update INSTALL file

Updated localisations from translation-project.org

Merge branch 'master' into import-gui

Made _xml_char_to_int less prone to crashing

Gnumeric Reader: Use symbolic value instead of literal integer

Made _xml_char_to_int less prone to crashing

Gnumeric Reader: Use symbolic value instead of literal integer

Merge remote branch 'origin/master' into import-gui

Fix corner case in calculation of trimmed mean.

The calculation of the trimmed mean was not quite correct.  This
change fixes it, and adds a test.

Reported by: Olaf Nöhring.

Merge branch 'master' into import-gui

Conflicts:
src/ui/gui/psppire-cell-renderer-button.c