lib/gvc/gvc.def: [nfc] strip trailing whitespace

implement next 'gvputs_nonascii' function

This is based on `fig_string`, `mp_string`, and `pic_string`, but outputting
data directly into the output file rather than a temporary buffer.

Related to #2051.

Merge branch 'smattr/E7C21AC3-3A42-4520-8BEC-C9657B7DC8F9' into 'main'

GDI+ plugin: some clean up and modernization

See merge request graphviz/graphviz!2307

GDI+ plugin: rewrite 'FileStream' reference counting to be thread safe

Based on Microsoft sample code,¹ it seems like the expectation is that consumers
of the `IStream` API may expect to concurrently manipulate reference counts.
This change guards against this by making the referencing counting code use
lock-free atomics, based on the same Microsoft sample.

Note that this is essentially proactively addressing a latent issue as it is, in
general, unsafe to use Graphviz libraries or plugins in a multithreaded
environment right now.

¹ https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ms752876(v=vs.85)

GDI+ plugin: [nfc] avoid directly modifying 'FileStream' refcount

This change redirects all reference count modifications through `AddRef` instead
direction increments. Based on Microsoft sample implementation.¹

¹ https://docs.microsoft.com/en-us/previous-versions/windows/desktop/ms752876(v=vs.85)

GDI+ plugin: [nfc] remove assumptions on lifetime of 'FileStream' filename

Avoiding an assumption on the lifetime of `us->name` provides one less thing to
think about when iterating on this code.

GDI+ plugin: [nfc] use a const pointer for filename

The `FileStream` class does not need to modify this internally, so this avoids
the need for sketchy type casts.

GDI+ plugin: use 'override' instead of 'virtual' on 'FileStream' methods

This C++11 mechanism is an improvement as it causes compilation to error out if
any of these methods were not already declared virtual in a parent class. That
is, it provides a sanity check that these overrides are really doing what they
intend.

GDI+ plugin: [nfc] remove legacy use of a C-style cast

GDI+ plugin: [nfc] strip trailing whitespace

Merge branch 'smattr/18D413B7-67B7-4F3B-A544-823A3810D062' into 'main'

fix missing author+date line in Debian changelog

See merge request graphviz/graphviz!2309

fix missing author+date line in Debian changelog

Commit 42d7c87c291781ccaff47126a5ad3a2469e310cc omitted this required
information, resulting in warnings like the following in the build log:

  dpkg-gencontrol: warning:     debian/changelog(l7): found start of entry where
    expected more change data or trailer

Merge branch 'smattr/2504A45F-EA4A-41EA-BF7F-E758CD4AE122' into 'main'

lib/neatogen clean up

See merge request graphviz/graphviz!2308

solve: [nfc] squash a -Wmissing-prototypes warning

lu_decompose: [nfc] squash -Wmissing-prototypes warning

lu_solve: [nfc] squash -Wmissing-prototypes warning

matinv: [nfc] squash -Wmissing-prototypes warning

find_intersection: [nfc] squash -Wunused-parameter warnings

tclpkg sgnarea: [nfc] make sign extraction more obvious to the compiler

Squashes two -Wfloat-equal warnings. Equivalent of the previous commit, but
applied to tclpkg code that looks copy-pasted from neatogen.

neaotgen sgnarea: [nfc] make sign extraction more obvious to the compiler

Squashes two -Wfloat-equal warnings.

neato_layout: [nfc] squash a -Wsign-conversion warning

`Pack` is known non-negative in this branch.

gvbisect: [nfc] rephrase an open coded 'fabs'

lib/neatogen/edges.c: [nfc] remove unnecessary casts

Merge branch 'smattr/9F6685FF-CBD7-4AEA-B566-EA006B3BC33D' into 'main'

remove unnecessary casting

See merge request graphviz/graphviz!2306

myiddisc_open: remove unnecessary casts of 'gctx'

graphcmd: remove unnecessary cast of 'interp'

SparseMatrix_import_matrix_market: remove unnecessary casts of 'val'

