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.

sccmap.c: remove 'INLINE' alternatives

Modern compilers are easily capable of inlining these functions by themselves.

sccmap.c: use another unsigned type to squash -Wconversion warnings

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 an unsigned type to squash some -Wconversion warnings

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.

gvcolor.c: remove an unnecessary cast

gvcolor.c: remove duplicated #include

gvcolor.c: use a size_t to squash a -Wsign-conversion warning

Merge branch 'cross' into 'main'

cmd/dot: don't run dot -c when cross-compiling

See merge request graphviz/graphviz!2281

CHANGELOG.md entry for the prior commit

cmd/dot: don't run dot -c when cross-compiling

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

gvc.def: Don't override export name

As reported in https://gitlab.com/graphviz/graphviz/-/issues/2156,
the export name override breaks the win32/mingw32 autoconf build.
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.

Merge branch 'smattr/5E1B768C-337D-4ED1-9693-E6A6B1DFE934' into 'main'

cgraph man page formatting fix

See merge request graphviz/graphviz!2280

add a CHANGELOG entry for the prior commit

Correct boldness of agnxtsubg in cgraph.3

Merge branch 'smattr/6c18e731-115d-4167-92e7-bf96cbbe151e' into 'main'

gmlparse.y: fix all compiler warnings

See merge request graphviz/graphviz!2278

gmlparse.y: remove unnecessary parens

mkGraph: remove shadowing of 'G' parameter

addNodeLabelGraphics: remove unused 'xb' parameter

gmlparse.y: remove commented out code

free_attr: squash -Wunused-parameter warnings

gmlparse.y: consistently use unsigned short types for 'kind'

Squashes a -Wconversion warning.

gmlparse.y: consistently use unsigned short types for 'sort'

Squashes a -Wconversion warning.

gmlparse.y: remove unused 'ushort' typedef

gmlparse.y: use more appropriate stack entry count type

Squashes some -Wsign-conversion warnings.