UPDATE: Do not update from missing values in transaction files.

The documentation said that missing and all-spaces values in transaction
files did not update the master file, but the implementation was wrong.

Reported by Kees Varekamp <kees@mroffice.org>.

combine-files: Break apart apply_file_case_and_advance().

An upcoming patch will sometimes need to do something different from
apply_case() before advancing.

value: New function value_is_spaces().

Update Gnulib hash

PsppireOutputWindow (create_xr_print_driver) fix LC_NUMERIC issues.

This function used *printf with a %f directive, which  under certain
locales would have generated a string like "270,00x210,00mm" which the
print driver would have rejected.
This change uses the newly available c_xasprintf function from gnulib
to alleviate this problem.

Replaced sprintf with c_snprintf where appropriate

Updated the German translation from translationproject.org

Replaced strtod with c_strtod where appropriate

Should LC_NUMERIC ever get set, this will become important.

Converted crosstabs-dialog to the new way of doing things

measure.c: Use c_strtod instead of strtod

Since the input to these function come from hard coded strings, it is not appropriate
to interpret them in a locale dependent manner.

data-out.c: Added assertion to avoid out of bounds memory access

This could (has!) happened if the locale is not correctly set

SHOW: Add JOURNAL subcommand.

Bug #35665.
Requested by Harry Thijssen.

histogram: Add missing #include.

Commit 02b9b37698eb "Disable asserts in histogram code except in
--testing-mode" added a use of settings_get_testing_mode() but didn't
include the proper header.  This fixes the problem.

Disable asserts in histogram code except in --testing-mode

The assertions in histogram.c are algebraically correct,
but due to floating point errors may not actually be true.
This change disables them except when in testing mode.
The tests in charts.at rely upon them being enabled.
Closes bug #36395

DESCRIPTIVES: Make Z score calculation honor SPLIT FILE.

Until now, DESCRIPTIVES has not supported SPLIT FILE in Z score
calculations.  That is, Z scores have been calculated on the basis of a
single group for the entire file, which doesn't make any sense.  This
commit properly implements Z score calculation on a per-split group basis.

Bug #16351.

tests: Add tests to cover corner cases in DATESUM, DATEDIFF.

These corner cases weren't covered by previous tests.

Bug #30708.
Reported by Daniel E WILLIAMS <Daniel.E.Williams@state.or.us>.

Remove feature to automatically connect to GDB when debugging is enabled.

This feature caused a few problems in "make check" on some distributions
when debugging was enabled at compile time.  It is probably possible to
find and fix those particular problems, but the feature itself is
little-used, so this commit instead removes it.

Bug #37444.
Reported by Mindaugas.

file handle properties: Make encoding member const

This member is never modified.  So declare it as const.  This way it
can be initialised from a static string.

Aggregate Dialog: use global callback

Runs Dialog: use global callback instead of a private one

psppire-data-window: Only allow saving a file with at least one variable.

A data file with no variables can't be saved to a file.  Until now, this
wasn't detected until the user had already chosen a file name, and the
message used to report it was confusing: "SAVE is allowed only after the
active file has been defined."  With this commit, the menu item and toolbar
item for save operations is, instead, disabled if the file has no
variables.

Bug #30700.
Reported by eric thivant, via Harry Thijssen, with help from Michel
Boaventura.

DATA LIST: Accept freefield format types without widths use minimum width.

A user reported that SPSS accepts syntax such as "DATA LIST FREE/
d (DATETIME)." using a default width for the field with no specified width.
This commit makes PSPP accept this syntax too, although it does not get
the width exactly right (please let me know if you figure out how to do
that).

Bug #30690.
Reported by Andreas Lappe.

format-parser: Improve error message when format specifier lacks width.

Until now, parse_format_specifier() was willing to parse, e.g. "datetime"
as a format specifier.  It would then be rejected later by e.g.
fmt_check_input() with a message like "Input format DATETIME0.0 specifies
width 0, but DATETIME requires a width between 17 and 40," which makes no
sense to the user.  This commit improves the error message.

