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.

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.