lib/expr/Makefile.am: remove some commented out lines

Merge branch 'smattr/68B86F65-F677-4E11-B60B-95C2114AAC79' into 'main'

remove some legacy CI steps

See merge request graphviz/graphviz!2132

remove /etc/redhat-release path in install script

This is the equivalent of 80ccebe66e71c6da570ea5a27f62c21d8c8e39aa but applied
to the install script. Every Linux distro Graphviz CI now runs on appears to
have an /etc/os-release, so this code is unused.

Revert "ensure ID_LIKE is always set on Debian in CI"

This reverts the remainder of commit 029acf11d1d9a9eb20eddae84cadb61447da95f3.
There are currently no configured Linux CI jobs that run outside of Docker
containers, so this work around is unnecessary.

Merge branch 'smattr/1B5CEE04-220F-4D0C-8A5C-0A16DD97F370' into 'main'

-Wfloat-equal based clean up

See merge request graphviz/graphviz!2138

glCompSetDraw: remove an unnecessary cast

This pointer implicitly coerces.

glCompSetNew: remove an open coded NULL

glcompset.c: remove excess white space

glCompSetMouseUp: remove unnecessary cast

This pointer coerces implicitly.

glCompSetMouseUp: remove pointless 'static' on locals

The value of these variables retained from prior calls is immediately
overwritten on entry to this function.

squarify: replace an exact floating-point comparison

In this case, `w` could be `fillrec.size[0]` or `fillrec.size[1]`. Replacing the
exact comparison with `<=` squashes a -Wfloat-equal compiled warning.

fullArea: remove micro-optimization of 0 value of 'm'

This is no longer necessary on modern compilers/processors where square root is
less expensive and this code is not on a hot path.

sparse cmp: remove some unnecessary casts

These pointers coerce implicitly.

sparse cmp: rephrase a comparison to avoid a -Wfloat-equal warning

Simply makes it clearer to the compiler what this code is doing.

overlapSeg: rephrase the order of comparisons to avoid a -Wfloat-equal warning

Simply makes it clearer to the compiler what this code is doing.

Merge branch 'smattr/24E8A935-AED9-4957-8832-B2EFADDDED5F' into 'main'

-Wmissing-field-initializer warning squashing

See merge request graphviz/graphviz!2139

p_tab: squash -Wmissing-field-initializer warning

p_note: squash -Wmissing-field-initializer warning

p_octagon: squash -Wmissing-field-initializer warning

p_septagon: squash -Wmissing-field-initializer warning

p_hexagon: squash -Wmissing-field-initializer warning

p_pentagon: squash -Wmissing-field-initializer warning

p_house: squash -Wmissing-field-initializer warning

p_parallelogram: squash -Wmissing-field-initializer warning

p_trapezium: squash -Wmissing-field-initializer warning

p_diamond: squash -Wmissing-field-initializer warning

Merge branch 'smattr/B7DB66F6-0A3B-4F6B-B334-2888ADA7BFBE' into 'main'

some graphml2gv, gxl2gv clean up

See merge request graphviz/graphviz!2141

gxl2gv: remove unnecessary casts of 'userdata' pointers

These implicitly coerce.

gxl2gv: remove unused struct name

characterDataHandler: remove unnecessary casts of `agxbput_n` arguments

graphml2gv: remove unnecessary casts of `userdata` pointers

These implicitly coerce.

graphml2gv: remove unused struct name

Merge branch 'smattr/bb628679-594c-4502-a50b-9e32e5ed7316' into 'main'

neatogen warning squashing

See merge request graphviz/graphviz!2146

neatogen multispline freeIpair: squash a -Wunused-parameter compiler warning

neatogen multispline newIpair: squash a -Wunused-parameter compiler warning

neatogen multispline freeItem: squash -Wunused-parameter compiler warnings

neatogen multipline newItem: squash a -Wunused-parameter compiler warning

neatogen multispline swap_ends_p: squash a -Wunused-parameter compiler warning

neatogen multispline spline_merge: squash a -Wunused-parameter compiler warning

neatogen multispline: squash a -Wmissing-field-initializer warning

makeObstacle: squash a spurious -Wswitch-default compiler warning

The surrounding context ensures that the cases in this switch cover every
possibility.

neatogen cmpitems: squash -Wunused-parameter compiler warnings

