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.

cgraph: go through 'uintptr_t' when casting between pointers as IDs

The type `IDTYPE` is `uint64_t` which is wide enough to contain a pointer on all
supported platforms. However, on 32-bit platforms the compiler throws
-Wpointer-to-int-cast about this kind of back-and-forth shuffling. Going through
`uintptr_t` (or `intptr_t`) silences these.

sfoutput: squash a -Wpointer-to-int-cast warning

_sfpopen: fix uninitialized variable read on Windows

sfio: squash -Wunused-variable on platforms without SIGPIPE

This warning, for example, is triggered in Mingw builds.

plugin/gdiplus/gvloadimage_gdiplus.cpp: squash -Wunused-parameter warning

plugin/gdiplus/gvtextlayout_gdiplus.cpp: squash -Wunused-parameter warnings

plugin/gdiplus/gvrender_gdiplus.cpp: squash -Wunused-parameter warnings

plugin/gdiplus/FileStream.cpp: squash -Wunused-parameter warnings

Merge branch 'smattr/f7369310-375d-44c7-91cc-22e88c84d202' into 'main'

tclpkg/tclpathplan warning squashing

See merge request graphviz/graphviz!2286

vgpane: squash -Wunused-parameter warnings

vgpanecmd: squash -Wunused-parameter warning

makecw: squash a -Wmissing-prototypes warning

intpoint: squash a -Wswitch-default warning

This switch already covered all reachable cases.

tclpkg/tclpathplan/intersect.c: remove unnecessary parens

find_ints: remove unused 'polygon_list' parameter

find_ints: squash a -Wswitch-default warning

It seems clear from surrounding context that this switch is intentionally a
no-op for other values.

tclpkg/tclpathplan: use a size_t for 'nvertices'

Squashes three -Wsign-conversion warnings at the expense of introducing one.
Hopefully the introduced one can be squashed in future.

tclpkg/tclpathplan: squash 'find_ints' -Wmissing-prototypes warning

Merge branch 'smattr/70deb7de-4445-488a-be9f-b5f73e1f8d77' into 'main'

fix xdot JSON output to be valid JSON

Closes #1958

See merge request graphviz/graphviz!2295

fix xdot JSON output to be valid JSON

The output of libxdot’s JSON functions included identifiers as non-strings,
something that is not valid JSON. This change results in identifiers like `e`
now being emitted as a string `"e"`.

Fixes #1958.

Merge branch 'kf/gvcdef' into 'main'

gvc.def: Don't override export name

See merge request graphviz/graphviz!2290

Merge branch 'smattr/03501BAF-AD18-474D-B48E-D7A8F81262F3' into 'main'

fail deployment if a release exceeds 50 artifacts

See merge request graphviz/graphviz!2284

fail deployment if a release exceeds 50 artifacts

Following commit 0155cabbb7b22494b3d5da1425059387d8236cf2, the number of
artifacts produced during packaging is down to 40. We expect this count to rise
as the CMake build system continues to be worked on, operating systems move in
and out of support cycles, and the Cygwin/Msys builds are tweaked. This change
introduced a safe guard to fail deployment if the number of artifacts grows to
exceed 50 again.

Related to #2147.

Merge branch 'smattr/e936b86f-3ac0-4234-98eb-79f6d6e43a16' into 'main'

further gvusershape.c clean up

See merge request graphviz/graphviz!2287

jpeg_size: use C99 'true' in loop condition

jpeg_size: use a C99 bool for 'saw_bb' local

svg_size: use C99 bools for local flag variables

gvusershape.c: squash -Wmissing-prototypes warnings

gvusershape_find: fix mismatch between prototype and implementation

usershape_close: squash -Wunused-parameter warnings

usershape_close: remove unnecessary cast

get_int_msb_first: squash a -Wsign-conversion warning

jpeg_size: mark 'standalone_markers' const

jpeg_size: remove abuse of 'strchr' to search a byte array

Using `memchr` instead avoids the need to cast the array, shrinks the array
itself by avoiding the need for a NUL terminator, and makes it more obvious to
the compiler it can inline and unroll the whole thing.

jpeg_size: remove use of 'junk' variable

This code was reading big endian integers into this variable that was then
ignored as a way of skipping fields. It is simpler and more efficient to just
fast forward the file handle over those bytes. This assumes the file handle is
to something seekable (e.g. not a FIFO), but the handle was already seeked
elsewhere in this function, so this assumption already existed.

mingle usage: remove always-true if branch

This function is only ever called with `eval` non-negative.

Merge branch 'smattr/209CD786-B836-43BD-A488-535300F228D0' into 'main'

cmd/smyrna: more clean up

See merge request graphviz/graphviz!2288

smyrna menucallbacks: squash -Wunused-parameter warnings

smyrna: remove unused 'mHelp'

smyrna: remove unused engine slot functions

smyrna: remove unused 'show_close_nosavedlg'

smyrna: remove unused 'get_gtktextview_text'

smyrna: remove unused 'update_graph_properties'

smyrna: remove unused 'on_attrSearchBtn_clicked'

smyrna: remove unused 'attr_label_motion'

smyrna: make 'attr_list_new' static

This function is not used outside of its containing file.

Merge branch 'smattr/7e500215-5167-44d7-90c2-9e79ec4686dd' into 'main'

cmd/tools warning squashing and clean up

See merge request graphviz/graphviz!2283

gvcolor: remove unnecessary parens

sccmap: remove unnecessary parens

bcomps: remove unnecessary parens

bcomps getName: use size_t for buffer lengths

Squashes some -Wconversion warnings.

bcomps blockName: use size_t for buffer lengths

Squashes some -Wconversion warnings.

gvpack redoBBk: use size_t for buffer lengths

Squashes some -Wconverion warnings.

gvpack freef: squash -Wunused-parameter warnings

gvpack init: use a size_t to squash some -Wconversion warnings

sccmap.c: use libc support to define 'INF'

Squashes a -Wconversion warning.

sccmap.c: return an unsigned type from 'visit'

Squashes a -Wsign-conversion warning.

The `val` member of `Agnodeinfo_t` is unsigned and all uses of this function
store the return value in an unsigned. So it seems this function always should
have been declared as returning an unsigned.

sccmap.c: use unsigned types consistently in sccstate

Squashes a -Wconversion warning.