fix: remove unused format arguments in rtest

This was detected by `python3 -m pylint --errors-only rtest/rtest.py`.

Merge branch 'smattr/81334456-1889-4c1c-b6ee-3e83aa779c2f' into 'master'

fix macOS CMake build warnings

See merge request graphviz/graphviz!1644

fix printf format code in agnameof()

This fixes a build warning on macOS when using CMake. Related to !1620.

use safer snprintf in agnameof()

reduce the scope of a buffer in agnameof()

fix printf format code in write_nodename()

This fixes a build warning on macOS when using CMake. Related to !1620.

use safer snprintf in write_nodename()

reduce the scope of buffer in write_nodename()

fix buffer overflow in write_nodename()

This buffer was not large enough to print the string and uint64_t that gets
written into it. Related to !1620.

fix missing bracketing in a macro expansion

None of the calls to this macro were passing in multiple tokens, so this issue
was latent. This refactor is for the safety of any future introduced calls to
EMPTY().

fix handling of varargs in gvprintf

It is not safe to re-va_start a va_list the way this code was previously
operating. This probably worked by coincidence on platforms like x86-64.

switch to safer snprintf in portName()

fix print of an AGSEQ

This squashes a build warning on macOS when using CMake. Related to !1620.

fix a buffer overflow in string construction

The conditional in this code was the wrong way around. So the %s branch could be
taken only when the resulting sprintf would *definitely* overflow the target
buffer. From this we can conclude this branch was never safe and never used
correctly. For simplicity, we remove it, leaving only the %d option. Related to
!1620.

Merge branch 'smattr/029D9712-951E-4082-B254-9B0BB83BC384' into 'master'

remove a use of setjmp/longjmp

See merge request graphviz/graphviz!1645

remove a now unused jmp_buf

Related to #1801.

indicate failure in growops() by return value instead of longjmp()

Related to #1801.

indicate failure from growtris() by return value instead of longjmp()

Related to #1801.

add a return value to loadtriangle()

Related to #1801.

add a return value to triangulate()

Related to #1801.

remove unnecessary use of longjmp() in growdq()

Related to #1801.

remove an unnecessary use of longjmp() in growpnls()

Related to #1801.

Merge branch 'smattr/4b7c9c53-bdf9-48ef-8fa6-7c06fc5720b4' into 'master'

stop shipping strcasecmp.h

See merge request graphviz/graphviz!1642

stop shipping strcasecmp.h

The header strcasecmp.h is a portability shim that was introduced in
cadb72d6157603338a990be4e27a599c7b96b214. However, no installed Graphviz headers
#include this header, so we do not need to install it. It is cleaner not to
clutter the shipped Graphviz include directory with an internal shim like this.

Note that there is no changelog entry for this change and this is not an API
break because there has not yet been a release since this header was added.

Merge branch 'orientation' into 'master'

Delete attrs.html from main graphviz repo

Closes graphviz.gitlab.io#65

See merge request graphviz/graphviz!1641

Delete attrs.html from main graphviz repo

This now canonically lives in the graphviz.gitlab.io repo (the docs repo):

https://gitlab.com/graphviz/graphviz.gitlab.io/-/blob/master/doc/info/attrs.html

This gives one central source of truth for the docs, so that people
updating the docs don't have to update 3 locations (doc/info,
doc/infosrc, and the docs repo).

It's perhaps a little controversial to redirect the old docs. The old
docs are still built into some OS's HTML docs packages, but I imagine
the vast majority of users access docs through the graphviz website
after using a search engine to find them.

Fixes https://gitlab.com/graphviz/graphviz.gitlab.io/-/issues/65

Merge branch 'smattr/f037c2f4-a5af-4f90-a41d-412fb74c4296' into 'master'

add CI testing of HTML files in the repository

See merge request graphviz/graphviz!1637

run pytest with --verbose in CI

This makes it a little easier to see from the logs which tests were skipped and
to validate that the tests you think are being found are actually running.

add test cases for validating in-repo HTML files

Related to #1861.

Merge branch 'smattr/38BA03F9-7DC4-4E4F-8B4C-51C3F360855D' into 'master'

remove unnecessary casts and fix a memory leak

See merge request graphviz/graphviz!1630

remove unnecessary casts from parameters to memcpy

These implicitly coerce to void* so a cast is not required.

rephrase some swapping code

C allows you to assign structs in this way, so there's no need to use a more
confusing memcpy sequence.

remove unnecessary casts on arguments to free()

These implicitly coerce to void* so no cast is needed.

remove unnecessary guards on calls to free()

Freeing NULL is a no-op, so there is no need for these extra conditionals.

fix memory leak in gvplugin_list

It seems clear these conditionals were reversed and were meant to read
`if (typestr_list)`. Rather than flip their polarity, we just remove them as it
is safe to free NULL.

remove some unnecessary casts of arguments to realloc()

These parameters implicitly coerce to void* with no casting required.

remove unnecessary casts of realloc() return value

realloc returns a void* which, in C, implicitly coerces to all other pointer
types

remove unnecessary casts of calloc() return value

calloc returns a void* which, in C, implicitly coerces to every other pointer
type.

remove unnecessary casts of malloc() return value

malloc returns a void* which, in C, implicitly coerces to every other pointer
type.

Merge branch 'add-some-missing-tools-to-macos-autotools-build' into 'master'