poppler plugin: remove unnecessary casts around 'us->data'

remove unnecessary casts on 'lt_symlist_t' members

rsvg plugin: remove unnecessary casts around 'us->data'

webp plugin: remove unnecessary casts when calling WebP functions

webp plugin: remove unnecessary casts around 'us->data'

gs plugin: remove unnecessary casts around 'us->data'

gtk plugin: remove unnecessary casts around 'job->context'

gtk plugin: remove commented out code

gd plugin: remove unnecessary casts around 'us->data'

pango plugin: remove unnecessary casts around 'us->data'

gdk plugin: remove unnecessary casts around 'us->data'

tcldot-graphcmd.c: remove commented out code

Merge branch 'smattr/2F6B4B43-6BDB-40C6-83D2-181D9E5CE03C' into 'main'

misc cleanup

See merge request graphviz/graphviz!2305

CMake: use a relative path to locate configure_plugins.cmake

`install` with `SCRIPT` supports using relative paths to the current directory,
so there is no need to use the less obvious `${TOP_SOURCE_DIR}`.

CMake: stop configuring CMAKE_INCLUDE_PATH

This variable affects `find_file` and `find_path`.¹ `find_file` is not used in
Graphviz and `find_path` already defaults to the correct values for the current
platform.² This change avoids, among other things, hard-coding of the target
architecture as x86-64.

Related to #1973.

¹ see https://cmake.org/cmake/help/latest/variable/CMAKE_INCLUDE_PATH.html#variable:CMAKE_INCLUDE_PATH
² https://cmake.org/cmake/help/latest/command/find_path.html#command:find_path

gvconfig_plugin_install_builtins: remove an unnecessary cast

gvconfig.c: fix comment typo

remove EXPERIMENTAL_MYFGETS-guarded code

Nothing in the build systems defines this.

remove getenv detection

The code base extensively uses `getenv` without checking the `HAVE_GETENV` macro
this detection supplies. This is not a problem because `getenv` is provided as
far back as C89, and can be guaranteed to exist on all platforms for which
Graphviz currently compiles, as we require C99.

Merge branch 'smattr/162479A8-250E-4F66-91BC-61BECABD6150' into 'main'

fix claimed minimum supported CMake version

See merge request graphviz/graphviz!2302

CMake: [nfc] standardize 'if' style

The if-style used in CMake files had no consistency. Meanwhile, the expression
in a CMake `endif` is ignored. We learned in the process of constructing the
prior commit that this is supported right back to CMake 3.1. So this commit
standardizes on no-space, no-expression if-style.

fix claimed minimum supported CMake version

Commit 354ed2c644452f300b316d166386bd40d8fec76f, among others, introduced the
use of `$<TARGET_OBJECTS>` in `target_link_libraries` calls. The CMake docs¹
claim this is only officially supported from 3.21 onwards, despite working in
some prior versions. CMake provides binaries going back many versions² so it is
possible to manually bisect exactly what this wording in the CMake docs means.
Running this process yields the answer that CMake 3.9 is the first version that
does not error out when encountering `$<TARGET_OBJECTS>` in a
`target_link_libraries` expression.

Note that the Graphviz CMake files also use some features like `endif` without
an expression, which claims to only be supported from CMake 3.14 onwards.³
Despite this, versions as far back as CMake 3.1 support this syntax just fine.

¹ https://cmake.org/cmake/help/latest/command/target_link_libraries.html#linking-object-libraries-via-target-objects
² https://cmake.org/files/
³ https://cmake.org/cmake/help/v3.13/command/endif.html vs
  https://cmake.org/cmake/help/v3.14/command/endif.html

Merge branch 'smattr/FC04297A-31DD-49E0-A0C0-42C6CB63E00E' into 'main'

ci/mingw-build.sh: install all packages with one pacman command

See merge request graphviz/graphviz!2313

ci/mingw-build.sh: install all packages with one pacman command

This is potentially faster.

