psppire-output-window: Use fallback PATH_MAX on GNU/Hurd.

This fallback is from lib/clean-temp.c in Gnulib.

interaction.c: interaction_dump: don't crash on empty interactions (ie those with no variables)

Remove unnecessary #include

moments.c: Implement calc_semean.

This function was always declared, but never implemented.  It's fairly
trivial, but for consistency it makes sense to have here.

oneway.c: Set destroy pointer to NULL, since it is unused here

Categoricals.c: Call payload->destroy inside categoricals_done

categoricals.c: Delete redundant function dump_interaction

 Remove dump_interaction, since there is such a function in interaction.c

Examine tests: Remove leading slash from factor variables.

I haven't seen any examples of syntax which does this, and it leads to
ambiguities: Does /STATISTICS refer to the STATISTICS subcommand or to
a factor variable named "STATISTICS".  I think it's better therefore to
disallow a slash here.

pool: Use 16-byte alignment on 64-bit architectures.

glibc jmp_buf needs 16-byte alignment on ia64.  PSPP allocates
jmp_buf inside a pool in a few places, so this change should fix
many tests on ia64.

Found by Debian autobuilder.

Sort attributes by name for display, to make order arch-independent.

When data file attributes or variable attributes were displayed by
e.g. DISPLAY ATTRIBUTES or SYSFILE INFO, they were sorted in hash
order, which varies by architecture.  Therefore, the tests that
depended on this order failed on big-endian machines such as mips.
This commit fixes the problem by displaying attributes in sorted
order.

This fixes a failure in the "Perl copying system files" reported
by Debian buildds.

identifier: Apply isdigit() only to values in valid range.

Applying isdigit() to a value that is not EOF or in the range
0...UCHAR_MAX yields undefined behavior and in fact caused a
segfault on several Debian architectures for U+FFFD.

Found by "lexer properly reports scan errors" test on Debian buildds.

zip-writer: Add missing #include <byteswap.h>.

Fixes a broken build on Debian GNU/Linux, mips architecture.

encoding-guesser: Fall back to windows-1252 when UTF-8 can't be right.

Until now the encoding-guesser code has used UTF-8 as a fallback in
situations where we can tell that the file is not valid UTF-8.  In
this kind of situation having a single-byte character set as a
fallback makes more sense.  This commit hard-codes windows-1252 as
that fallback, since it is a widely encountered encoding (and
compatible with ISO-8859-1 as well).

John Darrington originally suggested this, if I recall correctly.

The bug report that spurred this work was from Harry Thijssen.  With
this commit, PSPP properly reads his windows-1252 file when the
system locale uses UTF-8 encoding.

encoding-guesser: Remove spurious blank line.

zip-writer.c: Fix incorrect big-endian handling.

Don't update the icon cache if DESTDIR is non-empty.

Packagers generally use "make install DESTDIR=xxx" to generate their install
files, which are then copied in place to the target filesystem.  In such a
scenario, it's inappropriate to copy the icon.cache files.  This change
therefore does not update the icon cache if a DESTDIR has been specified.
Instead it's left to the packager to make his own arrangements for updating
the icon cache on the target filesystem.

Updated the German translation from translationproject.org.

Updated the Lithuanian translation from translationproject.org.

Postgres test: Create socket in /tmp instead of current directory

It seems that some systems do not allow unix domain sockets with long
pathnames.  This change therefore creates a directory in /tmp using
the mktemp command.

Kruskal-Wallis: Sort grouped ranks before displaying them.

Debian's autobuilder showed that the KW tests were failing because the
hash which holds the rank entries was being iterated in arbitrary order.
This change sorts the entries before displaying them, so they should
always appear in the same order.

zip-test.c: More thorough error messages

Generate pspp.desktop

This change autogenerates the pspp.desktop file instead of simply including it.
This allows us to have the translations updated from the po files instead of
someone having to manually enter the strings directly in the file.

Remove translator credits from AUTHORS file.

This was getting hard to maintain, and is redundant, since there is
a "translator-credits" string in each translation.

ascii.c: Reallocate the lines after the page is resized

Remove redundant assignment

ascii.c: Move auto out of the #if HAVE_DECL_SIGWINCH condtional

Whether or not the width/length should be automatic should depend only on whether the
output is a terminal, and not upon the existance of SIGWINCH

ascii.c (ascii_reserve) Added assertion.

Added this assertion to catch a yet-to-be fixed array bounds bug.

po: Update Dutch translation from translationproject.org.

Added SHOW options DIRECTORY, ENVIRONMENT, TEMPDIR and VERSION

Fix bug #35532: Error setting PsppireDict predicate

MEANS: plug memory leaks

Attempt to cope with endian issues in zip reader.

ODS Reader: fix uninitialised variables

str: Skip only up to one-past-the-end in ss_tokenize().

