psppire-dict: Return new var from psppire_dict_insert_variable().

An upcoming commit will introduce a user for the return value.

psppire-dict: Make auto_generate_var_name() public, and rename.

This function will acquire a few users in upcoming commits.

This commit renames it to have the same prefix as other public
functions in this file.

psppire-dict: Get rid of static var in auto_generate_var_name().

This isn't a big deal at the moment but I intend to add new users
of this function in future commits, so then it becomes more
important.

Allow dictionary 'var_deleted' callback to examine the deleted var.

The dictionary 'var_deleted' callback is the last chance to look at
a variable as it is getting deleted from a dictionary.  For some of
the information in the variable, it legitimately doesn't make any
sense to get it from the variable itself at this point, e.g. the
index into the dictionary or the case, since it no longer is in
a dictionary.  But for most of it, it's reasonable to get it from
the variable itself, but the callback doesn't provide any way to
do that, because it doesn't pass in the variable, just a dictionary
index that is no longer correct.

This commit changes the 'var_deleted' interface to pass in the
variable plus the information that can no longer be obtained from
the variable itself.  An upcoming change to the GUI will make use
of this.

This commit also adapts all the existing users to the new interface.

find-dialog: Change "Cancel" button to "Close" button.

This button doesn't cancel anything, it just closes the dialog box,
so "Close" seems like a better label.

gtkxpaned: Remove write-only variables.

psppire-buttonbox: Fix style in header file.

It's totally bizarre and unnecessary to define a macro in the
middle of a function prototype, so this commit moves it.

psppire-data-store: Use NULL for a null pointer.

psppire-data-store: Use PSPPIRE namespace instead of GTK+'s.

psppire-data-editor: Drop psppire_data_editor_set_font() redundancy.

set_font() recurses to a GtkContainer's children so
psppire_data_editor_set_font() does not need to also do so.

Revert "Fix DSO linking issue"

This reverts commit 34abd8c7de0cac8efeead7fb3848427fb944b5bd.
The author blindly posted the patch without actually testing
whether it was necessary.  It was not, because the libraries
in question were already linked against.  Thanks to John
Darrington for pointing that out.

Fix DSO linking issue

We must explicitly list every required library in case then linker is built
with default behaviour which prevents implicit DSO linking.

See this for the details on DSO Linking issues:

http://fedoraproject.org/wiki/UnderstandingDSOLinkChange

Signed-off-by: Peter Lemenkov <lemenkov@gmail.com>

PsppireDictView: Call gtk_widget_destroy instead of g_object_unref

GtkMenu is a toplevel window.  Therefore, gtk_widget_destroy is appropriate.

PsppireDialog: Added assignment for parent_class which it seems was forgotten

Replace regression.q with regression.c

Reviewed-by: Jason Stover

PspppireSelector: Prevent selection of disallowed entries.

We had been relying on the deselected state of the widget to prevent
selection with the "allow" predicate was false.   But clicking the
widget itself isn't the only way items can be selected (double clicking
the item for example).   This change adds a test at a lower level to
prevent selection of disallowed items.

i18n: Factor out encoding categories from GUI into common code.

It would make sense to use these encodings and categories elsewhere.

sys-file-reader: Remove obsolete comment.

The open_text_record() function does the recoding, so there's no
need for parse_long_var_name_map() to do any recoding of its own.

Examine dialog: Allow only numeric variables to be selected as the dependent variable

Examine gui: remove var-is-numeric predicate from dict-view

Since the factor variable can be string, this predicate is not appropriate.

EXAMINE: Implement MISSING=REPORT option

categoricals.c: Distinguish between missing values on the dependent and categorical variables.

Needed for upcomming change to the EXAMINE command.

Remove KMO and Bartlett statistics from bug check test.

Commit 6b351b28f47c1dfb1ce697eb50cd218b50122fd0 added a test for a bug
fix.  This test however included a display of a the KMO and Bartlett
test statistics, which in this case are unstable since the determinant
of the correlation matrix is close to zero.   These statistic are not
relevant to the bug being tested.  So this change removes them.

Documentation: Yet more markup

FACTOR: Fix crash with /ROTATION=NOROTATE.

Reported by abderrahim arif <arifabderrahim5@gmail.com>
at http://lists.gnu.org/archive/html/bug-gnu-pspp/2012-04/msg00001.html

spreadlevel-plot: Fix missing #include <stdlib.h>

Reported by GCC:

spreadlevel-plot.c: In function ‘spreadlevel_plot_chart_destroy’:
spreadlevel-plot.c:84: error: implicit declaration of function
‘free’
preadlevel-plot.c:84: error: incompatible implicit declaration of
built-in function ‘free’

Text Importer: Added filter to the file chooser.

