remove work around for stateful macOS CI runners

Graphviz was part of a closed beta program for Gitlab macOS CI runners.¹ During
the beta period, Graphviz had access to a single macOS VM environment that was
stateful. That is, the effects of any macOS CI task would persist to the next
macOS CI task. To work around this, logic was introduced to manually remove the
Graphviz installed in the macOS CI environment by the prior run.

The Gitlab program will transition into open beta on 2021-05-22, however the
Graphviz configuration was already migrated to the open beta model on
2021-04-07.² This means the macOS runner are no longer stateful:

  To recap, for the closed beta, we provisioned a static macOS virtual machine
  for each project. This meant that we executed any pipeline job in your
  associated repository on the same virtual machine each time. Since the VM was
  dedicated to your project, this also meant that your pipeline job did not have
  to wait in a queue before running.

  For the open-beta, each job that requires a macOS build environment will run
  in its own ephemeral virtual machine provisioned on demand by the GitLab
  Runner macOS autoscaler. This means that you can’t rely on persistent storage
  anymore.

This change removes the work around that is no longer needed. This reverts
62165551efb389ea14a6709ffd89b2b5ed79e35c.

  ¹ https://gitlab.com/gitlab-com/macos-buildcloud-runners-beta/-/issues/3
  ² https://gitlab.com/gitlab-com/macos-buildcloud-runners-beta/-/issues/3#note_536852935

Merge branch 'smattr/dc82b281-8d6d-492b-b2df-62c89447711e' into 'main'

more warning squashing

See merge request graphviz/graphviz!1939

squash some -Wfloat-conversion warnings

If we need -0.1 and -0.2 as floats (or the closest representation), we should
just say so. This improves the compiler’s ability to understand the intent of
this code.

use float square root function withing getEdgeLength

Like prior commits in this series, this avoids using the double function with
float arguments.

more explicit float computation in hsv2rgb

Instead of being vague with ints, doubles, and floats, this commit standardizes
on float literals in hsv2rgb. This squashes a number of -Wfloat-conversion
compiler warnings.

use hypotf to squash a -Wfloat-conversion warning

For the same reason as the previous commit, this fixes an intermingling of
doubles and floats, and also abbreviates code.

use floating point math functions to squash a -Wfloat-conversion warning

The function sqrt operates on doubles, which was triggering a -Wfloat-conversion
compiler warning. The float version of this function is sqrtf, but we can
further abbreviate this operation by observing it is computing the hypotenuse of
a right-angled triangle and use the library function for that instead.

use a float literal to squash a -Wfloat-conversion warning

avoid using double literals with a float variable

With -Wfloat-conversion, the compiler warns that going via double literals like
this resulted in an imprecise float. To squash this warning and be more precise,
we can use float literals instead.

remove use of double literals with ydelta in make_flat_labeled_edge

Commit ebe7517ae4eb86ee4bf34e1af31a174f1f0a4049 altered these lines to use
double literals instead of integer literals for `5` and `6`. It is not clear why
it did this as ydelta is an integer variable. The effect of that change was to
ask the compiler to perform the computation itself using doubles and then reduce
it to an integer result. This commit reverts that part of the diff to simply use
integer computation throughout. This change was motivated by the compiler
diagnosing this issue with -Wfloat-conversion.

remove some commented out code

squash a -Wfloat-conversion due to sqrt in distBetweenPts

There is a dedicated libc function for doing square root of floats, but this
code was incorrectly calling the version for doubles instead. This updated code
is more appropriate and could even be more efficient.

squash some -Wfloat-conversion warnings in lib/glcomp

These literals were doubles, but being implicitly converted to floats. This
resulted in precision loss. E.g. the compiler diagnoses:

  conversion from ‘double’ to ‘GLfloat’ {aka ‘float’} changes value from
  ‘-1.0e-3’ to ‘-1.00000005e-3’

By writing these as float literals to begin with, we can get a more precise
value.

mark interpol function as static

This is not used outside of its containing file.

remove unused interpol field in Smyrna