data-parser: Use data-file error types for data-file errors.

The SW error type that these calls to msg() used before this command
include the name of the executing command in the error message.  This can
be confusing because the executing command may not have any connection to
the command that said to read the data file.  This commit changes these
error types to DW, which is intended for errors in a data file (and omits
the command name).

Bug #26517.

INPUT PROGRAM: Don't break out of INPUT PROGRAM on interactive error.

Until now, any erroneous command typed within an input program at an
interactive command prompt implicitly "broke out" of the input program,
causing confusing behavior.  This commit fixes the problem: commands
following an erroneous command within INPUT PROGRAM continue to be part of
the input program.

Bug #33419.
Reported by John Darrington.

DATA LIST FREE: Warn when a quoted string is not followed by a delimiter.

A field parsed by DATA LIST FREE (or LIST) is either quoted or unquoted.
When it is quoted, this means that the entire field has to be quoted, that
is, it is not possible to quote some initial part of the field, then leave
some part of it unquoted (then possibly quote an additional part, etc.).
That means that the quoted portion should be followed by a delimiter,
such as white space, a comma, or end of line, but PSPP didn't warn when
it was not.  This commit makes PSPP warn in this situation, which makes
errors in the input easier to find.

Bug #26522.
Reported by Pascal Barbedor <pbarbedor@gmail.com>.

output: Make default output file name depend on the format.

Before this commit, "-O html" caused HTML output to be written to a file
named "-", which is confusing.  After this commit, it is written to a file
named "pspp.html".  Similarly for other output drivers, except that text
and CSV output are still written to stdout by default.

Bug #35030.
Reported by John Darrington.

CROSSTABS: Use FORMAT setting to control output format.

Reported by Daniel Harper.

Frequencies dialog: Convert to new system

Fixed bug in NPAR BINOMIAL

This bug would crash when an incorrect syntax was encountered.

Binomial test dialog:  Convert to new style of action objects.

Fix compiler warning

Refactoring: common function set_sensitivity from toggle

Many dialog boxes have a signal handler to set the sensitivity
of a widget, according to the state of a toggle button.  Until
now, they had all written their own.

Logistic Regression: Ignore cases with missing dependent variables.

Prior to this change, cases with missing values in the predictor
variables were ignored, but cases with missing values only in the
dependent variable would provoke an error.  Subsequent to this change
such cases are also ignored.  This seems to be the behaviour of other
software, despite what their documentation may suggest.

Logistic Regression: Added options dialog box

Logistic Regression: Added a test to check the confidence interval display

Logistic Regression: Fix bug where some confidence intervals were not reported.

Logistic Regression: Calculate the logs of the likelihood instead of the raw value.

The likelihood values can become extremely large which can result in
lost precision in a double.  This change therefore calculates the
logarithms of the likelihoods instead of their actual value.

Logistic Regression Dialog: Added missing shortcut target.

Changed the labels of the Regression menu to keep them shorter.

Added rudimentary version of the Logistic Regression Dialog

Analyze Menu: New submenu for Regression

Move the Linear Regression menuitem into a new submenu Regression.
This is in preparation for a new menuitem for Logistic Regression

Fix (again) problems displaying a GtkSourceView widget on Cygwin

Under certain compiler options, the effect of the gtk_*get_type
function call was optimised away.  This change fixes that.  It
seems that both static and volatile are necessary, in order to
avoid the compiler from complaining and/or optimising the call
away.

Logistic Regression: Optimise the classification calculations.

Instead of calculating the logit of the prediction for each case,
pre-calculate the inverse logit of the threshold before the process
starts.  This should give a small performance gain..

Fixed Texinfo problems which arise under Texinfo 4.13.90

Logistic Regression: Added the Classification Table

Added a table to logistic regression to show how the predicted values
would be classified by the calculated coefficients.

logistic.c: minor refactoring

Fix possible memory corruption when creating dialogs with selectors.

