remove prototype of unimplemented 'cmpNodeDegree'

remove prototypes of unimplemented ag callback functions

remove prototypes of unimplemented object set management functions

remove declaration of undefined 'AgAttrdisc'

remove unused 'NIL macros

Note that copies of these still exist in the shipped cgraph.h.

remove C++ guards on hashkey.h

These are no longer necessary now that this header contains no function
prototypes.

remove prototype for unimplemented 'strkey'

remove prototype for unimplemented 'html_inside'

remove unused 'colorCvt'

Merge branch 'smattr/2D1C5E3F-6FEC-4BF0-9291-DD8A9DBD3C12' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2344

constrained_majorization_new_with_gaps: [nfc] boolean → C99 bool

constrained_majorization_gradient_projection: [nfc] another boolean → C99 bool

constrained_majorization_gradient_projection: [nfc] boolean → C99 bool

constrained_majorization_new: [nfc] use a C99 bool instead of a boolean

power_iteration/iterativePCA_1D: [nfc] return C99 bools instead of boolean

stress_majorization_with_hierarchy: [nfc] replace a boolean with a C99 bool

user_spline: [nfc] use a C99 bool instead of boolean for 'warned'

gvconfig_libdir: [nfc] use a C99 bool instead of boolean for 'dirShown'

gvpack: [nfc] use a C99 bool instead of boolean for 'doPack'

gvpack cloneGraph: [nfc] replace a boolean with a C99 bool

bend: [nfc] remove unnecessary parens

limitBoxes: [nfc] abbreviation some accumulation

Merge branch 'smattr/03D71147-1B60-4F43-83A3-9DD0DB36BFFC' into 'main'

mingle: fix mismatched format string to 'agerr'

See merge request graphviz/graphviz!2330

mingle: fix mismatched format string to 'agerr'

Based on line 395 of this same file, it is clear this is what this code was
intended to look like. It is unclear to me why this was not caught by the GNU
`printf` annotations on `agerr`.

Merge branch 'smattr/271D49D3-BD64-4100-9DED-91D288393801' into 'main'

makeStraightEdge: remove array allocation optimization

See merge request graphviz/graphviz!2342

makeStraightEdge: remove array allocation optimization

This code had two alternative paths, one that used a heap-allocated array for
edge counts above 20 and one that used a stack-allocated array for edge counts
of 20 or less. The latter was an optimization to avoid heap allocation for
smaller graphs. Modern dynamic allocators are fast enough that this kind of
optimization is no longer relevant.

Merge branch 'smattr/BCD82356-CBC7-4DAA-9AA9-4FE6D9B9A55F' into 'main'

upgrade Pylint

See merge request graphviz/graphviz!2339

simplify test_1855 call to 'dot'

Using the wrapper added in 297ed0bc7b61b2a6616c335726334836c8ffcc10, we can
write this code more simply and concisely.

upgrade Pylint

use Python context manager idiom when reading and writing files

Newer versions of Pylint suggest this.

use Python 'subprocess.Popen' as a context manager

Newer versions of Pylint suggest this.

rtest.py: remove use of 'f3' as a global

This allows us to convert the use of this file to a more idiomatic `with` block.

remove an unnecessary Pylint 'global' statement

Newer versions of Pylint identify this.

specify an explicit mode and UTF-8 encoding in Python open calls with text mode

Newer versions of Pylint warn about this.

explicitly specify latin-1 encoding when dealing with PS files in tests

PostScript files are encoded in a character set called CCSID 1276.¹ This code
was working despite not specifying an encoding because the default encoding on
each platform coincidentally lines up with that used in their sample files. More
recent versions of Pylint warn that encoding should always be specified in
`open` calls, and attempting to use either ASCII or UTF-8 encoding fails. The
Linux sample files contain invalid ASCII bytes and the Windows sample files
contain invalid UTF-8 bytes. Using latin-1, which seems to be the closest
available encoding to CCSID 1276, works cross platform.

In future, this code should perhaps be adapted to do I/O in binary mode, thus
avoiding any encoding concerns.

¹ https://en.wikipedia.org/wiki/PostScript_Standard_Encoding

convert some legacy Python '.format' and string addition to f-strings

Newer versions of Pylint warn about this.

use 'run' instead of 'Popen' where more concise in test code

Not only does this lead to more concise and efficient code, this avoids some
upcoming warnings in newer Pylint releases.

Merge branch 'smattr/7042B009-FC97-470F-9196-E28ED6BEB967' into 'main'

add Fedora 35 testing in CI

See merge request graphviz/graphviz!2340

add Fedora 35 testing in CI

Related to #2163.

remove Fedora 33 Docker file

2a70afbdf049d5aa93470dcad5cf03c60841b6b3 removed used of this file and should
have removed the file itself as well.

Merge branch 'remove-lab-gamut-def-file' into 'main'

Remove lab_gamut.def file

See merge request graphviz/graphviz!2337

