caseproto_ref: Append compiler directive WARN_UNUSED_RESULT

make-file.c: Behave better under windows operating systems

Under w32 it is necessary to convert all file names to UTF-16 and prefix all
the file related system calls with "_w".  Otherwise, calls involving filenames
which contain non-ascii characters will not work properly.

QUICK CLUSTER: Avoid "unused parameter" warning.

QUICK CLUSTER: New subcommand: /PRINT

This subcommand can be used to show the initial cluster centres and
the final cluster membership.

Closes bug #41019

Skip fifo test for w32 hosted builds

Append the correct number of null bytes to the end of recoded strings

Previously the recode_string et al functions assumed that the correct way to
terminate a string was to append a single \0.  With encodings such as UTF-16
however, two \0 bytes are necessary and four with UTF-32.  This change fixes
the problem.

Change pattern of tmpfiles from <name>.tmpXXXXXX to <name>tmpXXXXXX

Under wine, and presumably certain windows systems, filenames with more than one dot are not allowed.

Move definition of handler inside the #ifdef HAVE_READLINE

Provide stubs for spreadsheet functions when no spreadsheet support is built

This prevents link errors when compiling with full optimisation.

Gnumeric Reader: Fix memory leak

Add tests for cases exercising the atomic replacement mechanism

Add a test case when writing to a fifo.
Add some test cases for situations where file creation fails.  Failure
to create a file should be reported gracefully.

Remove unnecessary #include directive

struct file_handle: Add new member file_name_encoding and an accessor function

Ensure all lex_readers have the encoding parameter

replace_file_start: remove parameter TMP_FILE

This parameter was never used (all callers passed NULL) and I cannot
imagine why anyone would be interested in it.

replace_file_start: Now takes a file handle instead of a file name

lexer.c: Remove file_name member from lex_reader_file

The parent class already has such a member so this is
redundant and complicates things.

Fix the INCLUDE /ENCODING subcommand and add a test for it.

It seems that INCLUDE /ENCODING had never worked nor had been
tested.

Zip Reader: Avoid undefined behaviour when reading short files

The zip reader makes many calls to fread.  However the return value was not checked.  This could
have had undefined consequences when reading short files.  This change checks all calls to fread.

tests: Distribute tests/data/test-encrypted.sps also.

Reported by John Darrington.

po: Update translations from translationproject.org.

pspp-convert: Support decrypting encrypted SPSS syntax files.

Thanks to charlesjohnsont@outlook.com for providing an example.
Bug #45974.

Fix memory leak in initialization

doc: Add missing file to doc_pspp_dev_TEXINFOS.

Reported by John Darrington.

PsppireSpreadsheetModel: Take reference to underlying model

doc: Document encrypted syntax file format.

Thanks to charlesjohnsont@outlook.com for providing an example.
Bug #45974.

Fix memory leak in PsppireVarSheet

Fix memory leak in ODS reader

Text file importer: Initialise the total_is_exact variable.

The text file importer was supposed to have a feature where the total number of lines
in the file was measured if the file was short, or estimated if it was long.  However
the variable which was supposed to hold this information was never initialised. This
change fixes that.

Ref count struct spreadsheet

struct spreadsheet is a base class for which all current derivations
are ref counted.  It makes sense then to have the base class refcounted
instead of the more derived ones

any-reader: Add some comments on interface and implementation.

Also, adjust pfm_detect() and sfm_detect() code to be consistent with
pcp_detect().  This should not cause a change in behavior.

pc+-file-reader: Fix handling of short files.

The code failed to return an error properly when fread() read past end of
file.

The function comment didn't properly reflect what the function actually
returned, so this commit also updates the comment.

Reported by John Darrington.

gui: Mark print_startup_time() unused to avoid warning.

Fix dereference to reallocated pointer in data-parser.c (parse_delimited_no_span, parse_delimited_span)

The parse_delimited_span and parse_delimited_no_span functions were
accessing a pointer from within a loop, but the pointer had been
initialised outside the loop.  However the loop itself was deallocating
the memory referenced by that pointer.

This change initialises the pointer within the loop so as to avoid
a potential crash here.

po: Update translations from translationproject.org.

