src/output/output-item.c (output_item_unref): Fix memory leak

src/output/output-item.h (outpt_item_ref): Add WARN_UNUSED_RESULT attribute

src/ouput/output-item.c (output_item_unshare): Use result of output_item_ref

src/output/pivot-table.c (pivot_value_ex_destroy): Fix memory leak

Windows/automake.mk: Use more portable code

* Windows/automake.mk (environment_dir): Avoid GNU Make extensions
* Windows/automake.mk (libgcc_dir): Avoid GNU Make extensions

Suggested-by: Ben Praff <blp@gnu.org>

lexer: Fix typo in comment.

Output drivers: Remove assertions on OUTPUT_ITEM_GROUP

This case was indeed not reached when run from the command line, but certainly
was when exporting from the output viewer.

Added rules to build nsis installers for w32

* Windows/AdvUninstLog.nsh: New file.
* Windows/MUI_EXTRAPAGES.nsh: New file.
* Windows/automake.mk: New file.
* Windows/pspp.nsi: New file.
* Makefile.am: include Windows/automake.mk.
* configure.ac: Add conditional for sizeof ptr.

Added a script to assist building for w32

* Windows/README: New file.
* Windows/build-dependencies: New file.

Refactor SET and SHOW commands so it's harder to forget implementing SHOW.

settings: Add macro settings.

settings: Use modern initializer style.

segment: Allow '.' as separate token with a line, for macros.

The description of macros just says that ^ and \ can't be used as macro
delimiters, so . is probably OK.

psppire-dialog-action-regression: Fix unused variable warning.

GCC said that 'ok' was unused because it's declared in the 'for' loop.

segment: Improve segmentation when DEFINE command has early terminator.

The DEFINE command should not have a command terminator before the final
!ENDDEFINE.  When it does, that ends the whole thing early.  The segmenter
didn't handle this well, instead treating everything up to !ENDDEFINE as
part of the command.  That's probably not the best way because it makes
for confusing error messages.  This improves the behavior.

doc/automake: Use correct syntax for configuring native build

Add w32 handler for "activate-link" signal

The default activate-link handler for GtkAboutDialog does not seem to work on
w32.  So this change implements a special one.

src/ui/gui/help-menu.c (about-new): Add signal handler

Remove unused module subcommand-list

tests/automake.mk: Fix building of test runners

For certain architectures, the order of libraries is important.  Dependent
libraries must appear before the libraries which depend upon them.

src/ui/gui/var-display.c (missing_values_to_string): Reduce scope of local variable.

src/ui/gui/psppire-output-view.c (psppire_output_view_new): Reduce scope of local variables.

src/ui/gui/psppire-dict.c (psppire_dict_set_name): Reduce scope of local variable.

src/ui/gui/psppire-dialog-action-regression.c (generate_syntax): Reduce scope of local variables.

src/ui/gui/psppire-data-store.c (psppire_data_store_get_reader): Reduce scope of local variable.

src/libpspp/float-format.c (extract_hex): Remove useless assignment.

Found by cppcheck.

src/language/stats/crosstabs.c (display_risk): Avoid warning found by cppcheck

command: DEFINE command is still unimplemented.

Commit ed109bf49821 ("lexer: Add support for macro punctuation.")
mistakenly changed this.

lexer: Add support for DEFINE...!ENDDEFINE.

This doesn't support the command itself, just the lexical analysis needed
for it.

lexer: Add support for macro punctuation.

These punctuation symbols can be used to delimit macro arguments, even
though they aren't allowed anywhere else in the language.

lexer: Add support for macro identifiers (that begin with '!').

segment: Fix segmentation of integer followed by "." at end of file.

Previously this was segmented as a single number containing a trailing dot,
but it should be an integer followed by end-of-command.

Mention Harry Thijssen in THANKS

src/ui/gui/help-menu.c (open_windows_help): Reimplement

This change reimplements the function open_windows_help, because:

1.  It didn't seem to work reliably.

2.  It seems overly complicated.

3.  It relied upon a binary called "wscript.exe" which according to
    a brief web search, many windows users consider a security hole
    and routinely remove it.

src/language/stats/glm.c: Fix compiler warnings

