GtkXPaned: Avoid direct inclusion of gtk library header

gtk.h wrapper: Remove workarounds which are no longer required.

Disable the remaining Gtk+2 features.

These do not seem easy to work around.  But none of them are
essential.  Postponing them till after the transition.

PsppSheetViewColumn reparented to GObject

Previously the parent was GtkObject which no longer exists in Gtk3

More sealage

Sealage old_width

Rename allocation -> col_allocation

More sealage

Get window

Use gtk_adjustment methods instead of direct access

Yet more sealage

Temporarily disable the splash screen

Yet more Gseal stuff

GtkSelectionData: only access using functions

Avoid yet more sealed access

PsppireDataWindow : Avoid direct access to sealed widget members

Independent Samples dialog: Avoid direct access to sealed widget members

Missing value dialog: Avoid access to sealed Gtk members

Avoid direct access to sealed Gtk Widget members.

Text import dialog : Avoid deprecated Gtk functions

Replace GSEAL in widget headers by a different symbol

We don't want our own data members to be sealed by Gtk

Disable code which is not used and is Gtk3 unfriendly

Remove grid_line_gc

Avoid use of gdk_draw_line

Avoid use of deprecated GTK_SET_FLAGS

Based upon a patch from the Gtk+ project:

From: Emmanuele Bassi <ebassi@linux.intel.com>
Date: Mon, 12 Apr 2010 14:54:41 +0100
Subject: [PATCH] tree-view: Use gtk_widget_send_focus_change()

Remove the second (and last) internal user of the GTK_HAS_FOCUS flag.

https://bugzilla.gnome.org/show_bug.cgi?id=593671

Avoid use of deprecated gdk_drawable_get_size

Use gdk_window_get_{width,height} instead.  Some calls were completely unnecessary, so those
have simply been deleted.

Avoid use of deprecated gdk_drawable_get_screen

Avoid use of deprecated gdk_drawable_get_display

Avoid gdk_drawable_get_size which is now deprecated

Avoid problems with G_CONST_RETURN

This is no longer used by us.  But may be used by other 3rd party libraries.

Avoid use of deprecated GtkComboBoxEntry

Recent GTK version to 2.24

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

Revert "Work-around bug in GCC 4.4.5"

This reverts commit c7711e39f523f206a74f5d0f52480ad10267e0ca.
The claim of a bug in GCC proved to be unfounded.  The problem which
this change tried to work-around has in fact been fixed in
commit 19bf192a093527bf10b1de95fbe8798ba156f960

Fix stack corruption in ODS reader.

zip_reader_create takes a pointer to a struct string, which
is used to  store error messages during zip operations.  I had
been passing the address of a string declared as local variable
in ods_probe.  However since the zip reader survives ods_probe,
this caused stack corruption in later read operations.

This change makes the string a member of the reader created (if any).

Fix stack corruption in ODS reader.

zip_reader_create takes a pointer to a struct string, which
is used to  store error messages during zip operations.  I had
been passing the address of a string declared as local variable
in ods_probe.  However since the zip reader survives ods_probe,
this caused stack corruption in later read operations.

This change makes the string a member of the reader created (if any).

Update test to include new warning

ods reader: Report cells which cannot be converted

Gnumeric reader: Report cells which cannot be converted

Spreadsheetreader: new function create_cell_ref and rename existing function with that name.

Rename convert_cell_ref to convert_cell_range which is a more accurate
description of the function, and add a new function, with the original
name of the function we've just renamed.

The new function (create_cell_ref) does just that.

psppire-cell-renderer-button: Don't use gdk_window_coords_to_parent().

It was introduced in GDK 2.22, which is newer than we require.

Reported by John Darrington.

psppire-cell-renderer-button: Do not update 'event->window'.

Commit 5b59146f8d0 (psppire-cell-renderer-button: Translate coords before
propagating event.) caused GDK and GTK+ warnings and criticals and a crash
if the scenario described in the bug report were repeated a number of
times.  This change appears to fix the problem.

Reported by John Darrington.
Bug #38922.

Revert "psppire-cell-renderer-button: Translate coords before propagating event."

This reverts commit 5b59146f8d060754b14eba415dcc9066d8bc7b02.

Remove include directives for non-existant files

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

psppire-cell-renderer-button: Translate coords before propagating event.

When propagating a right-click upward, the x and y coordinates in the
event are those for the button, not for the parent window, but the parent
window's signal handlers will expect that they are relative to the parent
window.  This commit makes that so.

Without this commit, the following scenario occurs:

1. Open a dataset with at least two cases.
2. Switch to the Data View
3. Place the pointer over (say) the tenth row header button.
4. Click the left mouse button.
5. Click the right mouse button.
6. The popup menu appears (as it should), but row 1 gets selected instead
   of row 10.

With this commit, in step 6 the correct row is selected.

Bug #38922.
Reported by John Darrington.

Merge 'psppsheet' into 'master'.

This merge required considerable manual resolution in
text-data-import-dialog.c because that file was split into multiple files
on master.

Remove unused variable assigned in expression.

Found by clang analayzer

psppire-window: Disconnect signal handlers before unregistering.

Otherwise, the call to psppire_window_register_remove() calls back into
remove_menuitem() via the "remove" signal, but window->menu referenced in
that callback has already been freed.

Found by valgrind.