Error with g_critical instead of g_assert on invalid variable types.

Rewrite documentation for the RECODE command.

Reviewed-by: Ben Pfaff

Converted Sort dialog to a PsppireDialogAction object

po: Update German translation from translation-project.org.

Converted reliability dialog to a PsppireDialogAction object

Removed erroneous duplicated line

Converted roc-dialog to a PsppireDialogAction object

Convert kmeans dialog to a PsppireDialogAction

Converted the correlations dialog to a PsppireDialogAction object

Move PSPP shared libraries from $(libdir) to $(libdir)/pspp.

The PSPP shared libraries do not have a stable ABI and so they
should not go in a public system library directory.  This moves
them to a private library subdirectory.

Name the src/libpspp convenience library consistently.

src/libpspp.la and src/libpspp/libpspp.la have the same name,
although they are in different directories.  This seems unduly
confusing.  This commit renames the latter to liblibpspp.la, which
is at least different from libpspp.la and is also consistent with
the naming of the other libtool convenience libraries as
lib<directory>.la.

Don't crash on Games-Howell test when there are small numbers of cases per category.

Fixes bug #34978

gui: Fix warnings.

This fixes warnings introduced in commit 53508ccb5785 (New abstract
class PsppireDialogAction).

perl-module: Fix bad arguments in call to settings_init().

Found by clang.
Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

perl-module: Add missing "#include"s to PSPP.xs.

Found by clang.
Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

perl-module: Add "!= NULL" to assignment used as condition.

Suppresses a "clang" warning.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

pspp-dump-sav: Fix sizeof on "array" that is actually a pointer.

Found by clang.
Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

New abstract class PsppireDialogAction

Added a new object PsppireDialogAction which abstracts the behaviour of
dialog boxes.

Converted the Descriptived Dialog and the Variable Info Dialog to use this
new abstraction.

Reviewed-by: Ben Pfaff

Makefiles: Define dist_man_MANS before trying to add to it.

Fixes Automake complaint:
utilities/automake.mk:2: dist_man_MANS must be set with `=' before using `+='

Reported-by: John Darrington <john@darrington.wattle.id.au>

pspp-dump-sav: Mark manpage to be distributed.

RANK: Avoid putting declaration after statement.

pspp-dump-sav: Add manpage.

gui: Use $(INSTALL_DATA), not $(INSTALL), to install data files.

Plain $(INSTALL) always adds the executable bit to the installed
files, which has the wrong effect for data files.

Found by lintian.

Moved rank.q to rank.c

Reimplemented the logo as a svg image and converted to a 64x64 bit icon

README: Update.

Suggested-by: John Darrington <john@darrington.wattle.id.au>

Added documentation for the MEANS command

Improve behaviour of resizing the value chooser widget

MEANS: Added further tests for missing value behaviour

MEANS: Implemented /MISSING=TABLE subcommand

MEANS: implemented the /CELLS={ALL,NONE,DEFAULT} options and added a test

Add German translation from translationproject.org.

This translation is by Matthias Keil <matthias.keil@uni-jena.de> and
the German translation team.

Remove stray include directive

Corrected various misspellings in the documentation

MEANS: remove arguments to callbacks which were always unused

Fix warnings

MEANS: This command is IMO now stable enough to be used. Adding to command.def

Also added some tests.

Categoricals cleanup:  New structure 'payload' which reduces the
number of arguments to categoricals_create

Avoid deprecated Perl syntax.

Apparently new perl versions don't like qw(x y z) without outer  parentheses.  See
http://search.cpan.org/~flora/perl-5.14.2/pod/perl5140delta.pod#Use_of_qw%28...%29_as_parentheses

This change adds these parentheses.

Reported-by: bojo42 <bojo42@gmail.com>

Use idle callback to write to the local config directory

When the users config directory is on a filesystem with high latency,
moving a dialog box resulted in many writes and thus an unacceptable
delay.  This change fixes that by consolidating the writes into a
low priority idle callback.

Fix PSPP link on OpenSUSE and possibly Ubuntu 11.10.