Merge branch 'smattr/1FA2B68F-FA19-4465-9BFD-8F82D6D786BD' into 'main'

don't override library names in .def files

See merge request graphviz/graphviz!2314

edgepaint: remove 'LIBRARY' declaration from the MSVC defs file

Similar to the prior commit and 2b92772d480f321bda6615140efa546abc5c66e9, this
is a compatibility change that allows MinGW to also understand this file
correctly.

Suggested-by: Keno Fischer <keno@juliacomputing.com>

pathplan: don't override library name in .def file

Similarly as reported in
https://gitlab.com/graphviz/graphviz/-/issues/2156, the export name
override breaks the MinGW and Cygwin autotools builds. By not
specifying the export name, each toolchain should use its appropriate
default, which should at least work, even if it's not consistent
across toolchains.

Thanks @Keno.

Merge branch 'smattr/E89137C0-15A3-4ACD-8BE0-1211A728BB7E' into 'main'

some bool modernization

See merge request graphviz/graphviz!2300

neato_set_aspect: use a C99 bool return type instead of boolean

extractMax_f: use a C99 bool return type instead of boolean

lib/neatogen/dijkstra.c: remove a commented out #include

extractMax: use a C99 bool return type instead of boolean

neatogen bfs: use C99 bool return types instead of boolean

idealsize: use a C99 bool return type instead of boolean

pathscross: use a C99 bool return type instead of boolean

conc.c: use C99 bools instead of booleans

noClip; use a C99 bool return type instead of boolean

htmlenv_t.objid_set: use a C99 bool instead of boolean

Merge branch 'smattr/b7d78258-59ca-4b36-b34f-3ba688801700' into 'main'

teach the Autotools build system that GTK headers are not part of Graphviz

See merge request graphviz/graphviz!2296

teach the Autotools build system that GTK headers are not part of Graphviz

This is the equivalent of fb5e2bf574739cc8225683aacfc6833d586a9e77, but for
Smyrna’s use of GTK. It squashes 27 warnings.

Merge branch 'smattr/70447D6E-8DE8-42FE-938C-6623A2094C4D' into 'main'

tclpkg: remove C++ guards from simple.h

See merge request graphviz/graphviz!2297

tclpkg: remove C++ guards from simple.h

This header is not shipped and not included by anything except C sources.
Moreover these guards gave the incorrect impression this header was usable from
C++. One of its structs contains a field, `final`, that is a C++ keyword, so it
is not possible to use this code from C++.

Merge branch 'smattr/A2038E24-6EA2-49E9-9C64-01EE8748A977' into 'main'

smyrna: more dead code removal

See merge request graphviz/graphviz!2298

smyrna renderNodeLabels: remove unused 'nodeSize'

smyrna edge_xdot: squash -Wunused-parameter warnings

smyrna renderSmGraph: remove unused 'g' parameter

smyrna set_boundaries: remove unused 't' parameter

smyrna changetopfishfocus: remove unused 'z' parameter

smyrna get_active_frame: remove unused 't' parameter

smyrna prepare_topological_fisheye: remove unused 'gg'

smyrna pick_object: replace use of large constant with 'FLT_MAX'

The compiler warns:

  selectionfuncs.c: In function 'pick_object':
  selectionfuncs.c:114:16: warning: conversion from 'int' to 'float' changes
    value from '999999999' to '1.0e+9f' [-Wfloat-conversion]
    114 |     float dist=999999999;
        |                ^~~~~~~~~

The usage of `dist` in this function is really that its initial value needs to
be greater than any other value encountered, so `FLT_MAX` serves this purpose
better.

smyrna combineCallback: squash -Wunused-parameter warnings

smyrna frmobjectui.c: squash -Wunused-parameter warnings

smyrna: remove unused 'on_attrRB0_clicked'

smyrna: remove an unused strcasecmp.h include

The only use of `strcasecmp` in this file was removed in
3fff5a1665b4be68f5dc69c5315b64dd556ce17a.

Merge branch 'smattr/2BC8555C-71B7-4C94-A426-C0222D7299D4' into 'main'