lib/edgepaint: remove obsolete lab_gamut.def file

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

lib/edgepaint: add support for the dllexport storage-class attribute in lab_gamut.h

Also, storage-class attributes are now added only when the GVDLL
symbol is defined.

This removes the need to use a .def file.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

lib/edgepaint: MSBuild: define GVDLL in edgepaintlib project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will continute to be
declared with the dllimport attribute.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

lib/edgepaint: MSBuild: define GVDLL in lab_gamut project file

An upcoming commit will change lab_gamut.h to use the GVDLL symbol to
enable storage-class attributes.

This change ensures that the lab_gamut symbols will be defined with
the dllexport attribute, which will eliminate the need for a .def
file.

Towards https://gitlab.com/graphviz/graphviz/-/issues/2058.

Merge MR !2331

CMake: propagate include flag for harfbuzz on Windows

Pango >=1.44.1 changed the way harfbuzz is included
xref: https://github.com/GNOME/pango/commit/a7b17aa2

This is not required on *nix because the PkgConfig CMake module
already takes care of this. For example, on Ubuntu it adds:
-I/usr/include/harbuzz to PangoCairo_CFLAGS

Merge branch 'smattr/A6C297DD-D456-46E0-99BD-E1043C7F5AB1' into 'main'

ci: standardize shell scripts on the same e,u,x,o options

See merge request graphviz/graphviz!2336

ci: standardize shell scripts on the same e,u,x,o options

All these scripts have the same expectation, of failing on anything unexpected.
So setting the same options the same way everywhere leads to more consistency
and predictability.

ci: apply '${ID_LIKE:-}' pattern from build script to install

This allows in future enabling `set -u` in this script.

Merge branch 'smattr/4DE85C4D-850E-457B-9C7E-A5F7767BFFAE' into 'main'

disable Windows Defender in all CI jobs

See merge request graphviz/graphviz!2334

disable Windows Defender in all CI jobs

Windows Defender significantly degrades I/O and slows CI tasks to the point that
it is difficult to get a passing result on many of the Windows jobs. It is
unclear what value Windows Defender is adding in CI when the entire environment
is transient and malware has no obvious way of gaining persistence. So far, the
main effect of Windows Defender has been to spuriously block MinGW tools,
preventing CI running altogether.

Merge branch 'smattr/2756CD40-47ED-43DD-A9D7-825137573F7C' into 'main'

drop Fedora 33 builds from CI

See merge request graphviz/graphviz!2332

drop Fedora 33 builds from CI

Fedora 33 was EOLed on 2021-11-30.¹

¹ https://docs.fedoraproject.org/en-US/releases/eol/

Merge branch 'smattr/B6B64825-8DDD-4591-98B8-B063F8DC4D69' into 'main'

some Python clean up and simplification

See merge request graphviz/graphviz!2328

tests: avoid a manual open of /dev/null

tests: rewrite 'Popen;communicate' code to use 'check_output'

When this testing code was first written, it looks like it was overlooked that
`check_output` supports the `input` parameter from Python 3.4 onwards. With
this, the relevant code can be written more concisely.

tests: abstract running 'dot' and checking its return code into library function

This makes the common pattern of “run `dot`, checking its exit status, and give
me the resulting output” more concise and readable in the test suite.

tests: rename 'dot' variables in test_regression.py

An upcoming change will introduce a `dot` function, which these variables will
then shadow, causing warnings. This is a pre-emptive change to avoid this.

Merge branch 'SVG-viewport-precision' into 'main'

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1855 where the SVG view...

Closes #1855

See merge request graphviz/graphviz!2327

add a changelog entry for the prior fix

Fixes https://gitlab.com/graphviz/graphviz/-/issues/1855 where the SVG view setup needs more than two digits of precision, otherwise clipping may result.

add a test case for scale precision issue

Merge branch 'smattr/7740732D-AFA2-4734-9DFE-051A94B5EE68' into 'main'

remove commented out code

See merge request graphviz/graphviz!2326

tclpkg/gdtclft/gdtclft.c: [nfc] remove commented out code

plugin.demo/xgtk/src/callbacks.c: [nfc] remove commented out code

plugin/gd/gvrender_gd.c: [nfc] remove commented out code

plugin/gd/gvloadimage_gd.c: [nfc] remove commented out code

plugin/gdk/gvloadimage_gdk.c: [nfc] remove commented out code

cmd/lefty/ws/mswin32/gtext.c: [nfc] remove commented out code

Merge branch 'smattr/b524e557-b4bb-48fb-a6b9-4b082a913219' into 'main'

more boolean → bool modernization

See merge request graphviz/graphviz!2325

poly_init: [nfc] use a C99 bool for 'isPlain' instead of boolean

lib/common/labels.c: [nfc] remove unnecessary parens

strdup_and_subst_obj0: [nfc] use a C99 bool for 'has_tp' instead of boolean

strdup_and_subst_obj0: [nfc] use a C99 bool for 'has_hp' instead of boolean

