move macOS CI jobs to Gitlab shared runner

The macOS CI closed beta program Graphviz has been participating in is winding
down. As of 2021-03-24, we have been requested to migrate to a shared macOS
runner as the program transitions into an open beta [0]. The cut over date for
this is scheduled for 2021-04-07. This change causes our CI setup to now run
macOS jobs on this shared runner instead of the reserved macOS VM Graphviz had
during the closed beta.

Note that this change also migrates these CI jobs from macOS 10.15.6 (Catalina)
to macOS 11.2 (Big Sur). With this, the XCode version is expected to move
forwards as well, though I do not know the precise version numbers.

Related to #1727.

  [0]: https://gitlab.com/gitlab-com/macos-buildcloud-runners-beta/-/issues/3#note_536852935

fix: install dependencies in macOS test tasks

These tasks were passing despite missing steps to install dependencies because
the current private beta macOS runner Graphviz uses is stateful. Moving to the
stateless shared macOS runner for the open beta exposes that testing fails
because the runtime dependencies of Graphviz are missing.

Merge branch 'smattr/02B780E5-FA0B-4FE5-94B8-9979AA32D4F4' into 'main'

maze.c warning squashing and clean up

See merge request graphviz/graphviz!1870

remove some commented out code

remove some unnecessary bracketing

remove some useless code

replace some unnecessary MIN calls with fmin

replace some unnecessary MAX calls with fmax

remove unused g parameter from markSmall

remove unused cp parameter from updateWt

squash some -Wunused-parameter warnings

Merge branch 'smattr/279B0FC2-0820-4378-A2DF-3A0EDB8B463E' into 'main'

fix: missing counting refactor and fatal errors in Pylint metrics

See merge request graphviz/graphviz!1867

fix: missing counting refactor and fatal errors in Pylint metrics

Merge branch 'smattr/CE58A1C5-3233-40B6-AB51-1094448247F0' into 'main'

replace Smyrna use of sfstropen with agxbuf

See merge request graphviz/graphviz!1868

replace Smyrna use of sfstropen with agxbuf

Related to #1873.

Merge branch 'grammar-align' into 'main'

docs: Remove deprecated HTML attributes in generation

See merge request graphviz/graphviz!1877

Remove deprecated HTML attributes in generation

Style the generated code with CSS instead.

Merge branch 'smattr/4D210743-7509-4642-AB03-14BCBC375538' into 'main'

clean up and warning squashing in lib/label

See merge request graphviz/graphviz!1866

placeLabels: move definition of i closer to its use

xlspdxload: move definition of op closer its use

xlhdxload: move definition of i closer to its use

xladjust: abbreviate some zeroing

The compiler understands a shorter memory-zeroing idiom.

xlintersections: move definitions closer to their use

aabbaabb: move definitions closer to their use

hd_hil_s_from_xy: move definitions closer to their use

xlnew: move definition of xlp closer to its use

PrintPVars: move definition of i closer to its use and use a better type

InitPVars: move definition of i closer to its use and use a bette type

LoadNodes: move definition of i closer to its use and use a better type

PickSeeds: move definitions closer to their use

MethodZero: move definition of i closer to its use and use a bette type

GetBranches: move definition of i closer to its use and use a better type

SplitNode: move definitions closer to their use and use better types

Contained: move definitions closer to their use an optimize algorithm

There's no need to track a result variable. We can just return early when we
know the answer.

Overlap: move definitions closer to their use and use better types

CombineRect: move definitions closer to their use and use better types

RectArea: move definitions closer to their use

PrintRect: move definition of i closer to its use and use a better type

NullRect: condense initialization of r

InitRect: move definition of i closer its use and use a better type

checkOpt: move definition of i closer to its use and use a better type

AddBranch: move definition of i closer to its use and use a better type

PickBranch: move definition of i closer to its use

NodeCover: move definitions closer to their use and use better types

PrintNode: move definition of i closer to its use and use a better type

InitNode: move definition of i closer to its use and use a better type

RTreeNewNode: move definition of n closer to its use

RTreeDelete2: move definition of i closer to its use

