Merge "master" into "psppsheet" to obtain bug fixes from "master".

PsppireVarStore: Fix memory leak

PsppireDataEditor: No longer keep a reference to the data window.

psppire-window-base: Avoid memory leak in get_window_id().

g_object_get() returns a string copy, so the caller has to free it,
but gtk_widget_get_name() gets the same string without a copy.

Found by valgrind.

psppire-data-window: Destroy dataset from finalize.

One rule of thumb for "destroy" is that it should just unref
GObjects.  This commit fixes that up for
psppire_data_window_dispose().  Previously, this could not be done
because of a reference leak in psppire_data_window_finish_init()
that caused doing the dataset destroy in finalize to crash, but
that problem was fixed in the previous commit.

psppire-data-window: Avoid reference leak to PsppireDict.

psppire_var_store_new() takes its own reference to PsppireDict so
the original reference owned by psppire_data_window_finish_init()
wasn't getting tracked anywhere.

Fixed memory leak in cairo driver.

psppire-data-editor: Unref old data_window upon change.

Otherwise there's a reference leak.

Found by inspection; I haven't looked for clients that actually
change the data window after setting it.

psppire-data-window: Delete dataset when closing window.

When the user closes the window it means that he wants to close the
underlying dataset also.  Otherwise the dataset essentially leaks
(in theory it's still there in the list of datasets but in practice
that takes a DATASET ACTIVATE <name> WINDOW=FRONT command to show
it).

This also fixes a Gtk-CRITICAL and following crash in the following
scenario brought up by John Darrington:
1. Start psppire
2. File | Open and load x.sav (dataset1)
3. File | New | Data (a new data window will appear: dataset2)
4. Using the window manager close the window for dataset1
5. In dataset2:  File | Open and load x.sav

I'm not 100% certain of the reason that this fixes it but I think
that there's some lingering association between the dataset and the
window that this change eliminates, by eliminating the dataset along
with the window.

The code added to psppire_data_window_dispose() in this commit
should probably go in a "finalize" function instead but there seems
to be a separate problem with references to PsppireVarSheet such
that if this new code is in "finalize" then dataset_destroy() will
call dict_clear() which will invoke the callbacks for PsppireDict
which will try to access the destroyed window.  On the other hand,
with this code in "dispose" the window isn't fully destroyed yet so
while the same code path gets followed it isn't fatal.

psppire-var-store: Chain up to correct parent method from "dispose".

I'm pretty sure that "dispose" should not chain up to "finalize".

Merge "master" into "psppsheet" to obtain bug fixes from "master".

psppire-dict: Make PsppireDict not own its "struct dictionary".

I see two places where PsppireDict are created.  The first is in
psppire-data-window.c.  In this case, the PsppireDataWindow is
borrowing a dictionary owned by a "struct dataset" for use in its
PsppireDict.  When it destroys the PsppireDict, therefore, the
dictionary must not be destroyed because the dataset still owns it.
However, this is not what actually happens, and doing the
following:

1. Start psppire
2. File | Open and load x.sav (dataset1)
3. File | New | Data (a new data window will appear: dataset2)
4. Using the window manager close the window for dataset1
5. In dataset2:  File | Open and load x.sav

or similar will cause a use-after-free error.

The second use is in text-data-import-dialog.c.  This code does own
the struct dictionary, but it can simply free it itself at the same
time as the PsppireDict.

There is still some underlying issue with the above scenario,
because it still reports a GtkCritical, but valgrind no longer
reports a use-after-free error.

Reported by John Darrington.

gui: Fix Glib-GObject-WARNING when closing the active dataset.

The DATASET CLOSE command, when it acts on the active dataset, just
removes the active dataset's name, changing it to the empty string.
(This is the documented behavior.)  However, the GUI relies on
every dataset having a name, so this caused the following warning:

