Merge branch 'smattr/0E5378C5-27A2-4E7A-A4A8-8BC527899AF6' into 'main'

smyrna: clean up and warning squashing

See merge request graphviz/graphviz!2533

cmd/smyrna: remove unused return value from 'close_graph'

cmd/smyrna/viewport.c: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

cmd/smyrna: fix a comment typo

cmd/smyrna: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

cmd/smyrna: remove no-op if statement

When enabling Smyrna in the CMake build system, compilation fails with a
-Wempty-body warning about this line. I am guessing that it was intended to not
be a conditional here.

Gitlab: #1836, #1854

fix: teach Smyrna '-?' on macOS and Windows

This is the equivalent of cd80b6262c7e1b95307756f13c89a9f6b2900520 and friends
but for Smyrna. It was not previously detected that this problem also affects
Smyrna because it is not currently built on macOS or Windows in CI. This will be
improved in future.

Gitlab: #1852

lib/glcomp: fix -Wnon-literal-null-conversion with Clang

This was causing the CMake build (to be enabled in a future commit) on macOS to
fail.

Gitlab: #1836, #1854

lib/glcomp: remove unused 'glCompButton.customptr'

With Clang on macOS, this causes compiler warnings due to initializing a pointer
with `'\0'`. This fails the CMake build which will be enabled in a future
commit. We could rephrase this, but it is simpler to just remove this unused
member.

Gitlab: #1836, #1854

smyrna: remove unnecessary 'gpointer' cast

`gpointer` is just a typedef for `void*` that implicitly coerces to all other
pointer types.

smyrma: remove unnecessary 'gint' cast

`G_PRIORITY_DEFAULT` is already a `gint` (typedef for `int`).

smyrna: remove use of 'guint' cast

`guint` is just a Glib typedef for `unsigned int`. So we can write this simpler.

Merge branch 'smattr/1D9B0E32-37CA-4547-A966-AE877344C814' into 'main'

tests: remove unused cdiff.c

See merge request graphviz/graphviz!2532

tests: remove unused cdiff.c

Merge branch 'smattr/63D62C3D-D094-4ED8-A675-538D7283B9E6' into 'main'

use suffixed literals instead of casts to construct 'GLfloat' values

See merge request graphviz/graphviz!2530

use suffixed literals instead of casts to construct 'GLfloat' values

The `GLfloat` type is 32-bit IEEE-754 floating-point.¹ As of C99, the `float`
type is also 32-bit IEEE-754 floating-point on all platforms with support.² In
contemporary usage, this is effectively all C99 platforms that support
floating-point arithmetic. Graphviz has been compiled in C99 mode since commit
fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc. Putting those things together, we can
abbreviate the way we write `GLfloat` literals, leading to more readable code.

¹ https://www.khronos.org/opengl/wiki/OpenGL_Type
² https://en.wikipedia.org/wiki/C99#IEEE_754_floating-point_support

Merge branch 'smattr/26236D4A-346B-44B1-ACAB-ED8D7E59E09E' into 'main'

gvc: remove unused unistd.h #include

See merge request graphviz/graphviz!2531

gvc: remove unused unistd.h #include

Gitlab: #2204

Merge branch 'smattr/9691B69C-F05E-4819-B327-8258BB2B3D23' into 'main'

move XML validation tests into a separate CI job

Closes #2216

See merge request graphviz/graphviz!2527

move XML validation tests into a separate CI job

This parametrized test was checking the validity of checked in Graphviz HTML
sources. There is no need to run this identical check in every single CI test
job nor on end users machines where they may have differing versions of
`xmllint` installed.

Gitlab: closes #2216

Merge branch 'smattr/A0DB9535-2265-4E6C-9389-16058C1BB35B' into 'main'

towards enabling Smyrna in CMake

See merge request graphviz/graphviz!2526

smyrna Init: squash -Wincompatible-pointer-types

OpenGL has an unorthodox API wherein the `gluTessCallback` function’s prototype
indicates it takes a `void(*)(void)`, but its docs¹ explain that the type
actually varies depending on the second argument. As a result, the compiler
(correctly) warns that some of these `gluTessCallback` calls are passing
function pointers that do not have the same ABI. Presumably this works out
because at the end of the day a function pointer is just some bits in C and the
OpenGL implementation branches on the `which` argument and invokes the pointer
correctly. But if OpenGL really wanted to discard type safety this way, it is
not clear to me why they did not make the function pointer argument a `void*`.
Anyway, this commit squashes the compiler warnings which emerge when enabling
this in the CMake build system, failing the build.

Gitlab: related to #1836

¹ This is not the authoritative source, but Microsoft’s docs for their
  implementation provide a good explanation.
  https://docs.microsoft.com/en-us/windows/win32/opengl/glutess

smyrna cb_mouseclick: remove unused 'g'

This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

smyrna on_attrAddBtn_clicked: remove unused 'value'

This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

smyrna: remove unused 'gvpr_select'