neatogen cmpitems: rephrase to avoid double→int conversion

This had a number of problems with respect to possible overflow and loss of
precision. This commit rephrases the comparison into something that does not
rely on any coercion or conversion.

neatogen freeitem: squash -Wunused-parameter compiler warnings

neatogen newitem: squash a -Wunused-parameter compiler warning

neatogen swap_ends_p: squash a -Wunused-parameter warning

neatogen spline_merge: squash a -Wunused-parameter compiler warning

find_closest_pairs: squash a -Wmissing-field-initializer compiler warning

neatogen splineInfo: squash a -Wmissing-field-initializer compiler warning

dotgen splineInfo: squash a -Wmissing-field-initializer compiler warning

Merge branch 'smattr/659C49B6-0A1A-4490-B1DD-DE326B38C850' into 'main'

clean up in tcldot

See merge request graphviz/graphviz!2145

tcldot-io.c: remove unnecessary bracketing

tcldot myiodisc_memiofread: remove unnecessary cast

This pointer implicitly coerces.

tcldot-io.c: remove commented out code

tcldot myiddisc_idregister: squash a spurious -Wswitch-default compiler warning

tcldot-id.c: remove unnecessary casts of state pointers

These pointers implicitly coerce.

tcldot: myiddisc_free: squash a -Wunused-parameter compiler warning

tcldot myiddisc_map: squash a -Wunused-parameter compiler warning

tcldot myagxset: use a 'size_t' for string length instead of 'int'

Squashes a -Wconversion compiler warning.

tcldot deleteNode: squash a -Wunused-parameter compiler warning

tcldot deleteEdge: squash a -Wunused-parameter compiler warning

tcldot obj2cmd: squash a spurious -Wswitch-default compiler warning

Tcldot_string_writer: remove unnecessary cast

This pointer implicitly coerces.

Merge branch 'smattr/DD0814CF-B2A0-4318-A2E6-A7F84616387A' into 'main'

fix remaining Pylint warnings; enabling gating Pylint CI check

See merge request graphviz/graphviz!2140

make final Pylint step fail for any non-FIXME warning/error

The gating Pylint check now fails if any warning or error is detected, excluding
FIXME comments. This should hopefully lead to improved code quality in Python
introduced into the code base in future.

disable Pylint duplicate-code warning

This triggers for a lot of Graphviz Python code because, by its nature of being
used to write test cases in Graphviz, there is inherently always going to be a
lot of similar looking code.

vuln.py: address a Pylint missing-function-docstring warning

shapes.py: address a Pylint missing-function-docstring warning

silence Pylint import-errors about regression_test_helpers

I do not know why, but Pylint does not understand this import while Python
itself does. I have tried different variations, but Pylint complains about all
of them.

regression_test_helpers.py: address a Pylint missing-function-docstring warning

vuln.py: remove variable shadowing

address Pylint missing-module-docstring warnings

test_regression.py: remove unused 'stdout' variable

chkPos: remove strcmp micro-optimization

Merge branch 'smattr/7890fb90-8b73-4677-a4f3-0f3a17dcf0ca' into 'main'

more SWIG bindings clean up

See merge request graphviz/graphviz!2147

OCaml bindings: fix: only pass -fpermissive when compiling C++

The `-fpermissive` flag is only valid for Objective-C and C++, but was being set
for C and C++. When compiling the C part of these bindings, this was generating
the warning:

  cc1: warning: command line option ‘-fpermissive’ is valid for C++/ObjC++ but
    not for C

This flag was introduced in 8f4667edb410a6d11b53746849304fb953b5c6ae to work
around non-conformant code generation by SWIG. In future, it might be worth
investigating whether the underlying issue has been fixed in SWIG and this work
around can be removed.

Java bindings gv_channel_writer: suppress -Wunused-parameter compiler warnings

Go bindings: suppress -Wunused-function compiler warnings

SWIG generates some functions that go unused in the Graphviz bindings. There is
no value to warning about these.

Merge branch 'smattr/ab033750-c4e7-4e83-930b-4c85fa29cdb3' into 'main'

BinaryHeap clean up

See merge request graphviz/graphviz!2148

BinaryHeap: switch 'mask' in sanity check to 0 vs 1

Using this instead of -1 vs 1 allows slightly more efficient code.