dot: fix declaration of 'MemTest'

This variable, declared in globals.h is an `int`, not a `boolean` (a.k.a.
`unsigned char`).

lib/neatogen/conjgrad.c: [nfc] remove a commented out #include

conjugate_gradient_f: [nfc] take a C99 bool parameter instead of a boolean

expand_t.doAdd: [nfc] use a C99 bool instead of a boolean

samedir: fix return type

Commit e1181697185289594304cfb22d5979d1fc5cbf0f altered this function to use C99
bools but did not update its return type.

htmlEntityUTF8: [nfc] use a C99 bool for 'warned' local instead of boolean

Merge branch 'smattr/D5394DF4-047C-45E9-B4B4-400B6140BE96' into 'main'

ci: [nfc] fuse tarball;mv steps into one

See merge request graphviz/graphviz!2324

ci: [nfc] fuse tarball;mv steps into one

When we are immediately going to the move a created tarball to a new location,
we may as well just save time and create it in the eventual location to begin
with.

Merge branch 'smattr/8EC56F87-69CE-47A9-8317-CBB835E23A02' into 'main'

remove Ming plugin

Closes #2160

See merge request graphviz/graphviz!2323

remove Ming plugin

The format this plugin produces, Adobe Shockwave, was officially EOLed by Adobe
in April 2019.¹ Legacy support for SWF files was available through Adobe Flash
Player. Flash Player was subsequently EOLed in December 2020 with Adobe
considering it such a security risk they flipped a kill switch in January 2021
making it no longer possible to run Flash Player anywhere.²

Closes #2160

¹ https://helpx.adobe.com/shockwave/shockwave-end-of-life-faq.html
² https://www.adobe.com/au/products/flashplayer/end-of-life.html

Merge branch 'smattr/35B3A0E0-A5D9-43EB-9837-79A1DD8788C8' into 'main'

[nfc] clean up legacy lines in Windows config.h

See merge request graphviz/graphviz!2322

[nfc] remove legacy libc header line in Windows config.h

The last use of this macro was removed in
70a4ec52014697210369c6c66c9af7d72579fdc9. Graphviz has been compiled with C99
since fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, under which standard C headers
are required to exist.

[nfc] remove legacy 'STACK_DIRECTION' lines in Windows config.h

This constant was introduced in the initial addition of the precreated Windows
config.h, 76ead5c74029a5ceea61dd6c7600dc1931c275cf, but it is unclear where it
originates from. None of the other build systems appear to define this and no
prior or current state of the Graphviz tree has ever contained use of this
macro. Graphviz does not use `alloca`.

[nfc] remove legacy sys/dir.h line from Windows config.h

The last use of sys/dir.h was removed in
0330a4d918dcbf6edb1ce46956a20bde9a2444f9.

[nfc] remove legacy memory.h line in Windows config.h

The last use of this macro was removed in
5cbd4ebcd8254e51569b7209cb964f7d260319fb.

[nfc] remove legacy NDIR lines in Windows config.h

The only use of these defines and ndir.h headers was removed in
0330a4d918dcbf6edb1ce46956a20bde9a2444f9.

Merge branch 'smattr/8B6E8ECF-9BB8-4D71-8F5E-EDA5F3EC7695' into 'main'

fix: correct Windows release file recognition during deployment

Closes #1979

See merge request graphviz/graphviz!2321

fix: correct Windows release file recognition during deployment

6117abe680037824d134149b0de42f589fb24466 updated the deployment script to add
Windows artifacts that previously needed to be appended locally to the JSON
manifest. However it failed to account for changes made to the packaging
hierarchy that removed release collection and a redundant architecture
component.

(Hopefully) closes #1979.

Merge branch 'msys2-ci-fix' into 'main'

CI: Add C:\tools\msys64 to MS Defender exclusion list

See merge request graphviz/graphviz!2333

CI: Add C:\tools\msys64 to MS Defender exclusion list

This is a workaround for:

    cc.exe: fatal error: cannot execute
    'C:/tools/msys64/path/to/cc1.exe':
    CreateProcess: No such file or directory

Merge branch 'smattr/EF4D0E71-8197-4826-862C-DD92EB96C820' into 'main'

install and use Ccache in Cygwin CI tasks

See merge request graphviz/graphviz!2303

README: fix unescaped '&'

enable persistent Ccache cache in Cygwin CI tasks

This allows cached compilation results to be preserved and reused across CI
runs. It is hoped this will accelerate these currently long running tasks.

install and use Ccache in Cygwin CI tasks

The Cygwin CI tasks come very close to the CI timeout ceiling, usually running
for close to an hour. It is hoped that this change (and upcoming caching) will
reduce this time, thus shortening the total CI time as these are the longest
running tasks at present.

Merge branch 'add-mingw-autotools-builds-to-ci-take3' into 'main'

Add MinGW autotools build jobs to CI

See merge request graphviz/graphviz!2291