If ss_tokenize() was called with '*save_idx' positioned at
one-past-the-end of a string, it would continue to advance it past
that point, which is surprising.  This avoids doing that.

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

sys-file-reader: Avoid read past end of buffer.

The "unexpected label source in mrsets" test caused a read past the
end of the input buffer in text_parse_counted_string().  This fixes
the problem.

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

tests: Avoid failure due to race to mkdir tests/valgrind.

categoricals: Mark dump_interaction() and categoricals_dump() UNUSED.

This avoids warnings that they are unused.

It might be better to just remove them, since they would still be
present in the Git history.

Revert "Mark separator names for translation"

This reverts commit 02c89b93ff0b5942e66fdfbb7562641a0ff29d2c.
Ben pointed out that this change was incorrect.

Remove erroneous case_unrefs. Found by valgrind

Use if(0) instead of #if 0

The Gnu Coding Standards recommend run time conditionals instead of preprocessor
conditionals.  I think this makes sense too, since #if 0'd code becomes obsolete
without anyone noticing.

categoricals.c: Fix memory access problems on empty interactions.

Make sure the categoricals behave correctly when passed
an interaction which contains no variables (ie, no catagories).

fix warnings

MEANS: Correct position of double vertical rule

Means: Report totals as well as catagories

Psppire: Allow syntax files to be provided as an argument

Reliability Test: Do not allow splits greater than the number of variables.

It's not meaningful to have a split point which is greater than the number
of variables (and not useful when they are equal).  And the system was
exhaughsting its memory when such a test was run.  This change therefore
disallows such a combination and updates the behaviour of the GUI accordingly.

Add missing translation markup

Use ButtonBox Mask nick field for it's correct purpose.

It seems that the GFlagsValue nick field was mistakenly being used as if
it was a translatable label.  Not only was this wrong, but it also created
unnecessary work for translators.  This change provides descriptions of the
flags and removes them from the translatable strings.

Mark separator names for translation

Added a dialog box for the MEANS command

examples: Change extensions from .stat to .sps.

.sps makes a lot more sense.

Reported-by: Harry Thijssen <pspp@sjpaes.nl>

tests: Improve test for supported encodings.

I misunderstood the purpose for the config.charset script.  I had a
notion that it was supposed to print all of the character sets that
are supported by the operating system.  It doesn't (it prints
character set names that need translation for the operating system).

This change switches to using the same code that will be used at
PSPP runtime to determine whether an encoding is supported, for
determining during the tests whether an encoding is supported.

This should cause fewer tests to be skipped on Mac OS.

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

perl-module: use $(MAKE) directly in rules that call make.

The GNU make manual says that $(MAKE) must appear directly in a rule
rather than through recursive expansion to work properly.  (This
change fixes a GNU make warning about an unavailable jobserver.)

Fixed bug #35494 - Levene test crash on no valid values.

Corrected typo in documentation

po: Update Slovenian translation from translationproject.org.

GET: Add an ENCODING subcommand.

For example, this allows a Swedish EBCDIC file that doesn't contain
any indication of its codepage to be read with "ENCODING='IBM278'".

sys-file: Support EBCDIC magic number in system files.

This makes PSPP able to read the 1980s EBCDIC-encoded system file
provided by Chris Muller <chris.muller@mullermedia.com>.

sys-file: Recode string fields in file header also.

Chris Muller <chris.muller@mullermedia.com> provided an example
EBCDIC-encoded mainframe system file from the 1980s.  This commit
goes one step toward being able to read and write these files in
PSPP, by recoding most of the string fields in the file header.

dictionary: Truncate file label in dictionary encoding, not UTF-8.

The limiting factor is bytes in the on-disk format and so there's
no point in truncating in terms of the in-memory format.

por-file-writer: Avoid infinite loop with value labels.

Bug report and fix by Kent Nassen <knassen@umich.edu>.

gui: Fix warnings due to missing GObject type conversions.

jonckheere-terpstra: Declare variables at beginning of block.

NPAR /J-T correct significance calculation when test statistic is greater than zero

gui: Fix grammar in user message in psppire-data-window.c

Reported-by: Harry Thijssen <pspp@sjpaes.nl>

npar.c: Added some missing initialisers.

Also reorder the initialisers to the same order as their declarations

Added an implementation of the Jonckheere-Terpstra test

Add Slovenian translation from translationproject.org.

This translation is by Matej Urbančič <matej.urban@gmail.com> and
the Slovenian translation team.

po: Update Dutch translation from translationproject.org.

po: Update German translation from translationproject.org.

Logo rework.

From work contributed by bojo42: Reworked the transparent part of the logo.
Also "harmonised" the fonts used in the logo and the splash screen.

New abstract base class: PsppireWindowBase

Created a new abstract base class for windows/dialogs and moved the
psppire_conf manipulation to there.  Should fix problems reported by bojo42
where the window geometry wasn't always correctly saved.

Increment version to 0.7.9 to send to Translation Project.

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