The last use of this was removed in 7e1f01ad52915498a252dd213f85daf5a058aa5d.
This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

smyrna on_attrAddBtn_clicked: remove unused 'prog'

This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

smyrna doApply: remove unused 't'

This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

cmd/smyrna: remove some unnecessary intermediate buffers

cmd/smyrna: remove some excess white space

cmd/smyrna: remove some open coded 'NULL'

CI: install libglade on CentOS 7, enabling lib/glcomp

A step on the way towards enabling Smyrna in the CMake build.

Gitlab: #1836

CMake: enable lib/glcomp

Interestingly enabling this revealed that lib/glcomp also depends on Pango,
something unexpressed in either the Autotools or MS Build build systems.

Gitlab: #1836

lib/glcomp: remove gtkglext CFLAGS tweak

As far as I can tell, glcomp does not use gtkglext. Smyrna, which depends on
glcomp, does. But Smyrna specifies its own linking against gtkglext.

lib/glcomp: remove unnecessary 'static' from some variables

These are unconditionally overwritten during their respective functions before
they are ever read.

lib/glcomp: remove unnecessary parens

lib/glcomp: remove some unnecessary casts

lib/glcomp: remove some open coded 'NULL'

lib/glcomp: remove some duplicate #includes

lib/glcomp: remove some trailing white space

lib/topfish: missing CMake dependencies

This seems to have been broken since CMake support for lib/topfish was
introduced in 45893ca105777e1c5b97788cd169e400250ffa1f. But it was never noticed
because there is no executable in the CMake build system that links against
topfish yet.

Gitlab: #1836

lib/topfish: fix CMakeLists.txt indentation

Unclear why `cmakelang.lint` does not warn about this.

lib/topfish maxmatch: remove unused 'sum_weights'

This causes compiler warnings resulting in build failure when enabling this in
the CMake build system.

Gitlab: related to #1836

remove unused 'HAVE_GTKGLEXT'

Aside from being unused, it is unclear why the Windows config.h was defining
this given it does not define `HAVE_GTKGL`.

pathfind: remove an unnecessary cast

Merge branch 'smattr/E88122C7-BB84-4919-9B41-D44586D94783' into 'main'

CMake: enable edgepaint

See merge request graphviz/graphviz!2525

CMake: add cmd/edgepaint compilation

Gitlab: #1836

CMake: enable lib/edgepaint compilation

Note that the target here is named “edgepaintib” in anticipation of future
enabling of cmd/edgepaint claiming the name “edgepaint”. This also does not
define a separate liblab_gamut, as the Autotools build does. This separate
library seems only useful for cmd/edgepaint, so it does not seem worth splitting
it out as a separate library.

Gitlab: #1836

lib/edgepaint: remove linking against libANN

For the same motivation as the prior commit, this is unused.

lib/edgepaint: remove application of ANN’s CFLAGS

This seems to have been a mistake in aadee957012e676c9aaf36ec40cb2860c178e269.
lib/edgepaint does not depend on libANN, so it does not seem correct to apply
ANN’s CFLAGS when compiling it.

Merge branch 'smattr/71744FB7-FA95-46C5-949E-64F986D85872' into 'main'

test case for #2215

See merge request graphviz/graphviz!2521

gvParseArgs etc: use a clearer idiom for accessing characters of 'arg'

gvParseArgs etc: remove tests for 'NULL' members in 'argv'

The strings in `argv` within the range of `argc` are guaranteed non-null.

add a test case for #2215

Merge branch 'smattr/875065A8-89A1-453D-9A36-2C2A8E259B84' into 'main'

gvpr: remove some sfio, unistd.h dependencies

See merge request graphviz/graphviz!2523

gvpr: rewrite debug graph processing to use stdio instead of sfio

This cuts a dependency on sfio, as well as a dependency on non-portable unistd.h
functions.

Gitlab: #1998, #2204

gvpr: rewrite debug I/O output to use stdio instead of unistd

There is no need to use the lower level `write` API for this I/O. By using
`fprintf` instead, we avoid a dependency on non-portable unistd.h.

Gitlab: #2204

Merge branch 'smattr/B24E1807-D40B-483F-8529-60C8EB2E4707' into 'main'

further pathcanon clean up and remove strings.h detection

See merge request graphviz/graphviz!2522

remove no longer used strings.h detection

render.h: remove unused strings.h #include

pathfind: remove unused strings.h #include

pathcanon: remove unused 'loop' variable

pathcanon: remove unnecessary 'errno' save and restore

a3db9ad8f8eb79a931f0bc6d087ca3b52de3dd62 removed any logic in this function that
affected `errno`. So there is no need to save and restore it anymore.

Merge branch 'smattr/9E1814ED-8451-43CC-A7A1-AAF2BE8EDF86' into 'main'

roll out cgraph allocation wrappers to edgepaint

See merge request graphviz/graphviz!2517

ccomps: abbreviate an open coded 'gv_strndup'

cgraph: add a 'strndup' allocation wrapper

