Merge branch 'smattr/0F3EB13A-FB8F-4B18-BA3B-2C77172988E0' into 'main'

Stable Release 2.47.2

See merge request graphviz/graphviz!1963

Stable Release 2.47.2

Merge branch 'dll-data' into 'main'

sfio: remove uwin specific handling

See merge request graphviz/graphviz!1959

sfio: remove uwin specific handling

The indirection of the data through the IAT is implicitly handled by
modern linkers and compilers when the data symbol is annotated with
`__declspec(dll{import,export})`.  This removes the support for the
older toolchains as part of the work to properly annotate symbols for
improvements to the Windows port.

Merge branch 'smattr/7B3F1422-4C87-4B72-8CBB-F395FF5AAB87' into 'main'

optimize 0 case in xdot_fmt_num

See merge request graphviz/graphviz!1958

optimize 0 case in xdot_fmt_num

Commit 894e94c66340187ff6f8ea97c9336ea048a3d6ad applied a performance
optimization to gvprintdouble that was contributing to a performance issue in a
user’s graph. The code touched in this commit has the same pattern as that in
gvprintdouble, and the same corresponding (latent) performance issue. While it
is not known to cause any problems currently, we may as well proactively apply
the same optimization here.

This preserves the same functionality, but simply takes an early exit from the
function when we already know precisely how to print the resulting string.

Merge branch 'fix-msvc-linkage-errors' into 'main'

Remove MSVC linkage warnings

See merge request graphviz/graphviz!1931

Remove duplicated declarations in smyrna topviewfuncs.h

Correct MSVC storage-class in gvpr.h. Removes one warning

Only set __declspec(dllexport) or __declspec(dllimport) when the gvpr
library is dynamically linked. This is defined by GVDLL which is set
in the Windows MSBuild builds which use dynamic linking, but unset in
the Windows CMake builds which use static linking.

Removes the following warning from the Windows CMake builds:

C:\GitLab-Runner\builds\graphviz\graphviz\lib\gvpr\gvpr.c(931,1): warning C4273: 'gvpr': inconsistent dll linkage [C:\GitLab-Runner\builds\graphviz\graphviz\build\lib\gvpr\gvpr.vcxproj]

Add missing GVDLL define in lib/gvpr Windows MSBuild Debug build

GVDLL is used by the autotools and MSBuild build systems to specify
dynamic linking on Windows.

Correct MSVC storage-class in geomprocs.h. Removes 19 warnings

Avoid setting __declspec(dllimport) when gvc is being exported.

Removes the following warnings from the Windows CMake builds:

C:\Users\magja\graphviz\lib\common\geom.c(20,1): warning C4273: 'mkbox': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(41,1): warning C4273: 'mkboxf': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(84,1): warning C4273: 'lineToBox': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(194,1): warning C4273: 'rect2poly': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(229,1): warning C4273: 'cwrotatep': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(257,1): warning C4273: 'cwrotatepf': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(285,1): warning C4273: 'ccwrotatep': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(313,1): warning C4273: 'ccwrotatepf': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(341,1): warning C4273: 'flip_rec_box': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(357,1): warning C4273: 'flip_rec_boxf': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(378,1): warning C4273: 'ptToLine2': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
C:\Users\magja\graphviz\lib\common\geom.c(395,1): warning C4273: 'line_intersect': inconsistent dll linkage [C:\Users\magja\graphviz\build\lib\common\common_obj.vcxproj]
LINK : warning LNK4217: symbol 'ptToLine2' defined in 'geom.obj' is imported by 'emit.obj' in function 'check_control_points' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4217: symbol 'rect2poly' defined in 'geom.obj' is imported by 'emit.obj' in function 'emit_map_rect' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4217: symbol 'ccwrotatepf' defined in 'geom.obj' is imported by 'postproc.obj' in function 'gv_postprocess' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4286: symbol 'ccwrotatepf' defined in 'geom.obj' is imported by 'shapes.obj' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4217: symbol 'flip_rec_boxf' defined in 'geom.obj' is imported by 'shapes.obj' in function 'record_path' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4217: symbol 'cwrotatepf' defined in 'geom.obj' is imported by 'shapes.obj' in function 'compassPoint' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]
LINK : warning LNK4217: symbol 'lineToBox' defined in 'geom.obj' is imported by 'utils.obj' in function 'overlap_bezier' [C:\Users\magja\graphviz\build\lib\gvc\gvc.vcxproj]

Merge branch 'smattr/5EE54780-9173-406B-8B95-DC903AD46E4F' into 'main'

remove diff and grep as Windows build and test dependencies

Closes #2069

See merge request graphviz/graphviz!1957

remove Windows CI check for diff

Following the prior changes, diff is no longer a test suite dependency on
Windows. Closes #2069.

replace use of diff in rtest with filecmp

This is a step towards removing diff as a test dependency, so Windows users do
not need to install it. Related to #2069.

replace use of diff in test_reference with difflib

This is a step towards removing diff as a test dependency, so Windows users do
not need to install it. Related to #2069.

remove Windows CI check for grep

Following the prior changes, grep is no longer a build dependency on Windows.
Related to #2069.

replace use of grep in Windows CI task with Select-String

This warning counting can be achieved without the use of grep. This is a step
towards removing grep as a Windows build dependency. Related to #2069.

replace use of grep in Windows build utility setup with Select-String

This is a built-in function in PowerShell with equivalent functionality for this
purpose. This change is a step towards removing grep as a build dependency on
Windows. Related to #2069.

Merge branch 'smattr/4B36E297-A6E1-4C9A-A3AE-9D765264D73C' into 'main'

some clean up in cmd/gvedit

See merge request graphviz/graphviz!1956

replace header guards in gvedit with more modern #pragma once

remove commented out code in gvedit

use isEmpty in preference to checking length against 0 in gvedit

Qt QStrings have two methods, isEmpty and length, analogous to empty and length
on std::string, respectively. Checking isEmpty is equivalent to checking length
against 0, is clearer, and has the potential to be more performant.

Merge branch 'smattr/A7790D85-5D41-49CD-A9BC-38AE5FA3B570' into 'main'

remove work around for stateful macOS CI runners

See merge request graphviz/graphviz!1955

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.