mark various v* functions in Smyrna as static

These are not used outside of their containing file.

remove unused frmObject_set_scroll function

mark GFXarrayresizecb as static

This function is not used outside of its containing file.

remove unused mm_read_mtx_crd function

mark compBB as static

This function is not used outside of its containing file.

remove some commented out code

mark attr_value_edited_cb functions as static

Both copies of this function are only used within their containing file.

remove unused Operator_uniform_stress_matmul_delete function

remove unused get_level function

remove unused colorType function

remove unused exlexname

This function is uncalled and not exposed to users.

remove unused Pobsbarriers function

Merge branch 'smattr/318A4F88-562E-4495-B0EE-41385C9F66AE' into 'main'

tidy up gen_version.py and squash Pylint warnings

See merge request graphviz/graphviz!1954

squash missing-module-docstring Pylint warning in gen_version

squash Pylint import warnings in gen_version

Merge branch 'smattr/8012B430-752A-41A9-9F70-01AC86CA1FB3' into 'main'

clean up of edgeType()

See merge request graphviz/graphviz!1952

more precisely indicate the argument to edgeType is not modified

make edgeType() static

This function is not used outside of lib/common/utils.c and utils.h is not a
shipped header.

simplify edgeType()

This change avoids some micro-optimizations that were unnecessary and making
this function harder to read. The function’s behavior is intended to be
identical after this change.

Merge branch 'smattr/C44CD3B6-6BBA-4D56-BB6F-5BACB7265AD4' into 'main'

fix: remove dangling reference to textfont.* in Autotools file

See merge request graphviz/graphviz!1953

fix: remove dangling reference to textfont.* in Autotools file

The file textfont.c was renamed to textspan.c in
65f337664d177b588c5c2418fe12bac7eef62c2a but this rule was incorrectly not
updated. Rather than relying on manual dependencies which the Autotools docs
suggest can be problematic,¹ we just remove this rule. This removes the ability
to `make textspan.o`, but clearly no one was relying on this anyway as it has
been broken since 2013.

¹ https://www.gnu.org/software/automake/manual/html_node/Built-Sources-Example.html

Merge branch 'smattr/7cf525be-5d5f-4ed7-9982-44399733c868' into 'main'

more lib/vpsc clean up and modernization

See merge request graphviz/graphviz!1950

manage graph collection in VPSC::blockGraphIsCyclic with smart pointers

This avoids some manual memory management.

remove redundant conditional clause

This is always known-true due to the condition on the containing loop.

use a range-based for loop to abbreviate some code in VPSC::blockGraphIsCyclic

manage graph collection in VPSC::constraintGraphIsCyclic with smart pointers

Removes some manual memory management.

use range-based for loops to abbreviate some code in constraintGraphIsCyclic

remove dead conditional clause

This code is only reachable when this condition is already known-true, due to
the containing loop.

use a range-based for loop for iterating over block total order collection

use a range-based for loop for iterating over VPSC::inactive

manage VPSC::bs as a value instead of pointer

This leads to simpler code and less manual memory management.

use range-based for loops to abbreviate some iteration over block collections

Merge branch 'smattr/FD618CD5-B7F3-40D0-96D9-1F9ED031AA1F' into 'main'

narrow dependencies of Docker-push tasks

See merge request graphviz/graphviz!1949

narrow dependencies of Docker-push tasks

The Docker push jobs were depending on all prior jobs which was overly
restrictive. Each push job only needs the image it itself needs to push. Closes
#1988.

Merge branch 'smattr/C36AA7A4-42E6-44FE-B678-7949BA77BD24' into 'main'

more warning squashing and clean up in lib/ortho and lib/sparse

See merge request graphviz/graphviz!1917

replace has_position boolean with a C99 bool

replace first boolean with a C99 bool

replace noclusterinfo boolean with a C99 bool

replace boolean has_positions with a C99 bool

replace boolean first with a C99 bool

use itos to remove an intermediate buffer

remove dead writes to buf

It's unclear what the intention of this code was. It writes data to buf that is
then never read.

