Import GtkTreeView from GTK+ 2.0-2.20.1 renamed as PsppSheetView.

This includes minimal modifications to make it compile and link
cleanly.

I used the following Perl script to do the bulk of the renaming:

for $pair (['tree_view_column', 'sheet_view_column'],
           ['tree_selection', 'sheet_selection'],
           ['tree_view', 'sheet_view'],
           ['rbtree', 'rbtree'],
           ['rbnode', 'rbnode']) {
    my ($from, $to) = @$pair;

    my ($tfrom) = join ('', map ("\u$_", split ('_', $from)));
    my ($tto) = join ('', map ("\u$_", split ('_', $to)));

    s/gtk_$from/pspp_$to/g;
    s/\Ugtk_$from/\Upspp_$to/g;
    s/GTK_TYPE_\U$from/PSPP_TYPE_\U$to/g;
    s/GTK_IS_\U$from/PSPP_IS_\U$to/g;
    s/Gtk$tfrom/Pspp$tto/g;
}

s/GtkTreeCellDataFunc/PsppSheetCellDataFunc/g;
s/GtkTreeDestroyCountFunc/PsppSheetDestroyCountFunc/g;

PsppireValueEntry: Simplify 'if' condition.

The statement
  if ((model != NULL) && (model != old_model))
can be simplifed to just:
  if (model != NULL)
because if 'model' is nonnull then it is different from old_model,
because 'model' is a newly allocated GtkListStore.

psppire-dialog-action: Remove unused variable.

gui: Put <?xml> declaration at beginning of roc.ui.

xgettext objected to the blank line at the beginning.

gui: Properly manage ref counts of combo box and tree view models.

gtk_combo_box_set_model() and gtk_tree_view_set_model() add a
reference to the passed-in model, instead of transferring a
reference from the caller.  A lot of the calls to these functions,
however, assumed that a reference was transferred and therefore
leaked a reference.  This fixes them up.

This doesn't change PsppireValueEntry, because there's an ongoing
discussion about that.

psppire-acr: Clarify acr ownership of its list store.

PsppireAcr retains a pointer to the GtkListStore inside it but it
didn't, until now, keep a reference to it.  This commit makes
PsppireAcr retain a reference, documents the interface, and updates
callers to properly maintain reference counts.

ROC Dialog: Use PsppireValueEntry for the state value entry

Add PsppireValueEntry to the list of pre-declared widgets

PsppireValueEntry: Unref model after the combo box assumes it

PsppireValueEntry: Remove unnecessary function calls

val_labs_equal: Accept null pointers

Instead of checking for NULL before calling val_labs_equal, put the onus
on the function itself to deal with null pointers and behave appropriately.

PsppireValueEntry: Clear the entry, when the underlying model changes.

Since the labels will have changed, the existing text is most likely
inappropriate.

PsppireValueEntry: Remove GNU_CONST declaration because it prevents the type being found by GtkBuilder

crosstabs-dialog: Line up backslashes.

tests: Fix reference to old library libpspp.la.

Reported by John Darrington.

Improve Independent Samples T-test dialog

PsppireButtonBox: Remove Close from the default set of buttons

helper: Move escape_underscores() here.

An upcoming commit will introduce a new user in a different file.

gui: New widget PsppireValueEntry for editing a "union value".

This is intended to be a convenenient way to allow the user to
view and edit a "union value" that includes the ability to see
and select values by value label.

gui: New boxed type for struct fmt_spec.

This makes it possible to use a struct fmt_spec as the value of a
property on a GObject, which I plan to use in an upcoming widget
PsppireValueEntry.

gui: New type PsppireEmptyListStore.

PsppireEmptyListStore is an GtkTreeModel implementation that has a
client-specified number of rows and zero columns.  It is a useful model for
GtkTreeView or PsppSheetView when the client can easily synthesize cell
data using a callback set with gtk_tree_view_column_set_cell_data_func().
In that situation, GtkListStore can be wasteful (because it uses a lot of
memory to store what does not need to be stored) and situation-specific
custom models require additional boilerplate.

This commit also modified the text data import dialog code to use
PsppireEmptyListStore.  Future commits will add other users.

gui: Add undocumented --measure-startup option.

This is useful for testing the effects of code changes upon the
time required to start PSPPIRE.

New "range-tower" data structure.

A range tower is a bitmap, implemented as an augmented binary tree.

Beyond the usual features of a bitmap, a range tower can efficiently
implement a "splice" operation that shifts ranges of bits left or right.
This feature does cost memory and time, so use a range tower only if this
feature is actually needed.  Otherwise, use a range set (see range-set.h),
which can do everything that a range tower can do except the "splice"
operation.

Each operation has O(lg N) cost, where N is the number of contiguous
regions of 1-bits in the bitmap.  Also, a cache reduces the second and
subsequent containment tests within a single contiguous region to O(1).

range-set: New macro RANGE_SET_FOR_EACH to make iteration easier.

range-set: Rename "insert" function "set1", "delete" to "set0".

An upcoming commit will introduce a new "range-tower" data
structure that can actually insert and delete ranges.  It
makes sense not to use different terminology for the two data
structures, so renaming these functions makes sense.

abt: New function abt_is_empty().

abt: Drop child parameters from 'reaugment' function.

The function can simply inspect the 'down' members.

value-labels: New function val_labs_find_value().

The new data sheet implementation (in an upcoming commit) will use
this.

helper: New function value_to_text__().

One potential caller of value_to_text() doesn't have a "struct
variable" but does have the other needed information.

format: Fix typo in comment.

struct fmt_spec's 'd' member is not normally a number of "implied"
decimal places.  It is normally the number of decimal places that
will be presented in formatted values.  Thus, update the comment.

format: Introduce a new type "enum fmt_use".

This seems slightly nicer than using a "bool" to distinguish
input and output formats.

format: New functions fmt_change_width(), fmt_change_decimals().

These will be used in an upcoming commit for GUI-based format
adjustments, where we want to implement the user's request for
width or decimal places to the extent that we can, while
adjusting decimal places or width, respectively, as necessary.

gui: Use canonical names for signals.

Hyphens are canonical in parameter names, at least, and they seem
to be used consistently in signal names within GNOME, so use them
here too.

DO IF test:  Correct expected outcome

Commit 27bd188495ec1a6085354b10545daba47771b517 added a test for an
error condition in DO IF, but appears to contain some cut and paste
errors.  This change corrects them.

SHOW: New subcommand N

This subcommand reports the number of cases in the active dataset.

Revert "psppire-dict: g_return_if_fail for idx in psppire_dict_get_variable()."

This reverts commit edf4f6b1f7e2b9c17cc86f1c63ff9b4b7e24af26.
John pointed out that it causes the GUI to crash at startup.

DO IF: Fix crash when END IF appears without DO IF.

Reported by Jill Covert.
Bug #36231.

psppire-dict: g_return_if_fail for idx in psppire_dict_get_variable().

Also, as long as we're checking for too-big we might as well check
for too-small too.

Correct bugs in PsppireSelector which cause the Examine Dialog to crash

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