src/libpspp/message.c (prepare_diagnostic_information): Relocate locale_dir

Output Window: Fix sensitivity of Save button. Also add option for png and svg files.

src/ui/gui/psppire-output-window.c (on_combo_change): Simplify logic and use
 correct limit for number of filename extentions.

src/ui/gui/automake.mk [src_ui_gui_psppire_CFLAGS]: Compile for a GUI under w32

src/ui/gui/psppire.c: Fold long line

src/ui/gui/psppire-dialog-action-runs.c (generate_syntax): Rename 'text' to 'str'

src/ui/gui/find-dialog.c (ip -> case_iterator): Rename.

src/ui/gui/psppire.c (struct icon_size): Remove --- unused

src/ui/gui/helper.c (clone_list_store): Remove unused variable

Bump micro version to 1.5.3

Generate cosmetics for windows hosts.

On w32 hosts, generate the resource file and link it to the psppire
executable.  Also generate the pspp.ico which is associated with the
executable.

segment: Properly handle DO REPEAT in batch mode.

In batch mode, a line that begins with a non-blank starts a new command,
but the segmenter didn't properly support this.  Without this commit,
the following wasn't properly handled:

DO REPEAT #x=1
ECHO 'hi'
END REPEAT

This commit also adds a test against regression, which also checks that
a blank line to separate DO REPEAT from the repeated syntax works (it
did before too, but there was no test).

lexer: New function lex_force_int_range().

This simplifies parsing code that needs an integer in a particular range.

Fix bug where attempting to rendering an empty bar chart would crash

doc: Update documentation for SET command to match recent code changes.

q2c: Remove, since it is no longer used.

This was the only tool that needed the build system's C compiler, so also
remove the parts of the build system that supported that.

SET: Reimplement parser without q2c.

CROSSTABS: Reimplement parser without q2c.

Avoid useless warnings on stderr.

src/libpspp/i18n.c (create_iconv): New argument 'warn'

src/ui/gui/main.c: Add a startup tip for Windows

psppire-import-spreadsheet.c: Fix typo

src/output/spv/automake.mk: Add some explicit dependencies to help cross building

src/language/expressions/automake.mk: Fix build issues when cross building

gui: Include line number in error messages generated from syntax windows.

gui: Label unnamed syntax windows as "Untitled" in error messages.

Seems a little more user-friendly than just omitted a filename entirely.

Add syntax mode radio button to syntax editor window.

A number of users have reported problems with the syntax mode.  I am not
sure why.  This should allow them to select the syntax mode they actually
need to use, just in case.  The default should be unchanged (it is still
Auto).

pivot-table: Reduce size of struct pivot_value from 80 bytes to 40.

pivot-table: Reduce size of struct pivot_value from 96 bytes to 80.

format: Reduce size of struct fmt_spec from 6 bytes to 4.

format: Reduce size of struct fmt_spec from 12 bytes to 6.

DELETE VARIABLES: Fix crash when deleting and adding vars at same time.

Thanks to Ricardo Mejias for reporting the problem:
https://lists.gnu.org/archive/html/pspp-users/2021-03/msg00025.html

Improve error message for creating a new string var with COMPUTE or IF.

tests: Ignore .pspprc in a different way, to fix running TeX on Fedora.

On Fedora, the 'tex' program generates per-user formats in the user's
home directory.  With the way that atlocal previously set $HOME to
/nonexistent, this failed, because formats can't be found or created in
/nonexistent.  This fixes the problem by using a different way to avoid
making pspp look for .pspprc.

ascii: Always include <sys/ioctl.h> if we're calling ioctl.

This file included sys/ioctl.h if it was necessary to use the TIOCGWINSZ
constant, but in fact it's needed to call ioctl() at all, so this commit
moves the #include so that it's included whenever we use ioctl().

Bug #60069.
Reported by Jeremy Lavergne.

psppire-output-view: Remove more dead code.

psppire-output-view: Fix display of groups.

