improve macro error messages

tons of progress on macros

fixup for TITLE/SUBTITLE commit

stringi-set: New functions for not necessarily null terminated strings.

TITLE and SUBTITLE: Don't treat an unquoted argument as a quoted string.

This will allow the argument to be processed through the macro processor.

!BREAK works

fix string lexer

Mostly documentation update but some code clarification too.

segment: Add support for segmenting "snippets".

Otherwise, segmenting a little bit of syntax like "-123" will produce
T_ENDCMD, T_POS_NUM.

lexer: Factor some token inspectors out into new token functions.

!LET

more operator precedence

test !NULL equality

logical operator bug fixes

macro conditional expressions (no tests yet)

doc: Fix operator precedence chart.

Also, improve a few entries.

MNEST

!*

!EVAL

!UPCASE

Implemented all the functions except !EVAL and !UPCASE.

Add test for !LENGTH.

macro: Implement uniformly in terms of character strings.

!BLANKS and !CONCAT pass basic tests.

Start writing documentation.

add some keyword tests

Keyword arguments work.

Basic tests for macros and arguments pass.

Some basic macros and tests work.

Work on DEFINE command.

identifier: Remove TOKEN_N_TYPES from enum token_type.

segment: Refine treatment of start of macro body.

Previously, if the first line of the macro body (the same line as the
closing parenthesis in the DEFINE) was blank, we reported it as a blank
line to the lexer.  The parser for DEFINE could check for that (by seeing
whether the first line of macro body was empty or all-spaces) but it seems
more elegant to do it in the segmenter.  This implements that.

segment: Fix implementation of is_all_spaces().

segment: The body of DEFINE does not have to be on separate lines.

Some examples of one-line DEFINE can be seen at:
https://www.spsstools.net/en/macros/macros-index/utilities/quote-string-on-the-fly/
such as:
DEFINE !client() ABC Inc !ENDDEFINE.

segment-test: Make debugging easier for some kinds of test failures.

token: Rename token_destroy() to token_uninit().

This better fits the convention that "destroy" functions free the argument,
whereas "uninit" frees only the members of the argument structure.

lexer: Remove lex_syntax_mode in favor of segmenter_mode.

These two enums have the same meaning and there was no benefit to having
both of them plus translation back and forth.

po/hu.po: Update to lastest version from translation-project.org

psppire-output-view: Avoid doubling output when exporting or printing.

Outputting a group outputs its contents, but the code here was also
separately outputing the contents, which effectively doubled it.  (PSPP
doesn't currently use nested groups but these would have led to additional
copies.)

This fixes the problem by only printing items that aren't nested inside
any group.

Bug #60464.
Thanks to John Darrington for reporting this bug.

box-whisker: Use consistent array size for box_whisker_hinges().

The prototype said the 'hinges' parameter has 2 elements but the definition
said that it has 3.  This fixes the prototype.

This is harmless because the C standard says these are actually equivalent.

Reported by GCC 11.

format-parser: Fix inconsistent use of "char *" vs "char[NUMBER]".

These are equivalent according to the C standard in function prototypes,
but GCC 11 warns about using them inconsistently between a prototype and
a definition.  This commit avoids the warning.

src/language/utilities/host.c (run_command): Fix memory leak

src/data/format.c (fmt_affix_free): Fix memory leak

src/output/tex.c (post_process_tokens): Fix memory leak

src/output/select.c (select_matches): Fix memory leak

src/output/spv/spv.c (spv_read): Remove unused variable

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