abbreviate construction of label_string

stack-allocate a statically sized buffer

1000 bytes is not a lot to allocate on the stack on a modern platform.

remove useless snprintf

Unclear what the purpose of this was when the content of buf is overwritten
later before being read.

abbreviate some manual strdup code

These locations were doing the exact functionality of strdup, but using manual
malloc. It is more readable to just call the library function for this.

remove commented out code

squash -Wshadow warning in Import_coord_clusters_from_dot

Similar to a prior change, there was a nested variable also called sz. By
removing this sz, we not only squash a compiler warning but make the code more
readable.

remove unused cat_string

squash a -Wshadow warning in SparseMatrix_import_dot

By moving sz closer to its use location it is moved passed a nested declaration
of a different variable also called sz.

abbreviate an increment

This leads to more readable and less error prone code.

squash a -Wfloat-conversion

This was an odd way to write division by 5.

squash a -Wimplicit-fallthrough warning

remove unused SparseMatrix_to_complex

squash a number of -Wconversion warnings

The fread function returns a size_t, so Graphviz should be using a size_t
variable to store its result.

squash a -Wmaybe-uninitiailzed warning

This was a false positive because all use of the variable is guarded by the
Concentrate flag, but it is still worth pacifying the compiler here.

fix: return -2 on error in seg_cmp and remove now unused longjmp path

Related to #1801. This removes two (accurate) warnings about possible clobbered
variables in orthoEdges.

anticipate a -2 return from seg_cmp indicating an error

Related to #1801.

anticipate failure from decide_point

Related to #1801.

change calling convention of decide_point

Related to #1801.

anticipate failure from addPEdges

Related to #1801.

add a return value to addPEdges

Related to #1801.

anticipate failure from add_p_edges

Related to #1801.

add a return vaule to add_p_edges

Related to #1801.

anticipate failure from add_edges_in_G

Related to #1801.

add a return value to add_edges_in_G

Related to #1801.

anticipate failure from add_np_edges

Related to #1801.

add a return value to add_np_edges

Related to #1801.

anticipate failure from assignTracks

Related to #1801.

add a return value to assignTracks

Related to #1801.

squash two -Wsign-compare warnings

The iterations modified here are iterating over a collection whose size is
tracked using int fields, ngcells and ncells. Both these should really be size_t
but this is complicated to change at this point. As long as they are int, using
int loop counters provides greater consistency.

squash a -Wsign-conversion warning

The npts variable is used to store an allocation quantity, so should have been
using the size_t scalar type.

Merge branch 'add-clang-format-file' into 'main'

add .clang-format file

See merge request graphviz/graphviz!1948

add .clang-format file

Generated with clang-format-12 --style=llvm --dump-config >.clang-format

Merge branch 'smattr/668B61F4-EDF3-4468-A872-715B28A34C37' into 'main'

output source data for Linux downloads on the website during deployment

See merge request graphviz/graphviz!1947

output source data for Linux downloads on the website during deployment

Commit 1b218752f5b6a1e661e158337fc1b976424fc6a0 of the website repository¹
changed how Linux downloads appear on the website to be driven by a JSON data
file. This commit updates the deployment script to produce this file for a
release as a CI job artifact. Related to #1979.

¹ https://gitlab.com/graphviz/graphviz.gitlab.io

Merge branch 'smattr/747bd6bd-fdc6-4847-b646-226c03399987' into 'main'

replace some hypotenuse calculations with calls to hypot

See merge request graphviz/graphviz!1945

replace hypotenuse calculations in bend with calls to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation in computeoffset_qr with a call to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation in estimateError with a call to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation in arrow_gen with a call to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation with a call to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation arrow_type_dot with a call to hypot

The function hypot is available in C99 and has the potential to compute the same
operation more efficiently and with greater precision.

replace hypotenuse calculation in tb_project_to_sphere with a call to hypotf

The function hypotf is available in C99 and has the potential to compute the
same operation more efficiently and with greater precision.

remove no-op adjustclip function in gcanvas.c