Remove some unnecessary #include directives

Have "make distcheck" run without errors

Splash screen: Flush display after drawing.

This avoids a few milliseconds of black which some users have reported.

Encoding selector: fix memory leak

PsppireEncodingSelector: Add comment

Don't bother freeing import dialog memory. Thus avoid crashes.

A number of users have complained that the import dialog assistant
crashes on exit.  This seems to be because some callback is accessing
the data after it has been freed.

This change "fixes" that by simply not ever freeing the data.  This
will cause a memory leak.  However during my investigations, I have
found that this dialog leaks so much anyway that these few bytes are
hardly significant.

A proper fix will hopefully be forthcoming.

Affects bug #45805

TextImportDialog: Gtk[VH]Box  --> GtkBox

TextImportDialog: Remove unused struct definition

TextImportDialog: GtkVPaned --> GtkPaned

TextImportDialog: GtkTable --> GtkGrid

Pulled the latest translations from translation-project.org

Restore the splash screen.

Re-implementat the splash screen, using functinos which are not
deprecated under Gtk3.  Note:  This implementation does not
use the gtk functions which the Gtk devs recommend for splash screens.
What they forget, is that a primary purpose of a splash screen is
that it should appear with the minimum of delay.  gtk_init can
have significant delay - hence the recommended way of doing splash
screens in Gtk is self defeating.

Pulled the latest translations from translation-project.org

Remove deprecation: gtk_misc_set_alignment -> gtk_widget_set_valign

Removed superfluous #include directive

sys-file-reader: Add PSPP version to messages that ask for files.

This may make it easier to advise users.

Suggested by Harry Thijssen <pspp4windows@gmail.com>.

REGRESSION: Relaxed the syntax parser slightly and include /METHO=ENTER in generated syntax.

This improves comparibility between PSPP and other software.

Addresses bug #45667

T-TEST: Allow comma to be omitted in GROUPS subcommand.

Reported by Douglas Bonett and Friedrich Beckmann.

T-TEST: Allow both CIN and CI on CRITERIA.

Upon reading the bug report further, I see that it says that SPSS also
allows both forms.

Bug #45664.

T-TEST: Use correct keyword CI (instead of CIN) on CRITERIA subcommand.

Reported by Douglas Bonett <dgbonett@ucsc.edu>.

psppire_cell_renderer_button_render: Avoid deprecation warning

PsppireDataWindow (on_switch_page): Add comment and reformat

PsppireDataWindow: reduce scope of incidental
variables. Add comments.

Replace deprecated GtkHBox with GtkBox

Data Editor: Remove the GtkHandleBox

This widget is deprecated, and I never saw the point of it anyway.

PsppireDialogAction and subclasses - use function to set class activate and thus minimise deprecation warnings

All subclasses of PsppireDialogAction needed to use GtkActionClass.  This widget however
is deprecated in recent Gtk versions.  Meaning that all instances of PsppireDialogAction
implementations would provoke a deprecation warning at compile time.

This change avoids that by providing a common function to set GtkActionClass->activate.
Thus, there is now only a single warning instead of many.

system-file-format; Add a little more information gleaned from examples.

Added notes about glade versions to src/ui/gui/memorandum.txt

sys-file-reader: Change some errors to warnings.

I encountered some system files with corrupt data in the long string
missing value record.  Until now, this entirely prevented the file from
being read.  This commit changes that to just a warning, enabling the file
to be read.

Also revises system file format documentation to better explain that
unknown or corrupted extension records should be ignored.

sys-file-reader: Avoid hanging forever if a system file has no variables.

dictionary: Make dict_delete_var() much faster in common case.

When a system file is opened, dict_delete_consecutive_vars() deletes lots
of variables if the system file has long string variables.  This was very
slow because of the O(n**2) behavior in dict_delete_var().  The biggest
cost of that O(n**2) behavior was in calling var_clone() on every variable
several times.  Fortunately, the var_clone() call is not necessary in this
case, because the cloned variable is only needed for callbacks, which
aren't set when a dictionary is being read.

This reduced the time to read the dictionary in a particular sample system
file from seconds to milliseconds.

Fix behaviour of the REGRESSION /STATISTICS subcommand.