Add some missing tools to macos autotools build

See merge request graphviz/graphviz!1650

Enable dotty, lefty, lneato and vimdot to be built on macOS.

Enabled by adding brew install libxaw to macOS builds

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1858

Towards https://gitlab.com/graphviz/graphviz/-/issues/1854

Move common commands from macOS jobs to macos_build_definition

Merge branch 'smattr/6f54cfb5-5fb4-42a0-aba0-41e155372757' into 'master'

refactor tests cases into CI-only and others relevant to developers and CI

Closes #1851

See merge request graphviz/graphviz!1647

Merge branch 'smattr/BDA848A6-E53C-4722-BDF7-B82677681621' into 'master'

clean up and a UB fix

See merge request graphviz/graphviz!1621

remove fine grained test skip logic and now depend on tool existence

The test suite should now pass on developers' machines, regardless of which
build configuration or subset of tools they use. Closes #1851.

add others to the CI tool check that do not need fine grained exceptions

Related to #1851.

add exemption list for Autotools macOS build in CI tool check

Related to #1851.

add MSBuild exemption list to CI tool check

Related to #1851.

add CMake exemptions to CI tool check

Related to #1851.

extend CI tool check to dot_builtins

Related to #1851.

extend CI tool check to Mingle

Related to #1851.

check that Smyrna exists in CI

Related to #1851.

move some pytest skip logic into a decorator

For more consistency with other tests.

adjust installation test to work even if GV_VERSION is unset

This test should now run correctly in users' development environments as well as
in CI. Related to #1851.

make gen_version run correctly regardless of CWD

add a home for test cases that are only relevant to run in CI

Related to #1851.

Merge branch 'ensure-all-existing-tools-are-tested' into 'master'

Ensure all existing tools are tested

See merge request graphviz/graphviz!1643

Add check that expected missing tool doesn't exist to tools test

Add removal of old graphviz version before install on RPM systems

Merge branch '4AF62F39-396B-483C-A035-F1C7A0E5D039' into 'master'

remove some unused lib/cgraph functions

See merge request graphviz/graphviz!1445

Add removal of old graphviz version before install on macOS

Remove gc from tools_not_built_with_cmake list in tools test

It was a mistake to put it there in the first place.

Ensure that Graphviz tools are found first in Windows CMake jobs

Prepend Graphviz bin directory to path instead of appending it.

The tools test found
C:\Program Files (x86)\Windows Kits\10\bin\10.0.18362.0\x86\gc.exe
before Graphviz gc.

Merge branch 'add-macos-test-jobs-to-gitlab-ci' into 'master'

Fix tool usage on macOS and add macOS test jobs to gitlab CI

Closes #1852, #1860, and #1727

See merge request graphviz/graphviz!1629

Add macos-cmake-test job to CI

Resolves https://gitlab.com/graphviz/graphviz/-/issues/1727.

Add macos-autotools-test job to CI

Towards https://gitlab.com/graphviz/graphviz/-/issues/1727.

Add skip of test_1818 also for macOS

Fails because of https://gitlab.com/graphviz/graphviz/-/issues/1858.

Add skip of test_1813 also for macOS

Fails because of https://gitlab.com/graphviz/graphviz/-/issues/1857.

Add skip of all tools not built for macOS in CI using autotools in tools test

Fails because not all tools are built for macOS in CI using
autotools. See https://gitlab.com/graphviz/graphviz/-/issues/1854

Add subset of macOS rtest reference files

Generated with ./rtest.py -g. The .gv files are identical to those in
the linux.x86 subdirectory. The .ps file has slight differences, but
no visible differences.

Add brew install pango to macos build jobs

Needed for png format and cairo renderer.

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1860.

Add support for macOS to ci/install.sh

Refactor ci/install.sh to check for build system before OS

Fix tred -? to actually print usage instead of error on macOS

Final commit that fixes https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix sccmap -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix nop -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix mm2gv -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix gvpack -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix gvcolor -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix gc -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix dijkstra -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix gvmap -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix cluster -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix edgepaint -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Fix ccomps -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852

Fix bcomps -? to actually print usage instead of error on macOS

Towards https://gitlab.com/graphviz/graphviz/-/issues/1852.

Restore whitespace and comment after earlier bad merge conflict resolution

Merge branch 'smattr/629f9eac-8e8a-486d-9695-cc857dce075c' into 'master'

some clean up

See merge request graphviz/graphviz!1620

remove rtest.sh redirect

Nothing in the repository uses this any more.

remove unnecessary 'global' statements in rtest

These variables were either unused in the contained function or only read from.

fix some comparisons between booleans and ints in rtest

This all worked fine, but for consistency we should be treating these variables
as booleans everywhere.

remove legacy comment

The variable this is referring to, LINECNT, was removed when this script was
ported to Python in a1b9ffc0c37a435607ac8b1c123c8c542ef1cec9.

remove some unused variables in rtest

squash a -Wabsolute-value warning

In 8414e62738bf7f52596eb83049ecef758582c760 I misread the surrounding code and
thought this was a float being operated on here.

squash -Wcomment warning

remove unused onexit() prototype

remove unused _exit() prototype

remove unused prototypes for frexp() and ldexp()

remove unneeded X_OK shim

Nothing in Graphviz uses X_OK.

remove POOLMTXRETURN()