GLib-GObject-WARNING **: value """" of type `gchararray' is invalid
or out of range for property `id' of type `gchararray'

This commit fixes the problem by giving any unnamed dataset a name
after running syntax.

Fix compiler warnings

PsppireDataEditor: Add comment emphasising lack of final unref of old_vbox_widget

PsppSheetView unrealize: Call parent method before any other operation

If this is not done, then gtk_widget_get_realized returns true, during
the unrealize execution, leading to mutual recursion.

Delete module customentry

This is no longer used, so am removing it completely.

PsppireDataWindow: Deal with ui_manager in dispose routine

PsppSheetView resize handler:  Do nothing if the widget is not realized

This seems to avoid the occasional GtkCritical that I get.

Reduce the flicker when redrawing the toolbar and menubar.

pspp-sheet-view.c: Separate destroy method into finalize and dispose.

This still needs work to disentangle them completely, but I think this
is an improvement.

PsppireDataStore: Move datasheet_destroy from dispose to finalize

This actually destroys stuff, not merely unreffing.  So it belongs in finalize.

PsppireVarSheet: Keep reference to the return value of _get_ui_manager

I changed the var_sheett to keep a reference to the uim object which it
returns from the _get_ui_manager objects.  This avoids a problem where
the object was getting its last reference unreffed by another object.

PsppireVarSheet: Replace Destroy with Dispose

Also, use a statically allocated array for signal handler ids, since
it just complicates things to have a dynamically allocated one.

PsppireCellRendererButton: Replace destroy with dispose

The destroy method did nothing except drop references, so was
appropriate for dispose.

PsppireDataSheet: Reference handler improvements.

Changed the destroy method to dispose, because all it does is drop
references.  Also, keep a reference to the object returned by _get_ui_manager
since this seems to be causing issues elsewhere.

psppire-data-editor: Avoid ' modifier with g_string_append_printf().

On Windows, g_string_append_printf() does not understand the '
modifier, so use ds_put_format() instead, which does.

Reported by John Darrington.

po: Update Slovenian translation from translationproject.org.

Fixed crash in two sample npar tests with (paired)

This change fixes a bug where an assertion would
fire if a two sample npar test was given where
a variable appeared twice in a variable list.

Documentation: correct punctuation and grammar

Use text_aa instead of solid black for sheet grid lines.

Use the widget's text_aa graphics context instead of black_gc to give a softer
appearance (and to ensure they are visible for themes which use black as their
background).

New Slovenian localisation from  translationproject.org

Updated po/sl.po from http://translationproject.org/latest/pspp/sl.po

psppire-var-store: Remove.

PsppireVarStore used to provide a model to allow GtkSheet to display
a dictionary.  Now, it is just a superfluous wrapper for
PsppireDict, so this commit removes it in favor of using PsppireDict
directly.

goto-case-dialog: Avoid gtk critical for invalid case number.

I noticed this when Go To Case was invoked with an empty data
sheet, in which case the default case number is the invalid case
number 0.

gui: Speed up initial file load with many columns

This makes a huge difference in startup time when there are hundreds of
columns, by setting up the data sheet before showing the window that
contains it.

Files opened from File|Open were already opened this way.  I was a little
surprised that File|Open was so much faster than specifying a file on the
command line until I figured out the difference.

gtk-contrib: Remove GtkSheet-derived PsppireSheet.

This code is no longer used.

gui: Redo var sheet, data sheet, text import with PsppSheetView.

This commit replaces the existing variable and data sheets in the
GUI by new ones based on PsppSheetView.

Thanks to John Darrington and Michel Boaventura for helpful testing,
bug reports and suggestions.

Readme.Git: Update gnulib SHA to include fixes for windows tempfile issues

text-data-import-dialog: Allow user to choose input encoding.

Otherwise only automatically detected encodings can be used.

Debian bug #676371.
Reported-by: Gunther Furtado <gunfurtado@gmail.com>

Add ENCODING subcommand to several commands.

I believe that this completes support for encodings for data input
commands such as DATA LIST and GET DATA/TYPE=TXT.  Support is still
incomplete for data output commands such as PRINT, which confuse
byte offsets with character positions.

Debian bug #676371.
Reported-by: Gunther Furtado <gunfurtado@gmail.com>

line-reader: New library for reading a file line-by-line.

This library reads a file line-by-line in an arbitrary 8-bit or wider
encoding, without requiring the file to be recoded.  This will be used
in an upcoming commit.

PRINT SPACE: Fix missing fh_unref() on error path.

The first "return" in this function failed to unref the handle.

ext-array.c: Ensure that fseek is called before switching between read and write.

ANSI C requires that a file positioning function (eg seek) occurs before a switch
from reading to writing a stream (or vici-versa).  We had not been doing this.
This had caused problems which manifested themselves on Windows operating systems,
by mysteriously failing to write the stream (and thus truncating the datafile).
This change corrects this.

Thanks to Harry Thijssen and Henry Gong for their very valuable assistance tracking
down the cause of this problem.

Reviewed-by: Ben Pfaff

CORRELATIONS: Fixed bug displaying non-sqaure correlation matrices

When CORRELATIONS was run using the WITH keyword, the wrong index
was calculated for the columns/rows - hence the program crashed.
This change fixes that.

Reported-by: Ebel Magnin

Extract the independent samples gui from  t-test.ui into its own file.

This will benefit upcoming commits, since other (non t-test) guis are
very similar.

Regression Dialog: Convert to PsppireDialogAction object

T-TEST Independent Samples Dialog: Converted to a PsppireDialogAction object

Add SET TVARS option. Closes bug #31566

Minor documentation formatting changes

HISTOGRAMS: Fix bin width problems on large numbers of bins

Histograms: Rotate labels when there are lots of bins

Move comment to correct function

i18n: Set LC_COLLATE locale category based on system locale.

LC_COLLATE controls the sort order of file names in GTK+ file
chooser dialog boxes, so we should set this locale category so as
to honor the user's preferences.

In the standard C library, only a few functions honor LC_COLLATE:
strcoll, wcscoll, strxfm, and wcsxfrm.  PSPP doesn't use any of
these functions, so this commit should not affect PSPP's use of
standard library functions.

The libunistring manual only mentions collation rules in the entries
for a few functions: u8_strcoll, u16_strcoll, u32_strcoll,
u8_normcoll, u16_normcoll, u32_normcoll, u8_casecoll, u16_casecoll
u32_casecoll.  PSPP doesn't use any of those functions either.

Reported by "ajk-eis" <ajk-eis@rvhome.de>.

gui: Make lex_gtk_text_buffer_read() count bytes, not characters.

The N parameter to lex_gtk_text_buffer_read() is a count of bytes,
but the implementation treated it as characters.

Also fixes a memory leak, since gtk_text_iter_get_text()'s caller
is responsible for freeing the memory that it returns.

Reported by Alle <ajk-eis@rvhome.de>.
First investigated by John Darrington <john@darrington.wattle.id.au>.
Reviewed by John Darrington <john@darrington.wattle.id.au>.

lexer: Check that 'read' function in lex_source returns valid value.

A 'read' function may only read as many bytes as the size of the
buffer passed into it, but the caller didn't check, which made an
actual violation of this invariant hard to diagnose.

Frequencies.ui: Adjust widget packing properties.

Reported-by: Mindaugas.

frequencies.ui: Reformat using glade-3

configure.ac: Link PSPPIRE explicitly against gthread.

Seems to be required on OpenSUSE.

PSPPIRE does not make explicit use of threads, but it does use
GTimer.  Thus, it calls g_thread_init() because of the following
note in the documentation for GTimer:

    GTimer uses a higher-quality clock when thread support is
    available.  Therefore, calling g_thread_init() while timers
    are running may lead to unreliable results.  It is best to
    call g_thread_init() before starting any timers, if you are
    using threads at all.

Separately, the documentation for threads in Glib says:

    Calling g_thread_init() with a NULL argument is somewhat more
    relaxed.  You may call any other glib functions in the main
    thread before g_thread_init() as long as g_thread_init() is
    not called from a glib callback, or with any locks held.
    However, many libraries above glib does not support late
    initialization of threads, so doing this should be avoided if
    possible.

    Please note that since version 2.24 the GObject
    initialization function g_type_init() initializes threads
    (with a NULL argument), so most applications, including those
    using Gtk+ will run with threads enabled.  If you want a
    special thread implementation, make sure you call
    g_thread_init() before g_type_init() is called.

Taken together, it seems to imply that PSPPIRE should call
g_thread_init() before g_timer_new(), or the timer might be
unreliable because Glib would call it later for us.  But the
documentation doesn't seem entirely clear.

Reported by Mindaugas.
Bug #36396.

MEANS: Use var_to_string instead of var_get_name for factor variable display

Frequencies dialog: Add Mnemonics

range-tower-test: Fix test failures on 64-bit architectures.

Reported by 'gandalfar' on #pspp.

Independent T-TEST Dialog: Use PsppireValueEntry in the group value definition sub-dialog

Converted Factor Dialog to PsppireDialogAction class

Convert Rank Dialog to PsppireDialogAction

Unref GtkBuilder objects when they are no longer required

PsppireDialogAction: Destroy dialog after it has been run

PsppireSelector: Don't try to set callbacks on null model

Regression Dialog: Tidy up and comment.

Correct erroneous g_object_unref calls

commit e89158e8abd2dae27d985e3574eb5aa1d265fc66 placed g_object_unref calls after every instance
of *_set_model - In most instances this was correct, but not all.  When (for example)
gtk_tree_view_set_model (view, model) is called, it takes a ref to model, but it will then
unref it upon the next call to _set_model and/or when view is destroyed.
Unreffing the model ourself, is therefore appropriate only when we have created it ourselves.
This change fixes the few instances where this was not the case.

po: Update Catalan translation from translation-project.org.

po: Update Spanish translation from translation-project.org.

psppire-cell-renderer-button: Add "slash" property.

When "slash" is set to true, the button gets a diagonal slash drawn
across it.  This is ugly, but it matches what SPSS does for cases
that are filtered out.

psppire-cell-renderer-button: New cell renderer for GtkButton.

This allows one to put a clickable GtkButton inside a PsppSheetView row.

pspp-sheet-view: Add "fixed-height" and "fixed-height-set" properties.

The data sheet will use this in the "split view" to make sure that
side-by-side data sheets have the same row height.

pspp-sheet-view: Make pspp_sheet_view_stop_editing() public.

pspp-sheet-view: Improve look of sheet when there are few columns.

GtkTreeView always expands one or more columns so that the visible
columns fill up the entire width of the treeview.  This looks funny
in a spreadsheet-like view.  This commit changes PsppSheetView so
that, instead, extra space is left blanks instead of being filled
up with columns.

pspp-sheet-view: Optimize by making column header widgets lazy.

This is a major speedup when there are hundreds of columns.

pspp-widget-facade: New code to measure and render some GTK+ widgets.

There are two upcoming uses for this in the tree, both related to
rendering fake buttons.  First, creating, maintaining, and rendering
the column header buttons in a PsppSheetView is a huge cost when
there are hundreds or thousands of columns.  Using this facade code
to "fake" buttons makes the sheet much faster.  Second, this code
will be used to implement a cell renderer for GtkButton, so that
buttons can be put into PsppSheetView rows.

pspp-sheet-view: Add "special-cells" property to speed drawing many columns.

A "special cell" is a cell that is editable or activatable.  When a row
that contains a special cell is selected, the cursor is drawn around a
single cell; when other rows are selected, the cursor is drawn around the
entire row.

With the default of "detect" (the only mode previously supported), whether
a given row contains a special cell is detected automatically.  This is the
best choice most of the time.  For sheet views that contain more than 100
columns, an explicit "yes" or "no" improves performance.

pspp-sheet-view: Start editing upon button release, not press.

This makes the user interface more natural.

pspp-sheet-view: Support rectangular selection, column popup menus.

A "rectangular selection" is where you can click and drag to select
a rectangular group of columns and rows, like in a spreadsheet.

In this commit, rectangular selections don't interact well with
"quick-edit" mode where the first click on a cell starts editing,
because clicking in a quick-edit cell starts editing instead of
rectangular selection.  The following commit will fix the problem.

This commit also adds a gtk_widget_queue_draw() call to
pspp_sheet_view_update_rubber_band() that should really
invalidate less sheet area.

pspp-sheet-view: Edit cells on the first click by default.

With GtkTreeView it takes two clicks to edit a cell.  The first click
selects the row and the second click starts editing.  This isn't a
great user experience for a spreadsheet, so this commit enables
single-click editing.

pspp-sheet-view-column: Add support for tooltips on columns.

pspp-sheet-view: Fix rendering moving cursor left or right with keyboard.

I don't understand why this is not necessary for GtkTreeView.  Without it,
the cursor doesn't get properly redrawn after a left or right arrow
keystroke moves it.

pspp-sheet-view: Fix visual artifacts for sheet views > 65535 pixels wide.

gdk_draw_line() draws lines by passing the coordinates to XDrawLine(),
which in turn casts the coordinates to "signed int" as it formats them for
the X Protocol.  Thus, large coordinate values wrap around and cause odd
visual artifacts.

This fixes the problem by only calling gdk_draw_line() with coordinate
values that should actually be visible.

pspp-sheet-view: Improve scrolling performance.

Drawing dashed lines is very slow, at least on my computer.  This commit
changes them to solid, which makes scrolling in a large sheet feel much,
much more responsive on my system.

pspp-sheet-view: Reduce time and memory cost to O(1) in number of rows.

The standard GtkTreeView uses time and memory linear in the number of
rows in the tree.  When the model is set, it iterates over every row
and creates a corresponding GtkRbNode.  After that, GtkTreeView uses
the GtkRbNodes for many operations on the tree.

This commit adapts GtkTreeView to avoid the need to iterate over the
entire model up front, as well as the need to maintain data for every
row.  It makes the following major changes:

  - The tree model must now be just a list (GTK_TREE_MODEL_LIST_ONLY);
    that is, rows may not have children.

  - The height of a row in the view must be fixed (as with
    gtk_tree_view_set_fixed_height_mode()).  Column widths must also
    be fixed.

The following minor changes follow mainly as consequences of the above:

  - Expander arrows, tree lines, and level indentation are no longer
    supported, because these only make sense with rows that have
    children.

  - Separator rows are no longer supported.  (They are ugly in
    fixed-height mode because they occupy an entire row.)

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