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().

q2c: Remove gettext.h #include from generated code.

The generated code no longer uses any of the gettext symbols.

q2c: Remove final untranslated strings from q2c.c.

The "restrictions" that could be specified in q2c put strings
in the output that needed to be translated but could not.  This
commit drops the feature and removes all of its prior users.
Some of the users required new validation code to be added, but
many of them in the SET command did not because the value was
not actually used anywhere.

q2c: Use new subc_list_error() to report too many subcommands.

This removes one string from q2c.c that needs translation but
cannot currently be translated.

lexer: New function lex_force_string_or_id().

This new function removes a string from q2c.c that requires
translation but currently cannot be translated.

lexer: New functions lex_spec_missing(), lex_spec_only_once().

These functions don't have many users yet but they make logical
sense to me.

lexer: Use error helper functions in more situations.

The error helpers print error messages that are just as helpful
as handwritten error messages in most cases, especially now
that the error messages include column numbers.  They also
have advantages for i18n.

One important purpose of this commit is to remove several
strings from q2c.c that require translation but currently are
not translatable.

lexer: Drop lexer parameter from lex_sbc_missing().

There's no point in passing in the lexer and using lex_error(),
because the extra information that lex_error() outputs does
not help the user find the error.

lexer: New function lex_error_expecting().

This centralizes some fairly common strings into lexer.c.

LOOP: Correctly implement MXLOOPS and add a test.

Previously the MXLOOPS value was implemented incorrectly: the
loop would actually iterate MXLOOPS+1 times.  This commit fixes
the problem.

settings: Implement MXLOOPS subcommand.

It had been overlooked.

The following commit adds a test.

settings: Update default MXLOOPS and document.

MXLOOPS should be 40 by default, for compatibility.

SET: Warn for more obsolete subcommands.

doc: Remove reference to unimplemented SET PAGER command.

Bug #33632.
Reported by John Darrington.

lexer: Rename 'close' to 'destroy' in struct lex_reader_class.

This seems like a cleaner solution to the problem of Gnulib
sometimes defining 'close' as a macro than to add an extra
include file and a big comment.

gui: Fix spelling error in macro name.

gui: Make .csv available for output export with File|Export.

The hard-coded 6 here was wrong: there are 7 file formats.
It's better to use the n_FT constant.

gui: Fix configuring Unicode box characters.

Commit 14b3603043 "ascii: Add support for multibyte characters"
changed how the box characters are configured but failed to
update psppire-output-window.c, so attempting to paste a table
into an application that accepts UTF-8 text would output many
warnings like "unknown option `box[0201]'" and not actually use
Unicode box characters.

This fixes the problem.

New object: psppire-lex-reader

This object reads lexer input direct from a GtkBuffer instead of converting to a string first.

Remove duplicate #include lines

Remove labelling options from Crosstabs GUI

The previous change removed the label options from the crosstabs commmand,
but didn't update the gui accordingly.  This change fixes that.

Use variable labels in crosstabs.

Use a variable's label instead of its name (if it has one) in the crosstabs
output.  Removed the FORMAT=LABEL subcommand which is no longer supported
by spss.  Updated documentation to reflect the code.

Remove redundant error message.

This error message is not required, because lex_force_string already emits such message.

Consolidate some very similar user visible strings

Added some missing gettext macros

Fix potential problems with non-literal printf arguments.

In a few places we were passing a constructed string to a printf-like
function as its format argument.  This could have been problematic if
the constructed string had contained a % (for example due to that
character occuring in a variable label).  This change avoids those
situations.

Fix incorrectly ordered arguments to xcalloc

The gnulib xcalloc call has the following signature:
 void *calloc(size_t N, size_t S);
where N is the number of objects and S is the size of each object.
In many places, we had these arguments transposed. In many implementations
this doesn't matter since the two arguments are simply multiplied together.
However, on some systems this can cause problems (ie crash), if S is zero.
This change fixes all calls where the size was being passes as the first
argument instead of the second.

Fixed a crash in the oneway command

Add a warning to GLM until we've sorted out its problems

Remove visibility from the univariate analysis menu until it works better

Added a base parameter to the interaction_case_hash function

Fix crash on parseing invalid GLM syntax

Implemented the SET=TNUMBERS subcommand

Optimisation for the type3 sum of squares procedure