deploy.py: include Windows artifacts in generated JSON

See merge request graphviz/graphviz!2301

deploy.py: simplify 'checksum' function

Commit 0b31ec29078ecee5b868b42453478f1dd8239d54 modified `checksum` so it only
produced a single checksum, SHA256. Therefore we can simplify the function to
just return that single checksum file instead of defining it as a generator.

deploy.py: include Windows artifacts in generated JSON

This avoids the error prone manual process we have been stepping through for the
last few releases.

deploy.py: outline functionality for deriving the 'format' field of JSON output

This will be used in a future commit to add Windows entries to the JSON data.

Merge branch 'smattr/054CE618-2658-48EF-AEDD-5402FB3AEB53' into 'main'

gen_version.py: add a '--output' flag, avoiding the need for encoding hacks

See merge request graphviz/graphviz!2293

use gen_version.py’s '--output' option to avoid redirection, encoding hacks

gen_version.py: add a '--output' flag

This allows the script itself to control text encoding used in the output. A
future commit will use this to avoid awkward juggling of locales and encoding
externally.

Merge branch 'smattr/788C09F1-D6B8-4193-B283-C68FE424A520' into 'main'

DEVELOPERS.md: document how to set dynamic loader path on macOS

See merge request graphviz/graphviz!2304

DEVELOPERS.md: document how to set dynamic loader path on macOS

Merge branch 'cross' into 'main'

cmd/dot: don't run dot -c when cross-compiling (CMake edition)

See merge request graphviz/graphviz!2299

cmd/dot: don't run dot -c when cross-compiling (CMake edition)

We probably can't execute the binary we've cross compiled.

Merge branch 'smattr/5D7A95BC-E75A-4C0C-A377-4C66AD66D428' into 'main'

enable -Werror for MinGW CMake jobs

See merge request graphviz/graphviz!2294

.gitlab-ci.yml: remove inaccurate comment

It looks like commit 406169f0382204d4bad8da9942bed9d865dfb022 accidentally
duplicated this line from earlier in the same list of commands.

fail MinGW CMake CI jobs on any compiler warning

As of the prior commit, these builds are warning free.

gvplugin_gdiplus_LTX_library: squash a -Wwrite-strings warning

In future, maybe `gvplugin_library_t` should be modified to make its string
member const, given it is always expected to be a string literal.

work around MinGW’s lack of support for '%zu'

MinGW warns:

  plugin/core/gvrender_core_dot.c: In function 'xdot_str_xbuf':
  plugin/core/gvrender_core_dot.c:88:24: error: unknown conversion type
    character 'z' in format [-Werror=format=]
     88 |     agxbprint (xb, "%s%zu -%s ", pfx, strlen(s), s);
        |                        ^
  plugin/core/gvrender_core_dot.c:88:29: error: format '%s' expects argument of
    type 'char *', but argument 4 has type 'size_t' {aka 'unsigned int'}
    [-Werror=format=]
     88 |     agxbprint (xb, "%s%zu -%s ", pfx, strlen(s), s);
        |                            ~^         ~~~~~~~~~
        |                             |         |
        |                             char *    size_t {aka unsigned int}
        |                            %d
  plugin/core/gvrender_core_dot.c:88:20: error: too many arguments for format
    [-Werror=format-extra-args]
     88 |     agxbprint (xb, "%s%zu -%s ", pfx, strlen(s), s);
        |                    ^~~~~~~~~~~~

gdiplusgen_textspan: squash -Wmisleading-indentation warning

gdiplus plugin: make 'FileStream' destructor virtual

`FileStream::Release` optionally deletes `this`. This is OK because nothing
inherits from `FileStream`, but the compiler believes this is dangerous because
this class participates in an inheritance hierarchy and does not have a virtual
destructor. By giving it one, we silence the -Wdelete-non-virtual-dtor warning,
make it safer to inherit from this class in future, and make it safe to delete
a `FileStream` through an `IStream` pointer.