Bug introduced in commit 507ebaea3673 ("output: Make groups contain
their subitems, and get rid of spv_item.").

Reported by John Darrington.

psppire-output-view: Remove dead code.

/ui/gui/psppire-var-info.c (__set_property): Remove redundant test.

Found by cppcheck.

src/libpspp/zip-reader.c (probe_magic): Fix inappropriate return value.

Found by cppcheck.

src/language/stats/autorecode.c (cmd_autorecode): Remove redundant test.

Found by cppcheck.

src/data/sys-file-reader.c: (read_header): Remove redundant test.

Found by cppcheck.

src/math/wilcoxon-sig.c (count_sums_to_W): Avoid use of unsigned value.

Use signed long int, not unsigned long int.

Found by cppcheck.

spv-light-decoder: Use unsigned int, not int, for %x with sscanf().

Found by cppcheck.
Reported by John Darrington.

spv-writer: Avoid obsolete function asctime().

Found by cppcheck.
Reported by John Darrington.

spvbin-helpers: Properly handle parsing strings with no destination.

binary-parser-generator will pass a null pointer to spvbin_parse_string()
or spvbin_parse_bestring() for a string that should be parsed but not
stored.  In such a case, the implementation would dereference a null
pointer.  This fixes the problem.

This is only a theoretical problem because the grammars PSPP uses now
always store the strings that they parse.

Found by cppcheck.
Reported by John Darrington.

spv-light-decoder: Fix check for valid decimal point characters.

Found by cppcheck.
Reported by John Darrington.

spv: Add support for page breaks.

spv: Remove code that was just #if 0'd out before.

Commit 507ebaea3673 ("output: Make groups contain their subitems, and get rid of
spv_item.") should have removed this code, but it was overlooked.

output: Replace OUTPUT_ITEM_PAGE_SETUP by a new driver function.

OUTPUT_ITEM_PAGE_SETUP was a special case that only the Cairo and SPV
drivers could implement, and it could only come first in the output stream
anyway, so it didn't make much sense.  This commit gets rid of it in favor
of adding a new driver class function for setting the page setup.

output: Make groups contain their subitems, and get rid of spv_item.

Until now, the output subsystem has had two different ideas for output
items: struct output_item, which is the primary representation, and
struct spv_item, which represents an output item read from an .spv file.
The biggest difference, until now, has been that spv_item contains its
children, whereas output_item bracketed children inside open/close pairs.
This commit unifies them under output_item, making output_item adopt the
container abstraction.

output-item: Add basic support for visibility.

zip-reader: Make the zip_reader reference counted.

This will be useful in an upcoming commit.

zip-reader: Switch to a more usual error reporting mechanism.

Having the client pass in a string buffer was clever but it gave the
zip_reader some shared state with its client that will be difficult to
deal with in future use cases.

zip-reader: Fix file descriptor leak in zip_reader_create().

zip-reader: Use endian conversion functions from integer-format.h.

DEBUG EVALUATE: Use output_log() instead of printf().

I found that with upcoming changes to the output engine, the relative
ordering of output from printf() and from the output engine changed.  This
commit makes it all go through the output engine, which keeps it
consistent as the output engine changes.

spv-legacy-decoder: Make 'data' parameter to decode_spvdx_table() const.

output: New function output_log().

llx: Introduce new iteration macros and some users.

str: Avoid passing null pointer to memcpy() in ds_put_substring().

pspp-output: Fix typo in user message.

output-item: Allocate correct size in message_item_create().

configure: Require @clicksequence to be present.

Occasionally someone tries to build with a really old Texinfo.  It doesn't
work now, although it once did, but before this commit it would
successfully configure.  By preventing configure from finishing, I think
that this will produce a clearer error message earlier in the build
process, ultimately making life easier for users.

Bug #59818.

output: Add missing chart.h.

Fixes: 29917c4f5908 ("output-item: Collapse the inheritance hierarchy into a single struct.")

output-item: Collapse the inheritance hierarchy into a single struct.

When I designed this inheritance hierarchy, what was really needed was a
bit of a mystery.  Now, it's pretty clear what is really going to be here,
and it simplifies and clarifies code a lot to just have a struct with an
embedded union.  It also removes, net, about 1000 lines of code.

output-item: Make command name part of every output_item.

Until now, group_items and text_items have had a command name, but SPV
files make the command name part of every output_item.  This change
make output_item better aligned.

cairo-fsm: Remove dead code in xr_fsm_create().

This code can't be reached because code above it exits the function in the
case that it handles.