pivot-table: Fix format specifiers.

Found by GCC.

Fixes: 5cab4cf3322f ("output: Introduce pivot tables.")

driver: Fix format specifiers and avoid fixed-size buffer.

The format specifier problem is real.  The fixed-size buffers should not
be, but it is not worth arguing with GCC.

Fixes: a1b7813a6993 ("output: New page-setup-item.")

por-file-writer: Use xasprintf() to avoid careful buffer sizing.

GCC 4.9 thinks that this string can overflow the buffer and there's no
reason not to use xasprintf() here.

por-file-reader: Use xasprintf() to avoid need for careful buffer sizing.

GCC 4.9 complained about this, saying that the output could exceed the
buffer size.  It doesn't seem worth arguing with it.

ods-reader: Use proper function type for xml read callback.

The difference between size_t and int is not important on 32-bit, but it
makes a bit difference on 64-bit.

pivot-table: Fix type mismatch in equal_indexes().

This function was being called with an 'unsigned int *' but it took it
as a 'size_t *', which worked fine on 32-bit systems but caused an
enormous number of tests to fail on 64-bit systems.

Thanks to Alan Mead and John Darrington for assistance.

Fixes: 5cab4cf3322f ("output: Introduce pivot tables.")

tests: Get rid of temporary debugging fix.

Fixes: c70c92497244 ("SYSFILE INFO: Get rid of nested tables.")

table: Avoid initializer warnings with GCC 4.9.

psppire-output-window: Avoid using options the ascii driver no longer has.

Fixes: 1a4052ac93c4 ("ascii: Get rid of the idea of pages.")
Fixes: 37a14e6b8ab9 ("output: Add support for fonts.")

configure: Entirely avoid broken Texinfo.

Texinfo 4.13 was still causing problems on SuSE despite the implemented
workarounds, so this commit removes the workaround in favor of just using
the distributed pspp.xml if makeinfo is broken.

Requested by opensuse.lietuviu.kalba <opensuse.lietuviu.kalba@gmail.com>.

psppire-output-view: Only rerender for real changes.

Rerendering is expensive and slow, so only do it when there's a real change
like font size, not just when the cursor moves to a different window.

psppire-output-view: Add a little extra space to GtkLayout.

It can be difficult to see the bottom and right sides of the GtkLayout
because of the scroll bars that, on newer versions of GTK+ 3, seem to
actually overlap part of the content.  Add a little extra space around
the output items to make it clearer.

cairo: Make xr_rendering_measure() tolerate NULL arguments.

output-item: Add 'name' member to class structure.

This occasionally makes debugging easier.

string-map: New function string_map_equals().

output: Fix XML parsing in output_get_text_from_markup().

The XML parser requires the entire input to be enclosed in an element.

pspp-dump-sav: Issue error message for too-large extension records.

CVE-2018-20230.

pivot-output: Fix argument order that made lots of weird rotated headings.

Fixes: 5cab4cf3322f ("output: Introduce pivot tables.")

output: Introduce pivot tables.

cairo: Minor cleanup by factoring out new function set_source_rgba().

output: Support decimal and mixed alignment,

output: Rename a few struct members for consistency.

output: Support rotation.

table: Make table cells have exactly one piece of content again.

output: New page-setup-item.

output: Add support for Pango markup of fonts and styles.

psppire-output-view: Render only the visible parts of tables.

This greatly improves scrolling performance for large tables.

output: Cache the script ltr versus rtl direction.

A profiling run showed that about 10% of the rendering time was occupied
with repeated calls to render_direction_rtl().  This caches the result.

psppire-output-viewer: Make sure that initial items are rendered at show.

When output items were shown in the output window, the first item
was not shown until the user moused over it and thereby triggered the
on_style_updated callback.  This fixes the problem.

psppire-output-view: Fix background color for output viewer.

psppire-output-view: Add titles for groups.

cairo: Render group_open_items.