GET DATA/TYPE=TXT: Remove ESCAPE subcommand and make its behavior default.

Frans Houweling reported that this is the default and only behavior for
GET DATA/TYPE=TXT in SPSS 14, at:
http://lists.gnu.org/archive/html/pspp-users/2015-07/msg00027.html

Therefore, this commit makes this behavior the default and only behavior
in PSPP as well.  (In my opinion it's the only sensible behavior, but SPSS
didn't document it so I assumed that it didn't do it that way.)

PsppireValueEntry: Enable can_focus in realize callback

PsppireDialogActionIndepSamples: Replace GtkTable with GtkGrid

Independent Samples T-Test dialog: Replace deprected GtkVBox and GtkHBox

GET DATA TXT: change syntax generation from /IMPORTCASES to IMPORTCASE

Apparently, spss only accepts IMPORTCASE (contrary to its documentation).

Reported by: Frans Houweling

Linreg: Reduce scope of variables

Avoid gtk_widget_reparent which is deprecated

Avoid GtkArrow which is deprecated

PsppireWindowRegister: Remove unused function

Fix bug when searching through an empty dataset.

Closes bug #45430

Correct bad button label

missing-val-dialog.c: fixed bug #45419 - added value type checks

This patch fixes bug #45419:
psppire crashes when entering missing data in new string variable.

The missing value width is now derived from the variable type. I noticed that
there are currently no type checks and no detailed warnings. I added the type
checks which were already provided by data_in and forward them to the existing
error dialog box. If an error is detected, the focus of the dialog box is
set to the field which produces the error.

Fix non-working accelerator key

PsppireMeansLayer: Remove useless widget

Not only did this GtkAlignment not do anything, but it is also deprecated in Gtk3

PsppireValChooser: remove deprecated functions

GtkTable -> GtkGrid

Remove deprecations from help-menu.c

OutputWindow: Convert deprecated widgets to non-deprecated ones

Removed unreachable code section

gtk_widget_size_request --> gtk_widget_get_preferred_size

Replace two instances of gdk_pointer_grab with gdk_device_grab

Revert "Replaced a few instances of gdk_window_get_pointer by gdk_device_get_position"

This reverts commit 70702237d6765a3cc276e952919211387eeee87c.
It seems to cause problems selecting areas on the sheet.

Replaced a few instances of gdk_window_get_pointer by gdk_device_get_position

psppire-output-view.c: added check for view->xr - fixes bug #45421

The output windows crashed when an empty syntax window is executed. The
rerender function got called without a valid view->xr (=NULL). I added a
check for this condition and create a new view->xr as it is done in the
psppire_output_view_put function.

This fixes bug #45421: Crash when trying to run empty syntax

Replace deprecated GTK_STOCK_ symbols with translatable strings

Paired samples T-TEST: fix bug calculating correlation coefficient.

The value did not properly consider the weights of the data.
Thanks to Douglas Bonett <dgbonett@ucsc.edu> for reporting this.

CROSSTABS: Fixed the reporting of the standard error of Cohen's Kappa

The Standard error assuming the null hypothesis was being reported
when the standard error NOT assuming the null hypothesis ought to
have been.

Thanks to Joanne Ingram <joanne.ingram@ed.ac.uk> for reporting this
problem, and to Douglas Bonett <dgbonett@ucsc.edu> for explaining
how to fix it.

gdk_cursor_unref -> g_object_unref

fix s390x build failure on debian due to wrong type casting

    The problem showed up on s390x for the 0.8.5 debian version
    where four graph tests using pdf export failed with the same assertion. The
    problem is due to wrong casting. intptr_t is long int (64 Bit) on s390x
    but nl_langinfo returns 0xd200000000 on s390x where 0 is the integer result.
    This results in wrong but positive height and width values for the paper size.

fix crash when exporting pdf via gui - fixes bug #45272

    The problem https://savannah.gnu.org/bugs/?45272 occurs when the pdf
    export is done via the gui. The problem is that tables containing only
    carriage returns are considered to have no vertical space consumption.

Select cases dialog: Change GtkTable to GtkGrid

Replace some obsolete Gtk2 functions

Fix typo in value chooser