The PsppireSelector widget's class structure contains a map of selectors
indexed by the source widget with which they are associated.  Until now, that
map was never depopulated.   This could have caused problems if newly created
widgets happened to use the same address as old, destroyed ones.  This change
removes selectors from the map upon destruction.

Logistic Regression: Handle missing categoricals

Ensure that missing values on categorical predictors are handled properly.

Logistic Regression: Fix crash if categorical variable has no distinct values

Remove empty _finalize function

Find dialog: Don't crash when searching for a non-existant label.

Small refactoring and cleanup of psppire-selector.c

Made explicit variables from some expressions.  This provides
the opportinutiy to test if they are valid.

Remove _base_init and base_finalize functions from psppire-selector

Gtk+ documentation recommends these are avoided in modern code.
They differ only from class_init in the case where the class has subclasses,
which PsppireSelector does not.

Logistic Regression: Added categorical variable support

With this commit Logistic Regression now supports the /CATEGORICALS
subcommand, allowing a dependent variable to be regressed against one
or more categorical predictor variables.

Categoricals.c: Ensure that categories are processed order of their categorical variable(s).

Some procedures require that categoricals are processed in a particular order.
Mathematically, the order is irrelevant, however for reporting purposes it is
sometimes required.  This change adds a level of indirection, so that the codes
are allocated in ascending order of the categorical variable.

Categoricals: Provide separate functions for effects and dummy coding.

Until now, the function categoricals_get_code_for_case returned the
"effects" coding (ie, a bipolar code).  Some uses require "dummy" coding
(a binary coding).  This change provides seperate functions to return
whatever is desired.

Conflicts:

src/language/stats/logistic.c

Fixed some compiler warnings, which arose from recent commits.

Added mnemonics to rank types dialog

Added markup to GUI elements where necessary

Fix bug in parse_design_interaction.

Previously this function would crash if it failed to succesfully parse
an interaction. This change fixes that.

Categoricals: Accept null pointer as subject.

Previously, certain functions would crash, or behave badly if their
argument was NULL.  This change makes them do nothing, or return
default values, as appropriate.  This saves callers having to always
test for NULL.

cateoricals.c categoricals.h clean up.

Removed a prototype whose implementation no longer exists.
Added a few comments.
Whitespace changes.

Added mnemonic to t-test options dialog

Added Mnemonics to the value chooser widget and to the recode dialog.

Added markup to the spilt-file dialog

Compute Dialog: Add mnemonics and focus widgets on activation.

logistic.c: Fix read from uninitialised memory.

Found-by valgrind.

Move function parse_design_interaction into variable-parser.c

This function is (or will be) used by several procedures, so
make it public.

glm.c: (lex_match_variable and parse_design_interaction) remove dependence on struct glm

Made these functions independent of struct glm, so that they can potentially be reused
for other procedures.

Anova Dialog: Fix widget associations

logistic.c  : Avoid compiler warnings

GUI: Added Mnemonic keys to Examine dialog

First attempt at a LOGISTIC REGRESSION command

Updated my entry in AUTHORS

doc/introduction.texi: Update documentations introductory section

doc/automake.mk: Added a convenience rule for building all the documentation formats

doc/automake.mk: Replace lines inadvertently removed by previous commit

doc/automake.mk: Added further mutations to docbook xml to workaround makeinfo bugs

doc/automake.mk: New mutations to generated docbook xml to workaround makeinfo bugs

Avoid deprecated function: gtk_button_released

configure.ac: Increase pango minimum version from 1.20 to 1.22

Commit bd4d6e7453 (cairo: Avoid deprecated
pango_cairo_font_map_create_context().) added a dependency on pango
version 1.22 or later, but configure.ac only checked for version 1.20
or later.  This commit fixes the discrepancy.

INSTALL already documented that Pango 1.22 or later was needed.

Reported by John Darrington.

tests: Better isolate search text in test of error reporting.