Added a FileChooserFilter to the file chooser for importing text files.
Closes bug #36054

Corrected bug in chart rendering which caused the ordinate label to never get displayed

Correct typo in examine.ui

Examine: Added the /PLOT=SPREADLEVEL option

examine.c: Fix a seg fault which caused a crash when recovering from bad syntax

Explore Dialog: Replace add hoc functions with the new DialogAction class

Documentation: More markup

Documentation: New markup for subcommands

Documentation: Add a @pspp macro

Histograms: Fix the alignment case where little slack exists on either side of the data

Correct errors in histogram geometry calcs and add test

Fix crash on datasets with single values (again)

Readjust histogram parameters if on the first attempt, the number of bins is too large

Histograms: Improve the code calculating the bin ranges and comment

Simplify algebra

xrchart_write_scale: Use integer arithmetic

Examine: Print extreme values using the print format of the variable

Histogram calculations: Use integer arithmetic

There is no advantage in using floating point numbers in this code.

Boxplot: Remove function boxplot_draw_yscale

An identical function exists in a lower level, so use that instead.

Histogram plotting: (hist_draw_bar) Added label argument.

Added a boolean argument to indicate whether or not a histogram bar
should be labelled.  Labelling every bar leads to overprinted and
hence illegible labels.

Histograms: Put hard limit on the number of histogram bins

Histograms with less than one bin are not useful (and tend to cause crashes!)
Histograms with very large numbers of bins are also of little use, and cause
other implementation headaches.

Cairo charts: Move check for near zero values to inside of draw_tick

Whitespace changes only

Fix a crashes where histograms of a single datum were trying to be created.

Reported-by: Ben Pfaff.

Fixed yet another problem caused by use of abs where fabs is appropriate

Mark histogram legend strings for translation

Avoid floating point precision problems in chart scale

Some charts were being drawn with a tick mark at positions
close to but not exactly zero Eg: 2.187433298e-27
This change avoids this problem.

cairo-chart.c: new struct xrchart_axis

This allows a bit of refactoring.

Histograms:  Ensure that the bins are aligned to an odd number of half bin widths

This has the benefit that the centres of the bins are aligned to integer multiples
of bin widths, and thus the labels are shorter.  For example, instead of labels
-1.5 -0.5 0.5 1.5 2.5 we get -2 -1 0 1 2 which is more aesthetically pleasing.

Corrected previous commit

chart_rounded_tick: Avoid arithmetic problems with inputs close to zero

Update the BUGS chapter in the manual

Updated the documentation for EXAMINE

Improve the way in which histogram bin ranges are chosen

EXAMINE: Allow the /ID subcommand to take a non-numeric variable

Correct geometrical error plotting histograms

histogram.c: histogram_create now takes bin width instead of the number of bins

NEWS: Mention changes to EXAMINE

sort: Allocate space for sort buffers on demand, not in advance.

Each call to sort_create_writer() was allocating possibly a very
large amount of memory when in many cases only a small fraction of
that memory would actually be used.  This commit switches to
allocating a small amount of memory and gradually increasing the
allocation on demand.

Reported by John Darrington.
Bug #35887.

Relax NPAR SIGN test

Improve translator comments.

Apparently some of the TRANSLATOR comments were not clear enough. This
change makes it less easy for translators to become confused.

Fix compiler warnings

Whitespace changes only

examine.c: Rename xxx0 and xxx1 to something more sensible

ods-reader: Fix memory leak.

Found by valgrind.

Fix memory leaks in the new EXAMINE implementation.

EXAMINE: Complete re-implementation

This change is a complete re-implementation of the EXAMINE command.
It includes a few minor changes to the tests, to accommodate slight
changes in semantics (to bring it in line with other software).
Closes bug #30732

order_stats: New function order_state_accumulate_idx

Categoricals: Dont crash when getting non-existant user data

ods-reader: Fix write beyond end of buffer.

The compiler multiplies by sizeof *var_spec for us here, so doing
it ourselves writes past the end of the allocated space.

Tracked down with valgrind.

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

math/np.c: Change np_create to take 3 doubles rather than a struct moment1*

Recode Dialog: Stringify the variable label

Ensure that the generated syntax has quotes around the variable label,
in case the label happens to contain spaces or other metacharacters.

Fix GtkCritical in recode dialog.

Commit 2b4fb7e846d62f1a70656bb42a0f769363ed0832 changed the internals of PsppireVarView.
Unfortunately recode-dialog.c was directly accessing the implementation instead of
using the interface.  This change fixes that.

Reported-by: kai hansen <kaiowesen@gmail.com>

po: Update French translation from translationproject.org.

Regression: Fix bug where missing values in the dependent variable were not getting ignored.

Fixes bug #35758

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.