Reported-by: friedrich nietzsche <f_nietsje@hotmail.com>
at http://comments.gmane.org/gmane.comp.statistics.pspp.devel/4850
Reported-by: Luis Cebamanos <luiceur@gmail.com>
at http://comments.gmane.org/gmane.comp.statistics.pspp.devel/4869
Reported-by: Mindaugus <embar@super.lt>
Tested-by: Mindaugus <embar@super.lt>

cairo: Only apply options to table items, to avoid crash.

Most of the members of struct xr_rendering, including the 'xr'
member, are used only for table items.  In chart items, these
members are zeroed.  But xr_rendering_apply_options() attempted to
use the 'xr' member even for chart items, which caused a crash.

This reverts and replaces the provisional fix already applied by
John Darrington in commit ee7b576822f (Avoid crash rendering charts
in the gui interface.), which also fixed the crash but added extra
unnecessary work.

Reported-by: Nick Macdonald <nick@nickmacdonald.net>

online_help: use g_spawn_async instead of g_spawn_command_line_async

This change avoids potential problems caused by metacharacters in the
constructed command arguments.

Suggested by Ben Pfaff.

Avoid crash rendering charts in the gui interface.

Reported-by: Nick Macdonald <nick@nickmacdonald.net>
Provisionally Reviewed-by: Ben Pfaff <blp@gnu.org>

Fix memory leaks parsing string expressions

Fix memory leaks

lex_get_text_buffer_read: Avoid potential buffer overflow.

Add files to manifest, inadvertantly omitted from previous commit.

New file: builder-wrapper.h and builder-wrapper.c

Move the functions wrapping GtkBuilder out of helper.c
and into their own file.

Set the output driver parameters dynamically from the output window style.

This change allows the output driver options to be changed after the driver
has been created.  This is necessary in the GUI to follow the style changes
(eg from theme engines etc)

psppire-dialog: Add missing prototype for psppire_dialog_help() to header.

doc/automake.mk: Added eacute to the list of docbook entities which need to be mutated

Correct packing properties of sort cases dialog.

Escape # character in call to g_spawn_command_line_async

It seems that this character was being treated as a comment start.

Added a callback to the dialog help buttons, and a handler to call
the manual when clicked.

Fix errors in the Texinfo source discovered by makeinfo 4.13.90

Output Viewer:  Try to adapt to changes in style

Set the fg and bg of viewer's container and its children to the
text and base of the output window upon style change.  This means that
the output viewer will respond to changes in style once even after the
program has been started.

We still need some way to set the foreground colours of the cairo output
after the driver has been created, but that doesn't seem to be currently
possible.

gui: Make Enter work in the Goto Case dialog.

Without this change, Enter does nothing in the dialog that Edit|Goto Case
brings up.  With this change, Enter activates the Goto button.

Output viewer: use the widget's style to set the colors of the
output driver.

Cairo output driver: Added options to set the foreground and background
colours.

Remove xr_draw_chart from cairo.h and make static

The function xr_draw_chart is used only in cairo.c, so it seems there
is little point in having it public.

Added basic framework for the MEANS command.

Not much here yet.  Just the parser and framework
for the output.

Added a function to clone an interaction.

Value Labels Dialog: Fix packing properties

gui: Fix behavior of print preview in psppire-output-window.

Print preview only showed one page because the GtkPrintOperation emits
"paginate" again even after it returns TRUE once and our "paginate"
implementation was not prepared for that.

Also fixes a memory leak in paginate(): the driver used for pagination was
not being freed.

Reported-by: John Darrington <john@darrington.wattle.id.au>

ONEWAY: Fixed crash when the dependent variable contained only missing values.

When forming a struct categoricals, there is no way in advance to know if the
given categorical variables will actually contain any values.  If one or more
of them contains no non-missing values, then it's not meaningfull to talk
about categories.

Such cases were causing oneway to crash.  Thanks to Dave Purdy
<David_Purdy@btinternet.com> for reporting this problem.

Remove unused global variables

gui: Fix printing of tables split across pages in psppire-output-window.

xr_driver_output_item() expects the caller to call xr_driver_next_page()
as long as xr_driver_need_new_page() returns true, but the printing code
didn't do that, so tables split across pages in printouts caused an
assertion failure at the next call to xr_driver_output_item().

