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

Corrected example files. Closes bug #34343

Fixed a crash in REGRESSION when there were no non-missing values.

Move interaction subset predicates out of glm.c into interaction.c

Renamed function:   is_subset --> interaction_is_proper_subset
                    drop_from_submodel --> interaction_is_subset
and moved them into interaction.c  This seems to me to be easier
to understand, and more efficient.

GLM: Add a test for a full factorial anova example

Smake give correct dependencies where they are well defined

Added dialog box for univariate anova.

Added a dialog box to conduct a univariate anova using the GLM
command.  Currently only entry of the dependent variable and the
factors are possible.   Choosing the interactions must be done
through syntax, if the default is not what the user wants.

Fix warnings and remove conditional compilation which is no longer necessary

Fixed type 3 sums of squares for models containing interactions

Added a summary introduction to INSTALL, to try to make it less frightening to inexperienced users

Moved call to gl_NEXT_HEADER outside the conditional.

Fixes bug #33585

File Open dialog: disable encoding selector when not applicable

Filter file choosers by mimetype instead of file name

Avoid compiler warnings about uninitialised variables.

Some compilers produced warnings about possible use of
uninitialised variables (although such use was in fact not possible)
These warnings were harmless (perhaps even erroneous) but
it's no problem to avoid them.

Fix make makedistcheck and re-organize icon paths.

This change does several things:

* renamed psppicon.png to pspp.png - The word "icon" is redundant
  (it's also a misuse of English).

* Changed the install target of the existing 16x16 pictograms out of the apps
  directory. Instead, they now go in a directory of their own called "pspp".
  This is more in line with what other applications seem to do.  It also makes
  it a lot easier to deal uninstall them, since we can be reasonably sure that
  everything in the "pspp" directory was put there by us.

* Ensures that make distcheck passes.

GLM: Disable the interactions option until it actually gives the right results

Added an implementation of the median test

Fix build error if "close" macro is not always visible.

Harry Thijssen <pspp@sjpaes.nl> reported the following errors
with recent Gnulib on Win32:

src/language/lexer/lexer.c: In function 'lex_source_destroy':
src/language/lexer/lexer.c:1455: error: 'const struct
    lex_reader_class' has no member named 'rpl_close'
src/language/lexer/lexer.c:1456: error: 'const struct
    lex_reader_class' has no member named 'rpl_close'

The problem was that the Gnulib substitute for <unistd.h>
defines close to rpl_close, but this macro definition wasn't
visible when lexer.c included lexer.h and became visible later.
This commit fixes the problem by making sure that the macro
definition is always visible in lexer.h.

A better solution would probably be to rename the 'close'
member to something other than a standard POSIX function name.

The change to control-stack.h is just precautionary; Harry
didn't report the same problem there.

range-set: Fix typo in comment.

Improve paste to Openoffice.org &c

Added "css" and "borders" options to the html driver, and
set these options to false when generating clipboard data.
This produces better looking results when cut & pasting to
OOo

Update the recommended gnulib commit, since it contains an important w32 fix

Fix warnings

po: Add French translation from translationproject.org

gui: Add missing $(DESTDIR) to install rule.

Also, delete white space from otherwise blank line.

glade: Fix building libglade_psppire.

Otherwise GCC can't find the gtksourceview header files.

gui: Streamline pspp.desktop

Remove the extension for Icon (unnecessary, in case in
future we want to provide for example SVG icon) and removed
unnecessary keys. Do we know for fact that we will never ever
support Notifications? And I guess if we provide the .desktop and
executable as part of the same tarball/package there is not much
chance we would have binary missing. I have also added a Czech
translation of keys.

Signed-off-by: Matìj Cepl <mcepl@redhat.com>

Merge remote branch 'origin/sourceview'