Cruft removal

Use u8_cpy instead of memcpy.

Use u8_cpy instead of memcpy since the
former provides better type checking.

Merged changes from branch interaction-review

Remove commented code

remove unused return value

Allow ranges with left to right and bottom to top sense to be selected.

Replace safe_strcmp with g_strcmp0.

Fix clipboard export of non-ascii strings.

Fixed problem pasting sheet data from psppire to
openoffice.org and gnumeric which contained non-ascii
characters.  Tested with openoffice.org 2.4 and
gnumeric 1.8.3

Fix bug in casemapping long strings.

Thanks to Ben Pfaff for pointing out the error.

Remove the RESIZE feature of the sheet.

This feature is currently unused, and probably
broken.

Re-enable signal emmission on row/column selection

Removed unused function

Removed unnecessary variable and consequently unreachable code.

Avoid the need for an extra click when de-selecting a region

Re-enable selection of complete rows/columns

Remove unused function psppire_sheet_cell_get_state

Prevent var sheet from traversing to an invalid cell

Redraw affected areas after de-selecting a region

Redraw selection when reducing its area.

This change ensures that the area of the selection
is properly indicated when it's in the process of
being decreasing its area as well as increasing it.

Replaced GTK_STATE_NORMAL with PSPPIRE_SHEET_NORMAL where appropriate

Removed unnecessary variables

Made selection a bit more reliable

Remove unnecessary variable

Partial fix of selection in the sheet.

This change re-enables the display of sheet
selections.  Unfortunately, the selected cells
aren't properly de-selected (yet).

Whitespace changes for the benefit of xgettext.

It would seem that xgettext wants the TRANSLATORS
comments to be on the line immediately preceeding the
translatable string.

Disable extend_selection call.

Temporarily disable the call to psppire_sheet_extend_selection
since it interferes with the active cell border and selection
is currently not working anyway.

Add "dictionary" property to PsppireVarStore and use it.

Instead of abusing the encapsulation of PsppireVarStore,
by accessing the internal dictionary member directly, add
a "dictionary" property.   This should also take care of the
reference counting of the dict, which wasn't working properly.

Change signature of text_to_value.

Now, text_to_value initialises the value, and expects
the caller to destroy it.

This fixes some gui bugs where strange things happened with long string
value labels and missing values.

remove return values from callbacks which don't need them

Fix warnings in flip command.

Use data_in / data_out to convert between names and values,
instead of doing it ourselves.

Remove reference to roc.h which doesn't exist.

Merge commit 'origin/stable'

Conflicts:

src/language/stats/t-test.q

Fix merge problems

Merge commit 'origin/roc'

Add perl functions to get the format of a variable

Remove double semicolons.

Replace caseproto_clone with caseproto_ref

Also unref the proto on destruction of the translator.

Added more (hopefully usefull) comments

Add some comments and macros to make the code more readable

Add assertion to check code consitency

Fix cleanup of ROC command.

Properly deallocate variables, and use correct
symbols for parser return values.  Also, delete
roc.h which is unnecessary.  Thanks to Ben Pfaff
for pointing out these problems.

Respect the constness of caseproto.

New function caseproto_clone.  This means we
can clone a proto, then mutate it as we want.

Corrected spelling of "consolidate".

Thanks to Ben Pfaff for pointing out my mistake.

Added the '=' to the plot subcommand's documentation.

Thanks to Ben Pfaff for pointing this out.

Avoid compiler warning

Merge commit 'origin/data-encoding'

Conflicts:

src/language/dictionary/split-file.c

Add comment explaining the meaning of encoding to data_out

The length of the string is now not always the
same as the format width.

Thanks to Ben Pfaff for pointing this out.

Improve code to trim leading spaces from numeric output.

Ben Pfaff pointed out that the code to chomp the results
of formated doubles was no longer correct. This change
fixes that.

Use data_out_pool in crosstabs.q

We were erroneously allocating a buffer before
the size of the contents were known.  Using
data_out_pool avoids this problem.  Thanks to
Ben Pfaff for pointing this out.

Before recoding a variable's name, check that it
doesn't clash with an existing one.

Thanks to Ben Pfaff for pointing out this potential
problem.

pspp_linreg(): Use cache->n_coeffs to set the dimensions of sw, instead of
       cache->n_indeps, which may give incorrect dimension in the case
       of categorical variables. Fixes bug referenced in bug report
       26861.

Ensure correct behaviour when the state var is missing.

When the state variable is missing, then the entire
case is skipped.

Update documentation regarding missing values.

Explicitly mention that cases are excluded on a
listwise basis.

Fix ROC behaviour in the presence of missing values.

Make sure that the ROC command's behaviour is correct,
when missing values appear in the result variable.

Corrected typos in the perl documentation

i18n: Merge translatable strings.

"Sig." is an abbreviation, "Exact" is not.

i18n: Eliminate some translatable strings.

i18n: Reduce translatable strings in SHOW command.

All the output from the SHOW command is of the form "%s is %s." but the
translators were being asked to translate similar strings over and over
again.  Reduce their load by getting rid of many translatable strings.

i18n: Change some strings to reduce work of translation.

PSPP has a number of strings that happen to be phrased differently for no
particular reason.  This commit changes some of those strings to be
exactly the same as other ones, to make the work of translators easier.

Separate table functions that format their arguments from those that don't.

The tab_text, tab_joint_text, and tab_output_text functions, until now,
had an option bit TAT_PRINTF that specified whether they passed their text
argument through sprintf.  This interface was bad because it made it
impossible for GCC to tell whether it needed to verify a printf format
string or not.

This commit solves the problem by breaking each of these functions into one
that does format its argument and one that doesn't.

