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()

remove POOLMTXSTART() no-op

remove POOLMTXUNLOCK() no-op

remove POOLMTXLOCK() no-op

remove SFMTXUNLOCK() no-op

remove no-op SFMTXLOCK()

remove useless extern declarations

These variables already have extern linkage.

remove now unused vthread.h

remove unused sf_mutex field from Sfextern_t

remove unused mtx parameter to SFCLEAR() macro

remove unused mutex field from SFIO structures

remove unused Sfonce functionality

remove SFONCE() no-op

remove vmtx mutex no-ops

remove unused SFIO vthread prototypes

remove unused sftell()

remove unused _sfstrtod()

Merge branch 'orientation' into 'master'

Combine `orientation` attr for graphs and nodes

See merge request graphviz/graphviz!1640

Combine `orientation` attr for graphs and nodes

This simplifies the generation quite a lot, gets rid of the duplicated
anchors for differently-named.

Removes a little confusion too, in my opinion.

It's not perfect: they have different datatypes.

Merge branch 'stray-a' into 'master'

attrs.html: Close TT tag

See merge request graphviz/graphviz!1639

attrs.html: Close TT tag

Merge branch 'stray-a' into 'master'

attrs.html: Remove stray anchor close tag

See merge request graphviz/graphviz!1638

attrs.html: Remove stray anchor close tag

Merge branch 'ps' into 'master'

attrs.html: Rename 'kind' to 'types'.

See merge request graphviz/graphviz!1636

attrs.html: Rename 'kind' to 'types'.

Matches the title of the table, and colloquial use.

Merge branch 'ps' into 'master'

attrs.html: Rename attr.uses to used_by

See merge request graphviz/graphviz!1635

attrs.html: Rename attr.uses to used_by

Matches table header.

Makes more sense when you're looking at an attribute definition

Merge branch 'ps' into 'master'

Add consistent indentation to attrs.html

See merge request graphviz/graphviz!1634

Add consistent indentation to attrs.html

No-op as far as appearance goes.

But makes looking at diffs much easier!

Merge branch 'ps' into 'master'

attrs.html: don't special-case double/int/string

See merge request graphviz/graphviz!1633

attrs.html: don't special-case double/int/string

Instead, add (very basic) type definitions for them.

This simplifies the template.

Merge branch 'ps' into 'master'

attrs.html: Stop special-casing bitmap output

See merge request graphviz/graphviz!1632