edgepaint: free 'lightness' prior to exit

Some small progress to becoming leak-free even on clean exit.

edgepaint: replace a 'strdup' with 'gv_strdup'

This allocation was not handling failure and the only reasonable response here
is to exit.

cgraph: add a 'strdup' wrapper to the allocation helpers

Merge branch 'smattr/2E1672CD-0A70-45B7-8770-A81BE7638FA4' into 'main'

document 'SIGUSR1' handling

See merge request graphviz/graphviz!2515

dot: make a man page note about 'SIGUSR1' effects

selfRightSpace: consistently use a double for calculating self width

This avoids going backwards and forwards between double and int, which was
causing some compiler warnings and unnecessary loss of precision.

Merge branch 'smattr/5d0e6877-4a8c-45a9-b1b5-f83e9328eeaf' into 'main'

add Mingle to the CMake build system

See merge request graphviz/graphviz!2383

add Mingle to the CMake build system

Gitlab: related to #1835, #1836, #1854

CI: install 'libANN' on macOS

This is a dependency of lib/mingle, and transitively cmd/mingle. Note that this
is not available through Homebrew so we need to use MacPorts.

CI: move macOS dependency installation out of line

Makes these a little easier to manage. Note that this also unifies the runtime
dependencies installed in Autotools and CMake jobs which simplifies ongoing
maintenance.

CI: rearrange macOS step related to Bison

It makes it more obvious why this step is being done when it is adjacent to the
Bison install itself.

CI: install 'libann-dev' on Ubuntu

This is a dependency of lib/mingle, and transitively cmd/mingle.

tests: move 'is_cmake' helper into gvtest library

tests: de-dupe some MinGW detection logic

tests: move 'is_mingw' helper into gvtest library

lib/mingle: remove commented out branch

This causes -Wconstant-logical-operand warnings, failing a CMake build on macOS.
It seems simplest to just remove this disabled code.

lib/mingle: squash -Wconstant-logical-operand warnings

When enabling this code on macOS, Clang complains about this.

fix: teach mingle '-?' on macOS and Windows

This is the equivalent of b4925c87f1e28a1bc66d111ef046e4e083db8c5b and friends,
but for mingle. It was not previously detected that this problem also affects
mingle because it is not currently built on macOS or Windows in CI. This will be
improved in future.

Gitlab: #1852

Merge branch 'smattr/5A4AA4D3-3163-49C0-9DF1-A0ADA71E3554' into 'main'

Revert "rewrite indexOf"

Closes #2211

See merge request graphviz/graphviz!2513

Revert "rewrite indexOf"

This reverts commit 84efe93d5c5a4da0e778dbf01ed8550d2d774fdf.

Gitlab: fixes #2211

add a test case for #2211

tests: abstract the common case for running 'gvpr'

Merge branch 'smattr/106DE02D-AD56-466A-96C1-9D2AC5315DD6' into 'main'

lib/common: some clean up and warning squashing

See merge request graphviz/graphviz!2511

processTbl: use unsigned shorts for 'r' and 'c' locals

Squashes two MSVC conversion warnings.

size_html_txt: use a C99 bool for 'simple'

Squashes an MSVC conversion warning.

initAnchor: squash C4244 MSVC warning for 'explicit_tooltip'

MSVC seems particularly pedantic about implicitly converting an unsigned 1-bit
bitfield to a bool:

  warning C4244: 'function': conversion from 'unsigned int' to 'bool'

These bitfields should really be signed (`int`) but it does not seem worth the
API break to do this.

htmlparse.y: remove some unnecessary casts

htmlparse.y: consistently treat 'kind' as a 'char'

The `kind` field in `htmllabel_t` is a `char`, so this code was causing
conversion compiler warnings when trying to assign an `int` to it. We can avoid
this ambiguity by consistently using the same type everywhere.

emit_end_edge: squash C4244 MSVC warnings for 'nodeIntersect' calls

MSVC seems particularly pedantic about implicitly converting an unsigned 1-bit
bitfield to a bool:

  warning C4244: 'function': conversion from 'unsigned int' to 'bool'

These bitfields should really be signed (`int`) but it does not seem worth the
API break to do this.

check_control_points: remove a use of int as boolean

Merge branch 'smattr/4C161FEF-32BC-4E3C-8E1C-1976BC9E6E9C' into 'main'

pathcanon: clean up dead code

See merge request graphviz/graphviz!2512

pathcanon: remove 'FS_3D'-guarded code

Nothing in the build systems defines this macro. It is also unclear how this
code could be used given it calls `pathnext`, a function that has never been
committted to the Graphviz repository.

remove now-unused 'pathgetlink'

pathcanon: remove 'flags' argument

This function was only ever called with `0` as `flags`, which disables a lot of
its logic. We can reduce the amount of ongoing code to maintain by paring this
back to just the core in use.

Merge branch 'remove-eol-spaces' into 'main'

2184.dot: remove eol spaces

See merge request graphviz/graphviz!2516