RTreeDelete: move definitions closer to their use and use better types

RTreeInsert2: move definition of i closer to its use

RTreeInsert: move definitions closer to their use and use better types

reflow RTreeInsert2 prototype

RTreeSearch: push definition of i closer to its use and use a better type

PrintData: push definition of i closer to its use and use a better type

PrintIndex: push definition of i closer to its use and use a better type

RTreeClose2: push definition of i closer to its use

RTreeNewIndex: push definition of x closer to its use

RTreeLeafListFree: push definition of tlp closer to its use

RTreeLeafListAdd: push nlp definition closer to its use

simplify printf format in PrintNode

Under C99, we have %p for pointers.

use more modern #pragma once guards in lib/label

remove unnecessary CoverAll declaration

This symbol is unused in this file.

remove commented out code

remove cryptic comment

I guess this is a hint to some editor I am not aware of.

remove unused parameter to RTreeReInsert

remove unused parameter to objplpmks

remove hand rolled floor(log2())

We do not need a custom optimized version of this. Modern compilers know these
functions as built-ins and can do their own better optimizations.

squash some -Wunused-parameter warnings

remove some open coded fmax

Merge branch 'smattr/450AD1C4-3E9D-4E93-95C6-279FC38EFCA3' into 'main'

some clean up of neatogen

See merge request graphviz/graphviz!1865

replace unnecessary max() implementation with libc fmaxf

No need to hand roll this ourselves now that we compile with C99.

replace neatogen header guards with more modern #pragma once

remove commented out code

remove C++ alternative in embed_graph.h

We only compile this header under C.

remove inline NULL definition

This is guaranteed to exist in C99.

remove C++ alternative in dijkstra.h

Only the C versions of these functions are implemented.

remove incorrect C++ alternative

The '&' is still required, regardless of whether this code is compiled with C or
C++. Note that this issue was latent as we never build this code as C++.

remove unused sparsegraph.h C++ alternative

Similar to previous commits, this code is unused because we do not include it in
any C++ compilation.

remove unused kkutils.h C++ alternative

This code is unused and in fact many of these prototyped functions are
unimplemented.

remove commented out inline keywords

remove some unnecessary __cplusplus guards

This file is C, and we build it as such. It is not expected that anyone will
compile this source as C++.

remove alternative C++ Queue implementation in neatogen

This code was written to provide two versions of the same API, one for C and one
for C++. This is unnecessary because the C API is already usable from C++.
Moreover, one can just use std::queue in C++. This code was not being used as we
only ever #include it from C files, so we just remove it to ease ongoing
maintenance.

remove commented out #include

Merge branch 'smattr/619aeafd-3505-4392-aa61-49cdae0077aa' into 'main'

fix: correctly handle HTML content ending in ]

Closes #1893

See merge request graphviz/graphviz!1859

fix: correctly handle HTML content ending in ]

As described in the source comment in this commit, expat attempts to recognize
CDATA section enders which results in some internal buffering that Graphviz did
not anticipate. Specifically, any HTML content ending in "]" or "]]" would be
parsed with these characters truncated. The next call to expat would flush this
buffered data, resulting in Graphviz believing the next content was malformed.

For example, the content "my label]]]" would be parsed as the text "my label]".
The next call to expat would return "]]" prepended to whatever was then being
parsed.

Fixes #1893.

add a test case for #1893

Merge branch 'smattr/F391EA36-8454-4A32-B815-1FAAEFD4006E' into 'main'

some cgraph clean up

See merge request graphviz/graphviz!1858

reflow some code

condense a malloc;memset

The calloc function does the zeroing for you and can even be more efficient for
large allocations.

use a stronger return type in not_default_attrs

use a stronger return type in has_no_predecessor_below

use a stronger return type in has_no_edges

use a stronger return type in write_node_test

use a stronger return type for node_in_subg

use a stronger return type in irrelevant_subgraph

use a stronger type for root variable

use a stronger type for hasName variable

simplify agcanonhtmlstr implementation

use a stronger type for backslash_pending variable

use a stronger type for needs_quotes variable