Drop macro set from most function arguments.

macro: Separate macro_expander from macro_call.

Rename macro_expander to macro_call.

Move segmenter_mode inside macro_expander.

DEFINE

lexer: Move lex_ellipsize() into string module, as str_ellipsize().

token: Update functional interface and add token_copy(), token_equal().

These will have users in an upcoming commit.

segment: Ignore !ENDDEFINE in /*comments*/ and "strings".

segment: Distinguish snippets from full files.

The comment on segmenter_init() explains what this means:

If IS_SNIPPET is false, then the segmenter will parse as if it's being
given a whole file.  This means, for example, that it will interpret -
or + at the beginning of the syntax as a separator between commands
(since - or + at the beginning of a line has this meaning).

If IS_SNIPPET is true, then the segmenter will parse as if it's being
given an isolated piece of syntax.  This means that, for example, that
it will interpret - or + at the beginning of the syntax as an operator
token or (if followed by a digit) as part of a number.

lexer: Factor out scan error messages into new function.

message: Make msg_emit() take full ownership of its argument.

The way it treated the argument before was just confusing.

message: Break message location out into a separate struct.

This will make it cleaner to have a stack of locations for use in reporting
macro expansion errors.

message: Get rid of 'shipped' member in struct message.

It seemed to me that it wasn't a very clean design, since it required a
message to be modified as part of emitting it.

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.

lexer: New function lex_next_representation().

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

doc: Fix operator precedence chart.

Also, improve a few entries.

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

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.