tutorial: Fix duplicate definition of node name "examine".

Emacs, at least, confused this "examine" with the "EXAMINE" node for the
EXAMINE command, and prevented me from visiting the latter documentation.

tests: Tag all the tests that take more than about 1s as "slow".

This makes it possible to run a quicker set of tests, which still have
pretty good coverage, with something like this:
    make -j10 check TESTSUITEFLAGS='-j10 -k !slow'

table-stomp: Remove dead code.

Nothing used this code anymore.

cairo: Improve comment.

cairo: Fix word-wrapping of ".000" and similar.

The Unicode line-breaking algorithm (or perhaps Pango's implementation of
it) is willing to break after the period in ".000".  This looks odd.  This
commit works around the problem.

output: Introduce group_item in place of TEXT_ITEM_COMMAND_OPEN and CLOSE.

This allows groups to be nested.

text-item: Merge "title" and "subtitle" items into a new "page title".

This better matches SPSS behavior.

cairo: New macros XR_LINE_WIDTH and XR_LINE_SPACE.

This makes the code a little easier to read in places.

driver: New function output_driver_parse_option().

This moves this function out of terminal-opts.c so that other code can use
it too.

output: Add support for alternating row colors.

output: Add support for layer info in table output.

output: Implement styling for titles and captions.

output: Implement styling for footnotes.

cairo: Remove debug print.

tab: Support cell-specific styles.

output: Add support for colors for rules.

output: Support styles for text items.

output: Add support for fonts.

output: Add support for cell margins.

Until now, space between cells was implemented by adding some space on the
sides of rules.  That worked fine before cells could have colored
backgrounds, but now it means that the backgrounds don't reach all the way
to the rules.  Thus, this commit adds configurable margins to the cells
themselves.

output: Add support for colors in cells.

Background colors aren't painted very well until the following commit adds
support for cell margins.

output: Add support for dashed, thick, and thin rules.

output: Add support for vertical alignment of cells.

It makes sense to bottom-align top headings but to top-align data cells,
for example.

tab: Expand space for table options from 8 to 16 bits each.

Upcoming commits will need the extra bits.

testsuite: Enable color output for tests.

render: Update style of footnote rendering.

output: Expand footnote support.

This adds support for multiple footnotes per table cell and the ability
to add footnotes to titles and captions.

format: New function fmt_from_u32().

This functionality will be useful for reading SPV files as well as SAV
files.

string-map: New functions for searching by non-null terminated keys.

The first user will be added in an upcoming commit.

str: New functions ss_starts_with() and ss_find_substring().

These will have their first users in an upcoming commit.

integer-format: Add some useful helper functions.

These functions already existed in float-format.c, but they are more
generally useful, so this commit moves them into integer-format.h.

CROSSTABS: Make treatment of crosstab variables and values more uniform.

CROSSTABS: Rename pivot_table to crosstabulation.

This will make things easier later when real pivot tables are introduced at
a higher level.

output: Factor common code for command name tracking out of all drivers.

This reduces redundancy.

message: Refactor initializations for struct msg throughout the tree.

A new member will be added in an upcoming commit, and the new style of
initialization ensures that the member will always be initialized

variable: Make property function return untranslated versions.

This makes it possible for the callers to do the translation at the time
it is needed.  It will be particularly useful in an upcoming commit.

missing-values: New function mv_to_string().

This seems like something that should be handled by the missing value code,
not by SYSFILE INFO.

bt, abt: Add initialization and iteration macros.

dictionary: New function dict_get_weight_format().

Just a convenience.

ascii: Get rid of the idea of pages.

In the era when I started writing this code, people still printed output on
dot-matrix printers and fanfold paper.  That's no longer a realistic use
case: today, no one prints plain text output.  It's just for viewing on a
screen, and on a screen there is no need or desire to break tables up
vertically into virtual "pages".

table: Get rid of TAL_GAP.