Reported-by: John Darrington <john@darrington.wattle.id.au>

Avoid declaring variables in the middle of a block, to avoid requiring C99.

categoricals.c: Rename function to more accurately reflect its purpose

Remove unused function

GLM: Mimic spss output with /INTERCEPT=EXCLUDE

Fix memory leak

GLM: Added implementation for the Type 3 sums of squares.

This seems to match expectations, except for the Intercept term.

GLM: Added test for type II sums of squares

Whitespace changes only

categoricals.c: Effects coding.

Implemented Effects (also called Contrast) coding instead of Dummy Coding
and corrected logic error.

GLM test: Remove redundant transformations and reorder.

This change does not affect the test or its result, but makes it
easier to understand.

GLM: Implemented the Type I sums of squares and added a test for that.

GLM: Update logic for choosing types of sums of squares

glm.c: Allow parser to accept METHOD=SSTYPE(n) subcommand

glm.c:get_ssq Remove unnecessary assignment

categoricals.c Fix the calculation of the per encoding sums

Rename df --> df_prod and comment accordingly

categoricals.c: Reimpliment the get_binary_by subscript function

Instead of making ad hoc encodings for interactions, use the product
of the encodings of the individual variables.  This should not affect
the outcome of any current test cases, but makes the data flow easier
to follow and to debug.

sweep.c: Use gsl_matrix_memcpy instead of element by element copying.

Although only the upper triangle needs to be copied, there is no
disadvantage copying the entire matrix (the lower triangle is never read)
and the expense of using two loops is likely to outweigh the saving achieved
by avoiding the other triangle.

sweep.c: Move repeated call out of if ... else

This line is identical for all cases, so it only needs to appear once.

sweep.c: Reduce scope of local variables and avoid reusing them.

This makes the code easier to follow, and helps the compiler with
optimisation.

sweep.c: Reverse sense of consistency tests.

This avoids numerous levels of indentation.

sweep.c: swap rows/columns instead of using indirection for last_col

This makes the code shorter, and I believe should make it faster too.

GLM: Add debugging option /SHOWCODES

Remove useless code

File Open: Don't try to analyse the file if it's a directory.

The File Open dialog box looks at the contents of the selected file
in order to decide whether to set the sensitivity of the encoding
selector.  But if the file was a directory this caused error messages
on windoze. Closes bug #34773

GET: Fix confusion over the type of the 'type' parameter.

The 'type' parameter was being used to pass in an "enum reader_command" but
it was then later assigned a value of "enum pfm_type".  This renames 'type'
to 'command' for clarity and just deletes the assignment of the "enum
pfm_type" value entirely because the value was not used.

Found by Clang.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

DO REPEAT: Properly convert lex_syntax_mode to segmenter_mode.

Clang reported that "enum lex_syntax_mode" was being implicitly converted
to "enum segmenter_mode".  Luckily, equal values in each enum have the
same meaning, but it seems better to convert by hand, so this patch
implements that.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

hmap: Mark definition of hmap_node_hash() "static inline".

This suppresses a Clang warning.  The prototype earlier in the file
was already marked "static inline", so I don't think that this is a
functional change.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

str: Mark definition of ss_empty() "static inline".

This suppresses a Clang warning.  The prototype earlier in the file
was already marked "static inline", so I don't think that this is a
functional change.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

ONEWAY: Fix printf format specifier.

Reported-by: Jeremy Lavergne <jeremy@lavergne.gotdns.org>

configure: Add option --enable-Werror to add -Werror to CFLAGS.

-Werror is useful for development, but it screws up configure because it's
impossible to guess what new warnings compilers will add in the future.
This commit adds a new configure option to add CFLAGS after the configure
checks are done.

The use of AC_CONFIG_COMMANDS_PRE is based on Eric Blake's suggestion on
the autoconf mailing list: "AC_CONFIG_COMMANDS_PRE probably fits the bill
as the ideal macro to use for guaranteeing that you inject your shell code
at the last possible moment."

The Autoconf code added in this commit, as well as this commit message,
come from Open vSwitch.

crosstabs-dialog: Remove unused vars from on_format_clicked().