Consistently capitalize the name "Gnumeric".

Thanks to Harry Thijssen for pointing out the inconsistency.

Fixed bug reporting the significance of paired value t-test.

Due to a cut and paste error, the significance of paired
t-tests was wrong when the value of T was positive.
Thanks to Mike Griffiths for reporting this problem.

New function prepare_cutpoints

Move the code which creates the cutpoints into its own
function.  This makes for easier reading IMO.

Updated the example with an easier to visualise one

Replaced the glade definition of about dialog with a C one.

The about dialog box is simple enough to maintain in C
rather than using a glade definition.

Removed unused function prototypes

Fix crash in find dialog and make code less horrible.

Fix bug encodig missing value keys in gui

Fix crash on text import dialog

Remove write-only variable from interaction_case_data.

Fix compiler warning in test program

Updated the developers' manual to reflect the new situation

Added a dict parameter to data_in and dealt with the consequences.

The data_in function now takes a pointer to a struct dictionary,
which must be the dictionary with which the output value is
associated.  Data_in now ensures that the data of string values
is converted to the dictionary's encoding if necessary.

Return 0.0 for mean of a categorical variable. Fixes bug mentioned in bug report 26861.

Remove recoding in data_store.

It's no longer appropriate to perform recoding in the gui.
Instead, this is expected to be done in the backend.

Change union value type to contain uint8_t types instead of char.

Make the members of the union value type in src/data/value.h be
uint8_t instead of char.  This is more logical since the contents
of values cannot be considered "strings" until they have been
formatted.  The unformatted values are merely arrays of bytes.

This has the added advantage of provoking compiler warnings when
a char * type is being implicitly cast to a uint8_t * or vici-versa.
When such a warning is encountered, is probably means that the
data needs to be re-encoded using recode_string.

Remove erroneously commited diagnostic statement

Use default encoding when reading system files if no encoding is given in file.

Fix problem running the perl module

Replace legacy_recode with recode_string.

Iconv seems to do a good job of converting between
ascii and ebcdic, so use the recode_string function
instead of our own conversion routines.

Fix compile warnings

Fix bug in value labels dialog box

Add dictionary argument to tab_value.

In order to properly display values, tab_value needs
to know the dictionary from whence the value comes.
This is necessary so that string values can be properly
decoded.

This change adds this argument to tab_value and updates
all callers.

Recode strings when writing system files.

The long variable names, variable labels and value labels are
now converted from utf8 to the dictionary encoding when
writing a system file.

Fix crash when opening empty dataset

Convert to utf8 in data_out function.

Previously, the output value of data_out was of arbitrary encoding.
This change attempts to ensure that it is always utf8.

data_out function to dynamically allocate return value.

Preparation for i18n of values.  Instead of asking the
caller to prepare the buffer for output, data_out now
dynamically allocates the output value, and expects the
caller to free it.  This is necessary since for utf8
strings, the caller cannot reasonably know the length of
the required output buffer.  It also simplifies some uses
of data_out.

Change enum legacy_encoding to const char *.

Preparation for i18n of union values.  Remove the
legacy_encoding enum and substitute it with a const
char *.  This makes it easier to integrate recoding
of union values in the data parsing stage.

Store variable names, labels and value labels as UTF8.

This change converts long variable names, variable labels
and value labels to utf8 encoding when system files are
loaded.  It is therefore no longer necessary (nor correct)
to convert them when displaying.

Fix bugs when input data is repeated

Added second ROC test

sys-file-reader: Don't warn if compression bias field is 0.

Some software writes a value of 0 to the compression bias field.  We
expect that this field has a value of 100 and warn if we see any other
value, but there is no real need to do so (because it is very likely
that we can read the file correctly in any case).  So suppress the
warning if we see a value of 0 here, because it bothers users.

Thanks to Tony Reardon <tony@sir.com.au> for reporting this
unnecessary warning.

por-file-reader: Tolerate short lines when detecting portable files.

When the portable file reader reads a file, it assumes that a line
shorter than 80 bytes should actually be padded out on the right with
spaces, because this is a fairly common problem in practice, perhaps
due to text editors or other software that drops spaces at the end of
a line.

However, pfm_detect, the function that is supposed to detect whether
a given file is an SPSS portable file, did not apply this heuristic to
the data that it read at the beginning of the file, and thus files in
which the first few lines were truncated this way were not detected
properly as portable files.

This commit fixes the problem by making pfm_detect a little bit
smarter.  It would probably be better to actually unify the
file-reading logic, instead of implementing it in two separate places,
but this appears to work adequately too.

Thanks to Tony Reardon <tony@sir.com.au> for reporting the problem
and supplying a file that demonstrated it.

Renamed interaction_variable_get_var to interaction_get_variable.

Renamed interaction_variable_get_member to interaction_get_member.

Split update_hash_entry into update_hash_entry and
update_hash_entry_intr for interactions.

inner_intr_loop: New function.

covariance_accumulate_pairwise: Loop separately over variables, then interactions.

interaction_variable_create: Make interactions type alpha when
appropriate.

interaction_value_create: Use value_resize to avoid copying more data than
necessary into new interaction_value.

Add new functions to define subcase orderings.

Allow subcases to be defined from a index and width,
rather from a variable.  This avoids much of the
need for var_create_internal.

sparse-xarray: Add missing #include <limits.h>.

Thanks to michel <michel@cecaps.ufmg.br> for reporting the problem.

Allow variables created by var_create_internal to have any width.

Until now, var_create_internal has always created a numeric variable.
In the long run we wish to phase out the use of internal variables
entirely, but this change should help Jason get some work done in the
short term.

Fix compile warning