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

2184.dot: remove eol spaces

Merge branch 'smattr/050F6C7D-7BC7-444D-9D6C-99C0762069CA' into 'main'

cgraph: implement memory allocation wrappers

See merge request graphviz/graphviz!2510

ccomps: replace local allocation wrappers with cgraph ones

A first example use of the functions from the prior commit.

cgraph: implement memory allocation wrappers

This change replicates and extends the exit-on-failure allocation wrappers from
lib/common/memory.c into a header-only implementation in cgraph. The existing
common wrappers worked but there were two limitations preventing their wider
use:

  1. lib/common is not the base of the Graphviz dependency graph. E.g.
     lib/cgraph does not depend on lib/common, preventing calling
     lib/common/memory.h functions from cgraph code. lib/cgraph is not the base
     of the dependency graph either (it depends on lib/cdt), but it has become a
     de facto home for header-only implementations of general utility functions
     called throughout the rest of the Graphviz code base.

  2. lib/common is not linked against by every Graphviz component. This meant
     other code could attempt to call functions in lib/common/memory.h but would
     fail to link because it was not pulling in lib/common/memory.c.

Relocating these functions neatly solves both the above two. It also has the
side effect of allowing compilers to inline these short wrappers. This could
(and would) previously be achieved with Link-Time Optimization, but now a
regular optimizing compilation can also achieve this.

Future changes will remove the lib/common/memory.c wrappers in favor of these.

Merge branch 'truenorth-issue-2184' into 'main'

Attempt to fix initialization of subgraph attributes on existing subgraphs...

See merge request graphviz/graphviz!2504

parenthesize conditional blocks and clang-format prior code changes

Attempt to fix initialization of subgraph attributes on existing subgraphs…

Attempt to fix initialization of subgraph attributes on existing subgraphs when
a new root graph attribute is created.

Fix issue 2184, initialization of existing subgraph attributes when a new top
level graph attribute is created. Looks ok, but new code unviewsubgraphsattr is
executed on many of the test graphs, so this should be tested carefully.

add #2184 test cases

Merge branch 'smattr/BD8C9D81-BF8B-4ED0-8A5F-3611689F010D' into 'main'

ccomps: some movement towards thread safety and some clean up

See merge request graphviz/graphviz!2508

ccomps: remove unnecessary parens

ccomps: remove redundant 'fflush'

Closing a file already flushes all written data to disk.

ccomps process: more tightly scope temporary buffer usage

This makes it harder to accidentally introduce a use-after-free in this code.

ccomps processClusters: more tightly scope temporary buffer usage

This makes it harder to accidentally introduce a use-after-free in this code.

ccomps: remove use of long lived 'getBuf' allocation

Allows more localized reasoning. This also makes progress towards thread safety.

ccomps: remove parameter to 'split'

This function was retaining a pointer to memory pointed to by its parameter,
`name`, in the global `suffix`. So it is unsafe to call this with anything
except a long lived string. We can make this more obvious/explicit by inlining
the use of `outfile`, the only parameter it is ever called with.

ccomps: make all internal variables static

Merge branch 'smattr/ec571e5b-baf1-40a7-add9-416a446bd4b2' into 'main'

some misc clean up

See merge request graphviz/graphviz!2507

finishEdge: remove now-unused 'p' and 'q' parameters

finishEdge: remove now-unused 'p1' and 'q1' locals