This is only necessary in pathological cases.  Those cases don't really
come up in normal usage and as PSPP shifts toward pivot tables they won't
come up at all.

cairo: Get rid of useless parameter 'y' to xr_layout_cell_text().

categoricals: Convert tabs to spaces.

categoricals: Update style and modernize.

categoricals: Get rid of 'df_prod' array.

It brought minimal value in that everything it did could be recalculated
on the fly without increasing the big-O cost.

categoricals: Drop 'n_vals' member from struct variable_node.

It was redundant with counting the elements in valmap, which wasn't
used many places anyway.

categoricals: Add array of variable_nodes to reduce hash lookups.

categoricals: Avoid setting value_node indexes multiple times.

A given variable_node can be in more than one interaction, but it's only
necessary to set the variable_node's value_node indexes once each.

categoricals: Give "reverse" members more descriptive, shorter names.

categoricals: Remove unused function categoricals_get_n_variables().

This function output debugging info on stdout, so clearly it wasn't
important.

categoricals: Don't warn on hash collisions.

Hash collisions are not errors.

categoricals: Improve comments.

categoricals: Remove 'dep_excl' parameter.

It was not used for anything.

tests: Add "categorical" keyword to tests that use categoricals.

This makes it easier to run tests just for these.

interaction: Better document and update code to modern style.

output: Remove support for nested tables.

They are no longer used.

SYSFILE INFO: Get rid of nested tables.

Nested tables aren't really necessary and make a number of things harder in
the output subsystem.  SYSFILE INFO and DISPLAY, which share code, were the
only code that used them, so this removes them from that code.

get-data-psql.at: Make test work with Debian pg_wrapper arrangement.

It had been years since I tested Postgres, but I was making a change that
would affect the Postgres test, and I found out that the test didn't work.
This fixes it.

segment: Fix behavior when #! line is not new-line terminated.

The code here is supposed to maintain a invariant that, when it returns a
nonnegative value, it initializes *type, but it failed to do that when a
#! line did not end in a new-line.  This fixes the problem.

Bug #55101.
Thanks for Friedrich Beckmann for narrowing down the problem.
Found by the Debian buildd: https://buildd.debian.org/status/fetch.php?pkg=pspp&arch=arm64&ver=1.2.0-1&stamp=1543183214&raw=0

ascii: Fix assertion in ascii_output_table_item().

The previous assertion did not actually detect the bug it was intended to
cover.

ascii: Update min_breaks along with width and length.

min_breaks has to be no larger than the page width (or length), otherwise
the rendering algorithm fails, but the automatic update for TTYs didn't
take that into account.

render: Use "long long" instead of "double" for arithmetic.

When I wrote this code in 2010, it seemed reasonable to me to use
floating-point arithmetic to do large integer calculations, because 64-bit
machines were not yet pervasive.  Over 8 years later, 32-bit machines are
now a rarity.  This commit switches types and modernizes the coding style.

This fixed an assertion failure that I was getting mysteriously while
rendering a particular table.  I didn't investigate further because using
integers seemed like a much better idea anyway.

tests: Disable Address Sanitizer SIGSEGV handler for intentional SIGSEGV.

FREQUENCIES: Avoid buffer overrun when there is no valid data.

Reported by Address Sanitizer.

ONEWAY: Fix use of stack-allocated variable outside its scope.

The 'payload' object is a kind of class and should be allocated statically.

Found by Address Sanitizer.

render: Fix pathological case where column minimum width exceeds maximum.

data-out: Clarify the meaning of the 'encoding' parameter.

Signed-off-by: Ben Pfaff <blp@cs.stanford.edu>

psppire: Fix multiple definitions of align_enum_type and two others.

These were defined in both psppire-dict.c and widgets.c, which causes a
problem building with -fno-common (which is desirable because it allows
Address Sanitizer to work better).

Check for python interpreter at configure time and document the requirement.