BinaryHeap: use 'size_t' to track heap sizes/positions instead of 'int'

BinaryHeap: use 'calloc' instead of 'malloc' when allocating arrays

There is not much we can do about the `realloc` calls, but we can at least do
initial allocation in a less error prone way.

BinaryHeap: rephrase 'sizeof(type)' code to 'sizeof(arr[0])'

This has no effect on functionality, but it makes it clearer what this code is
doing and makes it more robust against changing the underlying types.

BinaryHeap_realloc: rephrase division by 5 code

This makes the intent of this code clearer as well as squashing a
-Wfloat-conversion compiler warning.

Merge branch 'add-ubuntu21-04-static-build-job-to-CI' into 'main'

Add static build for Ubuntu 21.04 to CI

See merge request graphviz/graphviz!2134

CI: add ubuntu21-04-static-build job

CI: remove incorrect path to .gz file in deb build template

No .gz file is produced by the builds that use this template anyway.

ci/build.sh: add support for static builds using configure options

tclpkg: install .so files only when linking dynamically

Fixes errors like this when using static linking:

cp: cannot stat '/home/magjac/graphviz/build/lib/graphviz/ruby/libgv_ruby.so': No such file or directory

gvpack_static: force the C++ compiler to be used also for static linking

The trick to force the C++ compiler to be used for linking was applied
only for gvpack, not for gvpack_static.

Fixes many errors like this:

/usr/bin/ld: ../../lib/vpsc/.libs/libvpsc_C.a(csolve_VPSC.o): in function `newVariable':
csolve_VPSC.cpp:(.text+0x20): undefined reference to `operator new(unsigned long)'

Merge branch 'add-cygwin-cmake-build-to-ci' into 'main'

Add Cygwin CMake build to CI

See merge request graphviz/graphviz!2137

CI: add windows-cygwin-cmake-build job

CPack: silence CPACK_CYGWIN_PATCH_NUMBER not set warning

Silences this warning:

CPack Warning: CPACK_CYGWIN_PATCH_NUMBER not specified using 1

CPack: generate Cygwin binary installer on Cygwin

tools: add workaround for cpack problem with Cygwin

Copy the gml2gv executable to to the gv2gml alias instead of
symlinking it. Creating a symlink to gml2gv works fine in itself, but
results in an error like this when running cpack:

CPack Error: Problem while adding file </home/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml> to archive </home/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN.zip>, ERROR = Unable to read from file 'Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml': Couldn't list extended attributes

gv2gml is a symlink. Trying to read the extended attributes manually gives:

attr -l /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml
attr_list: Permission denied
Could not list /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.1~dev.20210902.2049-CYGWIN/bin/gv2gml

dot: add workaround for cpack problem with Cygwin

Copy the dot executable to its alias commands instead of symlinking
it. Creating symlinks to dot works fine in itself, but results in an
error like this when running cpack:

CPack Error: Problem while adding file </cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo> to archive </cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN.zip>, ERROR = Unable to read from file 'Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo': Couldn't list extended attributes

circo is a symlink. Trying to read the extended attributes manually gives:

attr -l /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo
attr_list: Permission denied
Could not list /cygdrive/c/Users/magja/graphviz/build/_CPack_Packages/CYGWIN/ZIP/Graphviz-2.49.0~dev.20210812.2025-CYGWIN/bin/circo

add ci/cygwin-build.sh script

ci/build.sh: remove trailing text after actual version ID

Fixes a problem on Cygwin where 'uname -r' gives:

3.2.0(0.340/5/3)

which, when used as the name of a directory, actually represents a
directory named '3.2.0(0.340', containing a subdirectory named '5'
containing a subdirectory named '3)'.

ci/build.sh: add support for Cygwin

The /etc/os-release file does not exist on Cygwin so we use the more
reliable built-in bash variable OSTYPE instead.

plugin/core: make awk crlf agnostic when generating ps.h

Fixes errors like this with Cygwin in a crlf environment with
SHELLOPTS=igncr:

/cygdrive/c/Users/magja/graphviz/build/plugin/core/ps.h:2:1: error: missing terminating " character
    2 | "%%BeginProlog ",
      | ^~~~~~~~~~~~~~

Although not shown in the error message, this line actually contained
a carriage return after %%BeginProlog:

"%%BeginProlog\r",