Some GNU/Linux distributions print additional text on a segfault,
e.g. "Cannot exec debugger: No such file or directory", which this
test was seeing in output and interpreting as a failed test.  This
commit fixes the problem by ignoring text up to the ***** line that
marks the start of the expected output.

Bug #37444.
Reported by Mindaugus.

Suppress GCC 4.7 warnings by changes to code that make it less clear.

I'm not entirely happy with these changes, because they make the code
slightly less obvious.  However, it's still not really bad code, and
it's convenient to have a warning-free build.

Suppress GCC 4.7 warnings due to unimplemented features.

GCC 4.7 complains that the variables removed by this commit are
assigned values that are not used.  Fair enough, so this commit
removes them.

gnumeric-reader: Add cast to suppress warning.

This suppresses a GCC warning for a mismatch between
xmlInputCloseCallback, which takes a "void *" parameter, and
gzclose(), which takes a "gzFile" (which is "struct gzFile_s *").

This came up with an upgrade from "squeeze" to "wheezy".  I assume
that a gzclose prototype became visible or that gzFile changed away
from "void *".

An alternative would be to write a trampoline to just transform one
pointer type to the other, but the xmlReaderForIO() invocation
already has one function cast so a second cast doesn't make it much
worse.

split-file-dialog: Add use of unused variable in split_file_dialog().

'selector' was assigned but never used, but here's a use.

Reported by GCC 4.7.

Suppress warnings for suspicious use of "enum" constants.

In each case, the code is using values of enums in valid ways that
GCC 4.7 does not expect, so use a cast to suppress the warnings.

gui: Check return value of gtk_tree_model_get_iter().

In these two cases the return value was assigned to a variable but
never used.  This commit adds a check for the return value.

cairo: Avoid deprecated pango_cairo_font_map_create_context().

Pango 1.30 that is in Debian "wheezy" has marked this function as
deprecated.  The replacement pango_font_map_create_context() was
introduced in Pango 1.22, which was also in Debian "squeeze" (which
had Pango 1.28), so it seems safe to switch without worrying about
a Pango version check.

Remove unused "finalize" functions.

A finalize function that just calls into the parent class finalize
function is a no-op that need not exist.

Remove variables assigned to but never used.

Each of the variables removed here was assigned a value that was
never used, the assignment didn't have any useful side effects, and
I don't see anything that the variable should have been used for.

Found by GCC 4.7.

EXAMINE: Avoid useless call to casereader_count_cases().

The return value is never used, and this call could have significant
expense (it could cause the data to be read an extra time), so do not
do it.

Reported by GCC 4.7.

CROSSTABS: Fix bug in check for REPORT in general mode.

Found by GCC 4.7.

expressions: Fix coercion of numbers to booleans at top level.

The check for the expected type in type_check() used the wrong
constant (a node type instead of an expression type) and so the
"if" test was never true.  Also, OP_NUM_TO_BOOLEAN is binary, not
unary, so fixing just the test caused an assertion failure.

This commit fixes both problems and adds a test.

Found by GCC 4.7 report of a type mismatch.

gtk.h wrapper: Added *_set_realized and set_mapped

We are using these functions, but they are not defined for older gtk versions.

src/data/data-out.c: Added assertion to catch negative values

If the contents of string does not contain a '.', then integer_digits
will end up negative, thus causing a crash later.  This change adds
an assertion to catch this condition and thus make the diagnosis easier
should it ever occur.

Added keybindings to move to the extremities of the sheet.

Altered the Home key to move far left instead of up.
Added Ctrl Left/Right/Up/Down to move to the far edges of the
sheet.

This behaviour is consistent with other software.

Perl Module: Avoid forcing scalar value into a pointer.

The code fragment:  struct dictionary *dict = (void *) SvIV (dict_sv);
is dangerous.  This fix uses the properl Perl XS mechanism to get
the pointer and the scalar associated with the dictionary.

Reviewed-By: Ben Pfaff

Update widgets.c from psppsheet branch